empire/empserver
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix initialization of configdir

Browse source 
Windows code leaked memory (result of _fullpath()).

POSIX code passed a null buffer to getcwd(), which is not portable,
and failed to check for errors.
This commit is contained in:
Markus Armbruster 2008-02-04 07:39:42 +01:00
parent b76e5a5eed
commit 91eefc3f3a
1 changed files with 22 additions and 25 deletions
Download patch file Download diff file Expand all files Collapse all files

47
src/lib/common/emp_config.c
View file

@ -66,7 +66,7 @@ struct keymatch configkeys[] = {
};
static struct keymatch *keylookup(char *key, struct keymatch tbl[]);
static void set_paths(char *);
static int set_paths(char *);
/*
* read in empire configuration
@ -150,7 +150,8 @@ emp_config(char *file)
done:
WORLD_X &= ~1; /* force even */
set_paths(file);
if (set_paths(file) < 0)
return -1;
return -errors;
}
@ -170,43 +171,39 @@ keylookup(char *command, struct keymatch *tbl)
return NULL;
}
static void
static int
set_paths(char *econfig)
{
char *slash;
char *cwd = getcwd(NULL, 0);
char *p, *slash;
#ifdef _WIN32
/* normalize path separator to '\\', for easier searching: */
econfig = _fullpath(NULL, econfig, 0);
slash = strrchr(econfig, '\\');
configdir = malloc(slash - econfig + 1);
memcpy(configdir, econfig, slash - econfig);
configdir[slash - econfig] = 0;
p = _fullpath(NULL, econfig, 0);
slash = strrchr(p, '\\');
#else
if ((slash = strrchr(econfig, '/'))) {
configdir = malloc(slash - econfig + 1);
memcpy(configdir, econfig, slash - econfig);
configdir[slash - econfig] = 0;
} else
configdir = strdup(cwd);
char buf[1024];
char *cwd;
if (configdir[0] != '/') {
char *tmp = configdir;
size_t len = strlen(cwd);
configdir = malloc(len + 1 + strlen(tmp) + 1);
sprintf(configdir, "%s/%s", cwd, tmp);
free(tmp);
cwd = getcwd(buf, sizeof(buf));
p = fnameat(econfig, cwd);
if (p[0] != '/') {
fprintf(stderr, "Can't get current working directory (%s)\n",
strerror(errno));
return -1;
}
if (p == econfig)
p = strdup(p);
slash = strrchr(p, '/');
#endif /* !_WIN32 */
*slash = 0;
configdir = realloc(p, slash + 1 - configdir);
infodir = fnameat(infodir_conf, configdir);
gamedir = fnameat(gamedir_conf, configdir);
builtindir = fnameat(builtindir_conf, configdir);
schedulefil = fnameat("schedule", configdir);
free(cwd);
return 0;
}
void