From b823a0743afc9d09ee93ff39b597e759e66fd1ed Mon Sep 17 00:00:00 2001 From: Dmitri Pal Date: Fri, 24 Dec 2010 21:25:56 -0500 Subject: Freeing memory used for source dir I decided not to redo the whole test. It would have been too much work. Just cleaned the code a bit and freed allocated memory. It is Coverity issue 10075. --- ini/ini_config_ut.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/ini/ini_config_ut.c b/ini/ini_config_ut.c index cbd4d4f..53ad334 100644 --- a/ini/ini_config_ut.c +++ b/ini/ini_config_ut.c @@ -1546,6 +1546,7 @@ int main(int argc, char *argv[]) int error = EOK; char *srcdir = NULL; char *rundir = NULL; + const char inidir[] = "/ini"; if ((argc > 1) && (strcmp(argv[1], "-v") == 0)) verbose = 1; @@ -1553,13 +1554,15 @@ int main(int argc, char *argv[]) srcdir = getenv("srcdir"); if(srcdir) { - rundir = malloc(strlen(srcdir)+4*sizeof(char)); - sprintf(rundir, "%s/ini", srcdir); + rundir = malloc(strlen(srcdir) + sizeof(inidir)); + sprintf(rundir, "%s%s", srcdir, inidir); if(chdir(rundir) != 0) { error = errno; + free(rundir); printf("Failed to change directory, error %d\n", error); return error; } + free(rundir); } if ((error = basic_test()) || -- cgit