pips/stt
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Address memory leaks

Browse source
This commit is contained in:
DUVILLIE Guillerme 2019-10-21 11:38:47 +02:00
parent 84a27df29d
commit 683e730734
1 changed files with 38 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

39
stt.c
View file

@ -17,6 +17,10 @@
#define VERSION "dev" #define VERSION "dev"
#endif #endif
#define free_and_null(x) do { \
if (x) { free (x); x = NULL; } \
} while (0);
char *argv0; char *argv0;
static char *timesfile = ".ttimes"; static char *timesfile = ".ttimes";
@ -45,6 +49,7 @@ int timesnode_getbalance(struct timesnode *);
struct timesnode *timesnode_rightrotate(struct timesnode *); struct timesnode *timesnode_rightrotate(struct timesnode *);
struct timesnode *timesnode_leftrotate(struct timesnode *); struct timesnode *timesnode_leftrotate(struct timesnode *);
void timesnode_print(struct timesnode *, time_t); void timesnode_print(struct timesnode *, time_t);
void timesnode_free(struct timesnode *);
void void
usage() usage()
@ -86,7 +91,7 @@ opentimesfile()
struct timesnode * struct timesnode *
loadtimes(FILE * fp, struct timesnode * p) loadtimes(FILE * fp, struct timesnode * p)
{ {
char *readline, *line, *tmp; char *readline, *line, *tmp, *tmpline;
time_t starttime, endtime; time_t starttime, endtime;
char *task; char *task;
size_t linesize; size_t linesize;
@ -98,12 +103,15 @@ loadtimes(FILE * fp, struct timesnode * p)
linesize = 0; linesize = 0;
while ((linelen = getline(&readline, &linesize, fp)) != -1) { while ((linelen = getline(&readline, &linesize, fp)) != -1) {
line = strdup(readline); line = strdup(readline);
tmpline = line; /* Stores the pointer to allow buffer free */
tmp = strsep(&line, ";"); tmp = strsep(&line, ";");
starttime = atoi(tmp); starttime = atoi(tmp);
if (starttime == 0) { if (starttime == 0) {
free_and_null (tmpline);
continue; continue;
} }
tmp = strsep(&line, ";"); tmp = strsep(&line, ";");
endtime = atoi(tmp); endtime = atoi(tmp);
@ -111,8 +119,13 @@ loadtimes(FILE * fp, struct timesnode * p)
task[strlen(task) - 1] = '\0'; task[strlen(task) - 1] = '\0';
p = timesnode_add(p, task, starttime, endtime); p = timesnode_add(p, task, starttime, endtime);
free_and_null (task);
free_and_null (tmpline);
} }
free_and_null (readline);
return p; return p;
} }
@ -291,6 +304,23 @@ timesnode_print(struct timesnode * p, time_t aftertime)
} }
} }
void
timesnode_free (struct timesnode *p)
{
if (p->left) {
timesnode_free (p->left);
p->left = NULL;
}
if (p->right) {
timesnode_free (p->right);
p->right = NULL;
}
free_and_null (p->task);
free_and_null (p);
}
int int
main(int argc, char *argv[]) main(int argc, char *argv[])
{ {
@ -362,5 +392,12 @@ default:
timesnode_print(timestree, mktime(&today)); timesnode_print(timestree, mktime(&today));
} }
fclose (fp);
timesnode_free (timestree);
free_and_null (nowtime);
return 0; return 0;
} }