emo/busybox
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

readprofile: do not close/free just before exiting

Browse Source 
function                                             old     new   delta
readprofile_main                                    1784    1762     -22

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
Denys Vlasenko 2017-08-06 20:39:27 +02:00
parent 277081e0a4
commit 035e71578e
2 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
NOFORK_NOEXEC.lst
View File

@ -48,7 +48,7 @@ ash - interactive, longterm
awk - noexec. runner
base64 - runner
basename - NOFORK
beep
beep - longterm: beep -r 999999999
blkdiscard - noexec. leaks: open+xioctl
blkid - noexec
blockdev - noexec. leaks fd
@ -127,7 +127,7 @@ findfs - suid
flash_eraseall
flash_lock
flash_unlock
flashcp
flashcp - needs ^C. flash writing may be slow, better to free memory memory by execing
flock - spawner, changes state (file locks), let's play safe and not be noexec
fold - noexec. runner
free - nofork candidate(struct globals, needs to close /proc/meminfo fd)
@ -278,7 +278,7 @@ raidautorun - noexec. very simple. leaks: open+xioctl
rdate - needs ^C (may talk to DNS servers, which may be down)
rdev - leaks: find_block_device -> readdir+xstrdup
readlink - NOFORK
readprofile
readprofile - reads /boot/System.map and /proc/profile, better to free more memory memory by execing?
realpath - NOFORK
reboot - rare
reformime - runner

6
util-linux/readprofile.c
View File

@ -266,8 +266,10 @@ int readprofile_main(int argc UNUSED_PARAM, char **argv)
printf("%6u %-40s %8.4f\n",
total, "total", total/(double)(fn_add-add0));
fclose(map);
free(buf);
if (ENABLE_FEATURE_CLEAN_UP) {
fclose(map);
free(buf);
}
return EXIT_SUCCESS;
}