quiet msvc/mingw compiler warnings.
[blender.git] / source / blender / blenlib / intern / path_util.c
index a496e6c..06dc510 100644 (file)
@@ -1,4 +1,4 @@
-/**
+/*
  *  $Id$
  *
  * ***** BEGIN GPL LICENSE BLOCK *****
  * various string, file, list operations.
  */
 
+/** \file blender/blenlib/intern/path_util.c
+ *  \ingroup bli
+ */
+
+
 #include <ctype.h>
 #include <string.h>
 #include <stdlib.h>
@@ -43,6 +48,7 @@
 #include "BLI_string.h"
 #include "BLI_storage.h"
 #include "BLI_storage_types.h"
+#include "BLI_utildefines.h"
 
 #include "BKE_utildefines.h"
 #include "BKE_blender.h"       // BLENDER_VERSION
 #if defined WIN32 && !defined _LIBC
 # include "BLI_fnmatch.h" /* use fnmatch included in blenlib */
 #else
-# define _GNU_SOURCE
-# include <fnmatch.h>
+#  ifndef _GNU_SOURCE
+#    define _GNU_SOURCE
+#  endif
+#  include <fnmatch.h>
 #endif
 
 #ifdef WIN32
@@ -70,7 +78,7 @@
 
 #else /* non windows */
 
-#ifdef __linux__
+#ifdef WITH_BINRELOC
 #include "binreloc.h"
 #endif
 
@@ -79,6 +87,8 @@
 /* local */
 #define UNIQUE_NAME_MAX 128
 
+extern char bprogname[];
+
 static int add_win32_extension(char *name);
 static char *blender_version_decimal(void);
 
