d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

ctm: fix memory leaks in Pass1(..)

Browse Source 
Call `Delete(..)` (a free(3) wrapper) on `name` when bailing from the
function.

Submitted by:	Tom Rix <trix@juniper.net>
Reviewed by:	ngie
Differential Revision:	D10097
Sponsored by:	Dell EMC Isilon, Juniper
This commit is contained in:
ngie 2017-03-22 17:28:53 +00:00
parent 2e1b5200e9
commit ef74488e54
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
usr.sbin/ctm/ctm/ctm_pass1.c
View File

@ -114,6 +114,7 @@ Pass1(FILE *fd, unsigned applied)
}
if (name[0] == '/') {
Fatal("Absolute paths are illegal.");
Delete(name);
return Exit_Mess;
}
q = name;
@ -121,6 +122,7 @@ Pass1(FILE *fd, unsigned applied)
if (q[0] == '.' && q[1] == '.')
if (q[2] == '/' || q[2] == '\0') {
Fatal("Paths containing '..' are illegal.");
Delete(name);
return Exit_Mess;
}
if ((q = strchr(q, '/')) == NULL)