diff options
Diffstat (limited to '')
-rw-r--r-- | src/directory.c | 118 | ||||
-rw-r--r-- | src/path.c | 39 | ||||
-rw-r--r-- | src/path.h | 3 |
3 files changed, 135 insertions, 25 deletions
diff --git a/src/directory.c b/src/directory.c index 671b6b9d4..bd4c0a81a 100644 --- a/src/directory.c +++ b/src/directory.c @@ -94,7 +94,7 @@ void deleteEmptyDirectoriesInDirectory(Directory * directory); void removeSongFromDirectory(Directory * directory, char * shortname); -int addSubDirectoryToDirectory(Directory * directory, char * shortname, +Directory * addSubDirectoryToDirectory(Directory * directory, char * shortname, char * name); Directory * getDirectoryDetails(char * name, char ** shortname, @@ -107,6 +107,8 @@ Song * getSongDetails(char * file, char ** shortnameRet, void updatePath(char * utf8path); +void sortDirectory(Directory * directory); + void clearUpdatePid() { directory_updatePid = 0; } @@ -312,15 +314,12 @@ int removeDeletedFromDirectory(Directory * directory) { if(!utf8) continue; - utf8 = strdup(utf8); - if(directory->utf8name) { s = malloc(strlen(directory->utf8name)+strlen(utf8)+2); sprintf(s,"%s/%s",directory->utf8name,utf8); } else s= strdup(utf8); insertInList(entList,utf8,s); - free(utf8); } closedir(dir); @@ -331,11 +330,14 @@ int removeDeletedFromDirectory(Directory * directory) { if(findInList(entList,node->key,&name)) { if(!isDir((char *)name,NULL)) { LOG("removing directory: %s\n",(char*)name); - deleteFromList(directory->subDirectories,node->key); + deleteFromList(directory->subDirectories, + node->key); } } else { - LOG("removing directory: %s\n",(char*)name); + LOG("removing directory: "); + if(directory->utf8name) LOG("%s/",directory->utf8name); + LOG("%s\n",node->key); deleteFromList(directory->subDirectories,node->key); } node = tmpNode; @@ -360,39 +362,104 @@ int removeDeletedFromDirectory(Directory * directory) { return 0; } +Directory * addDirectoryPathToDB(char * utf8path, char ** shortname) { + char * parent; + Directory * parentDirectory; + void * directory; + + parent = strdup(parentPath(utf8path)); + + if(strlen(parent)==0) parentDirectory = (void *)mp3rootDirectory; + else parentDirectory = addDirectoryPathToDB(parent,shortname); + + *shortname = utf8path+strlen(parent); + while(*(*shortname) && *(*shortname)=='/') (*shortname)++; + + if(!findInList(parentDirectory->subDirectories,*shortname, &directory)) + { + directory = (void *)addSubDirectoryToDirectory(parentDirectory, + *shortname,utf8path); + } + + /* if we're adding directory paths, make sure to delete filenames + with potentially the same name*/ + removeSongFromDirectory(parentDirectory,*shortname); + + free(parent); + + return (Directory *)parentDirectory; +} + +Directory * addParentPathToDB(char * utf8path, char ** shortname) { + char * parent; + Directory * parentDirectory; + + parent = strdup(parentPath(utf8path)); + + if(strlen(parent)==0) parentDirectory = (void *)mp3rootDirectory; + else parentDirectory = addDirectoryPathToDB(parent,shortname); + + *shortname = utf8path+strlen(parent); + while(*(*shortname) && *(*shortname)=='/') (*shortname)++; + + free(parent); + + return (Directory *)parentDirectory; +} + void updatePath(char * utf8path) { Directory * directory; Directory * parentDirectory; Song * song; char * shortname; - DIR * dir; + char * path = sanitizePathDup(utf8path); + + if(NULL==path) return; /* if path is in the DB try to update it, or else delete it */ - if((directory = getDirectoryDetails(utf8path,&shortname, + if((directory = getDirectoryDetails(path,&shortname, &parentDirectory))) { - /* if updateDirectory fials, means we should delete it */ - if(updateDirectory(directory)<0 && directory!=mp3rootDirectory) + /* if this update directory is successfull, we are done */ + if(updateDirectory(directory)==0) { + free(path); + sortDirectory(directory); + return; + } + /* we don't want to delete the root directory */ + else if(directory == mp3rootDirectory) { + free(path); + return; + } + /* if updateDirectory fials, means we should delete it */ + else { + LOG("removing directory: %s\n",path); deleteFromList(parentDirectory->subDirectories, shortname); } } - else if((song = getSongDetails(utf8path,&shortname,&parentDirectory))) { - if(song && updateSongInfo(song)<0) { - removeSongFromDirectory(parentDirectory,shortname); + else if((song = getSongDetails(path,&shortname,&parentDirectory))) { + /* if this song update is successfull, we are done */ + if(song && updateSongInfo(song)==0) { + free(path); + return; } + /* if updateDirectory fials, means we should delete it */ + else removeSongFromDirectory(parentDirectory,shortname); } - /* apth not found in the db, see if it actually exists on the fs */ - else if(isDir(utf8path,NULL)) { - /* create parent/get parent directory */ - /* create new directory and add to parent */ - /* explore direcotry */ - } - else if((isMusic(utf8path,NULL))) { - /* create parent/get parent directory */ - /* add song to directory */ + + /* path not found in the db, see if it actually exists on the fs. + * Also, if by chance a directory was replaced by a file of the same + * name or vice versa, we need to add it to the db + */ + if(isDir(path,NULL) || isMusic(path,NULL)) { + parentDirectory = addParentPathToDB(path,&shortname); + addToDirectory(parentDirectory,shortname,path); + sortDirectory(parentDirectory); } + + free(path); } int updateDirectory(Directory * directory) { @@ -479,7 +546,7 @@ int exploreDirectory(Directory * directory) { return 0; } -int addSubDirectoryToDirectory(Directory * directory, char * shortname, +Directory * addSubDirectoryToDirectory(Directory * directory, char * shortname, char * name) { Directory * subDirectory = newDirectory(name,-1); @@ -487,12 +554,13 @@ int addSubDirectoryToDirectory(Directory * directory, char * shortname, insertInList(directory->subDirectories,shortname,subDirectory); exploreDirectory(subDirectory); - return 0; + return subDirectory; } int addToDirectory(Directory * directory, char * shortname, char * name) { if(isDir(name,NULL)) { - return addSubDirectoryToDirectory(directory,shortname,name); + addSubDirectoryToDirectory(directory,shortname,name); + return 0; } else if(isMusic(name,NULL)) { Song * song; diff --git a/src/path.c b/src/path.c index e9031100d..71bacd7b3 100644 --- a/src/path.c +++ b/src/path.c @@ -20,6 +20,7 @@ #include "log.h" #include "charConv.h" #include "conf.h" +#include "utf8.h" #include <stdlib.h> #include <string.h> @@ -202,3 +203,41 @@ char * parentPath(char * path) { return parentPath; } + +char * sanitizePathDup(char * path) { + int len = strlen(path)+1; + char * ret = malloc(len); + char * cp = ret; + + memset(ret,0,len); + + len = 0; + + /* illeminate more than one '/' in a row, like "///" */ + while(*path) { + while(*path=='/') path++; + if(*path=='.') { + /* we dont want to have hidden directoires, or '.' or + ".." in our path */ + free(ret); + return NULL; + } + while(*path && *path!='/') { + *(cp++) = *(path++); + len++; + } + if(*path=='/') { + *(cp++) = *(path++); + len++; + } + } + + if(len && ret[len-1]=='/') { + len--; + ret[len] = '\0'; + } + + printf("sanitized: %s\n", ret); + + return realloc(ret,len+1); +} diff --git a/src/path.h b/src/path.h index dc66a0733..59fab7c5c 100644 --- a/src/path.h +++ b/src/path.h @@ -49,4 +49,7 @@ char * rpp2app(char * file); /* static char * returned */ char * parentPath(char * path); +/* strips extra "///" and leading "/" and trailing "/" */ +char * sanitizePathDup(char * path); + #endif |