@@ -196,7 +206,7 @@ void BLI_newname(char *name, int add)
 
 int BLI_uniquename_cb(int (*unique_check)(void *, const char *), void *arg, const char defname[], char delim, char *name, short name_len)
 {
-       if(name == '\0') {
+       if(name[0] == '\0') {
                BLI_strncpy(name, defname, name_len);
        }
 
@@ -286,14 +296,7 @@ void BLI_uniquename(ListBase *list, void *vlink, const char defname[], char deli
  * If relbase is NULL then its ignored
  */
 
-void BLI_cleanup_dir(const char *relabase, char *dir)
-{
-       BLI_cleanup_file(relabase, dir);
-       BLI_add_slash(dir);
-
-}
-
-void BLI_cleanup_file(const char *relabase, char *dir)
+void BLI_cleanup_path(const char *relabase, char *dir)
 {
        short a;
        char *start, *eind;
@@ -348,13 +351,6 @@ void BLI_cleanup_file(const char *relabase, char *dir)
                eind = start + strlen("\\\\") - 1;
                memmove( start, eind, strlen(eind)+1 );
        }
-
-       if((a = strlen(dir))){                          /* remove the '\\' at the end */
-               while(a>0 && dir[a-1] == '\\'){
-                       a--;
-                       dir[a] = 0;
-               }
-       }
 #else
        if(dir[0]=='.') {       /* happens, for example in FILE_MAIN */
           dir[0]= '/';
@@ -392,17 +388,21 @@ void BLI_cleanup_file(const char *relabase, char *dir)
                eind = start + (2 - 1) /* strlen("//") - 1 */;
                memmove( start, eind, strlen(eind)+1 );
        }
-
-       if( (a = strlen(dir)) ){                                /* remove all '/' at the end */
-               while(dir[a-1] == '/'){
-                       a--;
-                       dir[a] = 0;
-                       if (a<=0) break;
-               }
-       }
 #endif
 }
 
+void BLI_cleanup_dir(const char *relabase, char *dir)
+{
+       BLI_cleanup_path(relabase, dir);
+       BLI_add_slash(dir);
+
+}
+
+void BLI_cleanup_file(const char *relabase, char *dir)
+{
+       BLI_cleanup_path(relabase, dir);
+       BLI_del_slash(dir);
+}
 
 void BLI_path_rel(char *file, const char *relfile)
 {
@@ -416,8 +416,8 @@ void BLI_path_rel(char *file, const char *relfile)
        /* also bail out if relative path is not set */
        if (relfile[0] == 0) return;
 
-#ifdef WIN32 
-       if (strlen(relfile) > 2 && relfile[1] != ':') {
+#ifdef WIN32
+       if (BLI_strnlen(relfile, 3) > 2 && relfile[1] != ':') {
                char* ptemp;
                /* fix missing volume name in relative base,
                   can happen with old recent-files.txt files */
@@ -431,7 +431,7 @@ void BLI_path_rel(char *file, const char *relfile)
                BLI_strncpy(temp, relfile, FILE_MAXDIR + FILE_MAXFILE);
        }
 
-       if (strlen(file) > 2) {
+       if (BLI_strnlen(file, 3) > 2) {
                if ( temp[1] == ':' && file[1] == ':' && temp[0] != file[0] )
                        return;
        }
@@ -443,8 +443,8 @@ void BLI_path_rel(char *file, const char *relfile)
        BLI_char_switch(file, '\\', '/');
        
        /* remove /./ which confuse the following slash counting... */
-       BLI_cleanup_file(NULL, file);
-       BLI_cleanup_file(NULL, temp);
+       BLI_cleanup_path(NULL, file);
+       BLI_cleanup_path(NULL, temp);
        
        /* the last slash in the file indicates where the path part ends */
        lslash = BLI_last_slash(temp);
@@ -456,7 +456,12 @@ void BLI_path_rel(char *file, const char *relfile)
                char *p= temp;
                char *q= file;
 
-               while ((*p == *q)) {
+#ifdef WIN32
+               while (tolower(*p) == tolower(*q))
+#else
+               while (*p == *q)
+#endif
+               {
                        ++p; ++q;
                        /* dont search beyond the end of the string
                         * in the rare case they match */
@@ -587,19 +592,7 @@ int BLI_path_frame(char *path, int frame, int digits)
 
        if (stringframe_chars(path, &ch_sta, &ch_end)) { /* warning, ch_end is the last # +1 */
                char tmp[FILE_MAX];
-#if 0  // neat but breaks on non ascii strings.
-               char format[64];
-               sprintf(format, "%%.%ds%%.%dd%%s", ch_sta, ch_end-ch_sta); /* example result: "%.12s%.5d%s" */
-               sprintf(tmp, format, path, frame, path+ch_end);
-#else
-               char format[8];
-               char *p;
-               sprintf(format, "%%.%dd", ch_end-ch_sta); /* example result: "%.5d" */
-               memcpy(tmp, path, sizeof(char) * ch_sta);
-               p= tmp + ch_sta;
-               p += sprintf(p, format, frame);
-               memcpy(p, path + ch_end, strlen(path + ch_end));
-#endif
+               sprintf(tmp, "%.*s%.*d%s", ch_sta, path, ch_end-ch_sta, frame, path+ch_end);
                strcpy(path, tmp);
                return 1;
        }
@@ -614,9 +607,8 @@ int BLI_path_frame_range(char *path, int sta, int end, int digits)
                ensure_digits(path, digits);
 
        if (stringframe_chars(path, &ch_sta, &ch_end)) { /* warning, ch_end is the last # +1 */
-               char tmp[FILE_MAX], format[64];
-               sprintf(format, "%%.%ds%%.%dd_%%.%dd%%s", ch_sta, ch_end-ch_sta, ch_end-ch_sta); /* example result: "%.12s%.5d-%.5d%s" */
-               sprintf(tmp, format, path, sta, end, path+ch_end);
+               char tmp[FILE_MAX];
+               sprintf(tmp, "%.*s%.*d-%.*d%s", ch_sta, path, ch_end-ch_sta, sta, ch_end-ch_sta, end, path+ch_end);
                strcpy(path, tmp);
                return 1;
        }
@@ -667,8 +659,9 @@ int BLI_path_abs(char *path, const char *basepath)
 #endif
 
        BLI_strncpy(base, basepath, sizeof(base));
-       
-       BLI_cleanup_file(NULL, base);
+
+       /* file component is ignored, so dont bother with the trailing slash */
+       BLI_cleanup_path(NULL, base);
        
        /* push slashes into unix mode - strings entering this part are
           potentially messed up: having both back- and forward slashes.
@@ -697,17 +690,9 @@ int BLI_path_abs(char *path, const char *basepath)
        } else {
                BLI_strncpy(path, tmp, FILE_MAX);
        }
-       
-       if (path[0]!='\0') {
-               if ( path[strlen(path)-1]=='/') {
-                       /* remove the '/' so we avoid BLI_cleanup_dir adding an extra \ in WIN32 */
-                       path[strlen(path)-1] = '\0';
-                       BLI_cleanup_dir(NULL, path);
-               } else {
-                       BLI_cleanup_file(NULL, path);
-               }
-       }
-       
+
+       BLI_cleanup_path(NULL, path);
+
 #ifdef WIN32
        /* skip first two chars, which in case of
           absolute path will be drive:/blabla and
@@ -740,8 +725,8 @@ int BLI_path_cwd(char *path)
 #endif
        
        if (wasrelative==1) {
-               char cwd[FILE_MAXDIR + FILE_MAXFILE];
-               BLI_getwdN(cwd); /* incase the full path to the blend isnt used */
+               char cwd[FILE_MAXDIR + FILE_MAXFILE]= "";
+               BLI_getwdN(cwd, sizeof(cwd)); /* incase the full path to the blend isnt used */
                
                if (cwd[0] == '\0') {
                        printf( "Could not get the current working directory - $PWD for an unknown reason.");
@@ -800,12 +785,12 @@ void BLI_getlastdir(const char* dir, char *last, int maxlen)
 /* This is now only used to really get the user's default document folder */
 /* On Windows I chose the 'Users/<MyUserName>/Documents' since it's used
    as default location to save documents */
-char *BLI_getDefaultDocumentFolder(void) {
+const char *BLI_getDefaultDocumentFolder(void) {
        #if !defined(WIN32)
                return getenv("HOME");
 
        #else /* Windows */
-               char * ret;
+               const char * ret;
                static char documentfolder[MAXPATHLEN];
                HRESULT hResult;
 
@@ -845,11 +830,11 @@ static char *blender_version_decimal(void)
        return version_str;
 }
 
-static int test_path(char *targetpath, char *path_base, char *path_sep, char *folder_name)
+static int test_path(char *targetpath, const char *path_base, const char *path_sep, const char *folder_name)
 {
        char tmppath[FILE_MAX];
        
-       if(path_sep)    BLI_join_dirfile(tmppath, path_base, path_sep);
+       if(path_sep)    BLI_join_dirfile(tmppath, sizeof(tmppath), path_base, path_sep);
        else                    BLI_strncpy(tmppath, path_base, sizeof(tmppath));
        
        BLI_make_file_string("/", targetpath, tmppath, folder_name);
@@ -869,9 +854,9 @@ static int test_path(char *targetpath, char *path_base, char *path_sep, char *fo
        }
 }
 
-static int test_env_path(char *path, char *envvar)
+static int test_env_path(char *path, const char *envvar)
 {
-       char *env = envvar?getenv(envvar):NULL;
+       const char *env = envvar?getenv(envvar):NULL;
        if (!env) return 0;
        
        if (BLI_is_dir(env)) {
@@ -883,9 +868,8 @@ static int test_env_path(char *path, char *envvar)
        }
 }
 
-static int get_path_local(char *targetpath, char *folder_name, char *subfolder_name)
+static int get_path_local(char *targetpath, const char *folder_name, const char *subfolder_name)
 {
-       extern char bprogname[]; /* argv[0] from creator.c */
        char bprogdir[FILE_MAX];
        char relfolder[FILE_MAX];
        
@@ -894,9 +878,9 @@ static int get_path_local(char *targetpath, char *folder_name, char *subfolder_n
 #endif
        
        if (subfolder_name) {
-               BLI_join_dirfile(relfolder, folder_name, subfolder_name);
+               BLI_join_dirfile(relfolder, sizeof(relfolder), folder_name, subfolder_name);
        } else {
-               BLI_strncpy(relfolder, folder_name, FILE_MAX);
+               BLI_strncpy(relfolder, folder_name, sizeof(relfolder));
        }
        
        /* use argv[0] (bprogname) to get the path to the executable */
@@ -909,7 +893,7 @@ static int get_path_local(char *targetpath, char *folder_name, char *subfolder_n
        return 0;
 }
 
-static int get_path_user(char *targetpath, char *folder_name, char *subfolder_name, char *envvar)
+static int get_path_user(char *targetpath, const char *folder_name, const char *subfolder_name, const char *envvar)
 {
        char user_path[FILE_MAX];
        const char *user_base_path;
@@ -946,7 +930,7 @@ static int get_path_user(char *targetpath, char *folder_name, char *subfolder_na
        }
 }
 
-static int get_path_system(char *targetpath, char *folder_name, char *subfolder_name, char *envvar)
+static int get_path_system(char *targetpath, const char *folder_name, const char *subfolder_name, const char *envvar)
 {
        char system_path[FILE_MAX];
        const char *system_base_path;
@@ -954,7 +938,6 @@ static int get_path_system(char *targetpath, char *folder_name, char *subfolder_
 
        /* first allow developer only overrides to the system path
         * these are only used when running blender from source */
-       extern char bprogname[]; /* argv[0] from creator.c */
        char cwd[FILE_MAX];
        char relfolder[FILE_MAX];
        char bprogdir[FILE_MAX];
@@ -963,15 +946,18 @@ static int get_path_system(char *targetpath, char *folder_name, char *subfolder_
        BLI_split_dirfile(bprogname, bprogdir, NULL);
 
        if (subfolder_name) {
-               BLI_join_dirfile(relfolder, folder_name, subfolder_name);
+               BLI_join_dirfile(relfolder, sizeof(relfolder), folder_name, subfolder_name);
        } else {
-               BLI_strncpy(relfolder, folder_name, FILE_MAX);
+               BLI_strncpy(relfolder, folder_name, sizeof(relfolder));
        }
 
        /* try CWD/release/folder_name */
-       if(test_path(targetpath, BLI_getwdN(cwd), "release", relfolder))
-               return 1;
-       
+       if(BLI_getwdN(cwd, sizeof(cwd))) {
+               if(test_path(targetpath, cwd, "release", relfolder)) {
+                       return 1;
+               }
+       }
+
        /* try EXECUTABLE_DIR/release/folder_name */
        if(test_path(targetpath, bprogdir, "release", relfolder))
                return 1;
@@ -1013,7 +999,7 @@ static int get_path_system(char *targetpath, char *folder_name, char *subfolder_
 
 /* get a folder out of the 'folder_id' presets for paths */
 /* returns the path if found, NULL string if not */
-char *BLI_get_folder(int folder_id, char *subfolder)
+char *BLI_get_folder(int folder_id, const char *subfolder)
 {
        static char path[FILE_MAX] = "";
        
@@ -1085,7 +1071,7 @@ char *BLI_get_folder(int folder_id, char *subfolder)
        return path;
 }
 
-char *BLI_get_user_folder_notest(int folder_id, char *subfolder)
+char *BLI_get_user_folder_notest(int folder_id, const char *subfolder)
 {
        static char path[FILE_MAX] = "";
 
@@ -1109,7 +1095,7 @@ char *BLI_get_user_folder_notest(int folder_id, char *subfolder)
        return path;
 }
 
-char *BLI_get_folder_create(int folder_id, char *subfolder)
+char *BLI_get_folder_create(int folder_id, const char *subfolder)
 {
        char *path;
 
@@ -1171,9 +1157,10 @@ void BLI_setenv_if_new(const char *env, const char* val)
 
 void BLI_clean(char *path)
 {
-       if(path==0) return;
+       if(path==NULL) return;
+
 #ifdef WIN32
-       if(path && strlen(path)>2) {
+       if(path && BLI_strnlen(path, 3) > 2) {
                BLI_char_switch(path+2, '/', '\\');
        }
 #else
@@ -1183,7 +1170,7 @@ void BLI_clean(char *path)
 
 void BLI_char_switch(char *string, char from, char to) 
 {
-       if(string==0) return;
+       if(string==NULL) return;
        while (*string != 0) {
                if (*string == from) *string = to;
                string++;
@@ -1217,7 +1204,7 @@ void BLI_make_exist(char *dir) {
        }
 }
 
-void BLI_make_existing_file(char *name)
+void BLI_make_existing_file(const char *name)
 {
        char di[FILE_MAXDIR+FILE_MAXFILE], fi[FILE_MAXFILE];
 
@@ -1262,7 +1249,7 @@ void BLI_make_file_string(const char *relabase, char *string,  const char *dir,
        }
 #ifdef WIN32
        else {
-               if (strlen(dir) >= 2 && dir[1] == ':' ) {
+               if (BLI_strnlen(dir, 3) >= 2 && dir[1] == ':' ) {
                        BLI_strncpy(string, dir, 3);
                        dir += 2;
                }
@@ -1308,10 +1295,10 @@ int BLI_testextensie(const char *str, const char *ext)
 {
        short a, b;
        int retval;
-
+       
        a= strlen(str);
        b= strlen(ext);
-
+       
        if(a==0 || b==0 || b>=a) {
                retval = 0;
        } else if (BLI_strcasecmp(ext, str + a - b)) {
@@ -1319,7 +1306,7 @@ int BLI_testextensie(const char *str, const char *ext)
        } else {
                retval = 1;
        }
-
+       
        return (retval);
 }
 
@@ -1366,13 +1353,16 @@ int BLI_testextensie_glob(const char *str, const char *ext_fnmatch)
 }
 
 
-int BLI_replace_extension(char *path, int maxlen, const char *ext)
+int BLI_replace_extension(char *path, size_t maxlen, const char *ext)
 {
-       int a;
+       size_t a;
 
-       for(a=strlen(path)-1; a>=0; a--)
-               if(path[a] == '.' || path[a] == '/' || path[a] == '\\')
+       for(a=strlen(path); a>0; a--) {
+               if(path[a-1] == '.' || path[a-1] == '/' || path[a-1] == '\\') {
+                       a--;
                        break;
+               }
+       }
        
        if(path[a] != '.')
                a= strlen(path);
@@ -1409,12 +1399,12 @@ void BLI_split_dirfile(const char *string, char *dir, char *file)
 }
 
 /* simple appending of filename to dir, does not check for valid path! */
-void BLI_join_dirfile(char *string, const char *dir, const char *file)
+void BLI_join_dirfile(char *string, const int maxlen, const char *dir, const char *file)
 {
        int sl_dir;
        
        if(string != dir) /* compare pointers */
-               BLI_strncpy(string, dir, FILE_MAX);
+               BLI_strncpy(string, dir, maxlen);
 
        if (!file)
                return;
@@ -1422,7 +1412,7 @@ void BLI_join_dirfile(char *string, const char *dir, const char *file)
        sl_dir= BLI_add_slash(string);
        
        if (sl_dir <FILE_MAX) {
-               BLI_strncpy(string + sl_dir, file, FILE_MAX-sl_dir);
+               BLI_strncpy(string + sl_dir, file, maxlen - sl_dir);
        }
 }
 
@@ -1496,22 +1486,22 @@ int BKE_rebase_path(char *abs, int abs_size, char *rel, int rel_size, const char
        if (!strncmp(path, blend_dir, len)) {
 
                /* if image is _in_ current .blend file directory */
-               if (!strcmp(dir, blend_dir)) {
-                       BLI_join_dirfile(dest_path, dest_dir, base);
+               if (BLI_path_cmp(dir, blend_dir) == 0) {
+                       BLI_join_dirfile(dest_path, sizeof(dest_path), dest_dir, base);
                }
                /* "below" */
                else {
                        /* rel = image_path_dir - blend_dir */
                        BLI_strncpy(rel_dir, dir + len, sizeof(rel_dir));
 
-                       BLI_join_dirfile(dest_path, dest_dir, rel_dir);
-                       BLI_join_dirfile(dest_path, dest_path, base);
+                       BLI_join_dirfile(dest_path, sizeof(dest_path), dest_dir, rel_dir);
+                       BLI_join_dirfile(dest_path, sizeof(dest_path), dest_path, base);
                }
 
        }
        /* image is out of current directory */
        else {
-               BLI_join_dirfile(dest_path, dest_dir, base);
+               BLI_join_dirfile(dest_path, sizeof(dest_path), dest_dir, base);
        }
 
        if (abs)
@@ -1523,7 +1513,7 @@ int BKE_rebase_path(char *abs, int abs_size, char *rel, int rel_size, const char
        }
 
        /* return 2 if src=dest */
-       if (!strcmp(path, dest_path)) {
+       if (BLI_path_cmp(path, dest_path) == 0) {
                // if (G.f & G_DEBUG) printf("%s and %s are the same file\n", path, dest_path);
                return 2;
        }
@@ -1603,7 +1593,7 @@ static int add_win32_extension(char *name)
 #ifdef _WIN32
                char filename[FILE_MAXDIR+FILE_MAXFILE];
                char ext[FILE_MAXDIR+FILE_MAXFILE];
-               char *extensions = getenv("PATHEXT");
+               const char *extensions = getenv("PATHEXT");
                if (extensions) {
                        char *temp;
                        do {
@@ -1634,54 +1624,54 @@ static int add_win32_extension(char *name)
        return (retval);
 }
 
-void BLI_where_am_i(char *fullname, const char *name)
+/* filename must be FILE_MAX length minimum */
+void BLI_where_am_i(char *fullname, const int maxlen, const char *name)
 {
        char filename[FILE_MAXDIR+FILE_MAXFILE];
-       char *path = NULL, *temp;
-       
+       const char *path = NULL, *temp;
+
 #ifdef _WIN32
-       char *separator = ";";
-       char slash = '\\';
+       const char *separator = ";";
 #else
-       char *separator = ":";
-       char slash = '/';
+       const char *separator = ":";
 #endif
 
        
-#ifdef __linux__
+#ifdef WITH_BINRELOC
        /* linux uses binreloc since argv[0] is not relyable, call br_init( NULL ) first */
        path = br_find_exe( NULL );
        if (path) {
-               BLI_strncpy(fullname, path, FILE_MAXDIR+FILE_MAXFILE);
-               free(path);
+               BLI_strncpy(fullname, path, maxlen);
+               free((void *)path);
                return;
        }
 #endif
 
 #ifdef _WIN32
-       if(GetModuleFileName(0, fullname, FILE_MAXDIR+FILE_MAXFILE)) {
-               GetShortPathName(fullname, fullname, FILE_MAXDIR+FILE_MAXFILE);
+       if(GetModuleFileName(0, fullname, maxlen)) {
+               GetShortPathName(fullname, fullname, maxlen);
                return;
        }
 #endif
 
        /* unix and non linux */
-       if (name && fullname && strlen(name)) {
-               strcpy(fullname, name);
+       if (name && name[0]) {
+               BLI_strncpy(fullname, name, maxlen);
                if (name[0] == '.') {
-                       // relative path, prepend cwd
-                       BLI_getwdN(fullname);
-                       
+                       char wdir[FILE_MAX]= "";
+                       BLI_getwdN(wdir, sizeof(wdir));  /* backup cwd to restore after */
+
                        // not needed but avoids annoying /./ in name
-                       if(name && name[0]=='.' && name[1]==slash)
-                               BLI_join_dirfile(fullname, fullname, name+2);
+                       if(name[1]==SEP)
+                               BLI_join_dirfile(fullname, maxlen, wdir, name+2);
                        else
-                               BLI_join_dirfile(fullname, fullname, name);
-                       
-                       add_win32_extension(fullname);
-               } else if (BLI_last_slash(name)) {
+                               BLI_join_dirfile(fullname, maxlen, wdir, name);
+
+                       add_win32_extension(fullname); /* XXX, doesnt respect length */
+               }
+               else if (BLI_last_slash(name)) {
                        // full path
-                       strcpy(fullname, name);
+                       BLI_strncpy(fullname, name, maxlen);
                        add_win32_extension(fullname);
                } else {
                        // search for binary in $PATH
@@ -1696,9 +1686,9 @@ void BLI_where_am_i(char *fullname, const char *name)
                                        } else {
                                                strncpy(filename, path, sizeof(filename));
                                        }
-                                       BLI_join_dirfile(fullname, fullname, name);
+                                       BLI_join_dirfile(fullname, maxlen, fullname, name);
                                        if (add_win32_extension(filename)) {
-                                               strcpy(fullname, filename);
+                                               BLI_strncpy(fullname, filename, maxlen);
                                                break;
                                        }
                                } while (temp);
@@ -1716,7 +1706,7 @@ void BLI_where_am_i(char *fullname, const char *name)
                // spaces and double-quotes. There's another solution to this
                // with spawnv(P_WAIT, bprogname, argv) instead of system() but
                // that's even uglier
-               GetShortPathName(fullname, fullname, FILE_MAXDIR+FILE_MAXFILE);
+               GetShortPathName(fullname, fullname, maxlen);
 #if defined(DEBUG)
                printf("Shortname = '%s'\n", fullname);
 #endif
@@ -1724,52 +1714,53 @@ void BLI_where_am_i(char *fullname, const char *name)
        }
 }
 
-void BLI_where_is_temp(char *fullname, int usertemp)
+void BLI_where_is_temp(char *fullname, const int maxlen, int usertemp)
 {
        fullname[0] = '\0';
        
        if (usertemp && BLI_is_dir(U.tempdir)) {
-               strcpy(fullname, U.tempdir);
+               BLI_strncpy(fullname, U.tempdir, maxlen);
        }
        
        
 #ifdef WIN32
        if (fullname[0] == '\0') {
-               char *tmp = getenv("TEMP"); /* Windows */
+               const char *tmp = getenv("TEMP"); /* Windows */
                if (tmp && BLI_is_dir(tmp)) {
-                       strcpy(fullname, tmp);
+                       BLI_strncpy(fullname, tmp, maxlen);
                }
        }
 #else
        /* Other OS's - Try TMP and TMPDIR */
        if (fullname[0] == '\0') {
-               char *tmp = getenv("TMP");
+               const char *tmp = getenv("TMP");
                if (tmp && BLI_is_dir(tmp)) {
-                       strcpy(fullname, tmp);
+                       BLI_strncpy(fullname, tmp, maxlen);
                }
        }
        
        if (fullname[0] == '\0') {
-               char *tmp = getenv("TMPDIR");
+               const char *tmp = getenv("TMPDIR");
                if (tmp && BLI_is_dir(tmp)) {
-                       strcpy(fullname, tmp);
+                       BLI_strncpy(fullname, tmp, maxlen);
                }
        }
 #endif 
        
        if (fullname[0] == '\0') {
-               strcpy(fullname, "/tmp/");
+               BLI_strncpy(fullname, "/tmp/", maxlen);
        } else {
                /* add a trailing slash if needed */
                BLI_add_slash(fullname);
 #ifdef WIN32
-               strcpy(U.tempdir, fullname); /* also set user pref to show %TEMP%. /tmp/ is just plain confusing for Windows users. */
+               if(U.tempdir != fullname) {
+                       BLI_strncpy(U.tempdir, fullname, maxlen); /* also set user pref to show %TEMP%. /tmp/ is just plain confusing for Windows users. */
+               }
 #endif
        }
 }
 
 char *get_install_dir(void) {
-       extern char bprogname[];
        char *tmpname = BLI_strdup(bprogname);
        char *cut;