Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Enumerate files code: cleaned up memory leak, Doesn't throw away file…
… entry

 from FindFirstFile(), and handles paths more robustly.
  • Loading branch information
icculus committed Jun 8, 2002
1 parent a89efe6 commit 1a665ef
Showing 1 changed file with 13 additions and 4 deletions.
17 changes: 13 additions & 4 deletions platform/win32.c
Expand Up @@ -349,18 +349,27 @@ LinkedStringList *__PHYSFS_platformEnumerateFiles(const char *dirname,
HANDLE dir;
WIN32_FIND_DATA ent;
char *SearchPath;
size_t len = strlen(dirname);

/* Allocate a new string for path, "*", and NULL terminator */
SearchPath = malloc(strlen(dirname) + 2);
/* Allocate a new string for path, maybe '\\', "*", and NULL terminator */
SearchPath = alloca(len + 3);
/* Copy current dirname */
strcpy(SearchPath, dirname);

/* if there's no '\\' at the end of the path, stick one in there. */
if (dirname[len - 1] != '\\')
{
dirname[len++] = '\\';
dirname[len] = '\0';
} /* if */

/* Append the "*" to the end of the string */
strcat(SearchPath, "*");

dir = FindFirstFile(SearchPath, &ent);
BAIL_IF_MACRO(dir == INVALID_HANDLE_VALUE, win32strerror(), NULL);

while (FindNextFile(dir, &ent) != 0)
do
{
if (strcmp(ent.cFileName, ".") == 0)
continue;
Expand Down Expand Up @@ -388,7 +397,7 @@ LinkedStringList *__PHYSFS_platformEnumerateFiles(const char *dirname,

prev = l;
l->next = NULL;
} /* while */
} while (FindNextFile(dir, &ent) != 0);

FindClose(dir);
return(retval);
Expand Down

0 comments on commit 1a665ef

Please sign in to comment.