Flawfinder version 2.0.10, (C) 2001-2019 David A. Wheeler.
Number of rules (primarily dangerous function names) in C/C++ ruleset: 223
Examining data/deheader-1.7/test/chmod.c
Examining data/deheader-1.7/test/abs.c
Examining data/deheader-1.7/test/clock_getres.c
Examining data/deheader-1.7/test/atexit.c
Examining data/deheader-1.7/test/atan.c
Examining data/deheader-1.7/test/chroot.c
Examining data/deheader-1.7/test/clock.c
Examining data/deheader-1.7/test/bcmp.c
Examining data/deheader-1.7/test/ctime.c
Examining data/deheader-1.7/test/a64l.c
Examining data/deheader-1.7/test/acosh.c
Examining data/deheader-1.7/test/alarm.c
Examining data/deheader-1.7/test/ceil.c
Examining data/deheader-1.7/test/btowc.c
Examining data/deheader-1.7/test/advance.c
Examining data/deheader-1.7/test/duplicate.c
Examining data/deheader-1.7/test/umask.c
Examining data/deheader-1.7/test/abort.c
Examining data/deheader-1.7/test/fchown.c
Examining data/deheader-1.7/test/cfsetospeed.c
Examining data/deheader-1.7/test/bcopy.c
Examining data/deheader-1.7/test/brk.c
Examining data/deheader-1.7/test/atan2.c
Examining data/deheader-1.7/test/atoi.c
Examining data/deheader-1.7/test/clock_settime.c
Examining data/deheader-1.7/test/cfgetospeed.c
Examining data/deheader-1.7/test/atol.c
Examining data/deheader-1.7/test/atanh.c
Examining data/deheader-1.7/test/cfgetispeed.c
Examining data/deheader-1.7/test/fchmod.c
Examining data/deheader-1.7/test/clock_gettime.c
Examining data/deheader-1.7/test/string.c
Examining data/deheader-1.7/test/sbrk.c
Examining data/deheader-1.7/test/catopen.c
Examining data/deheader-1.7/test/cfsetispeed.c
Examining data/deheader-1.7/test/atof.c
Examining data/deheader-1.7/test/asin.c
Examining data/deheader-1.7/test/access.c
Examining data/deheader-1.7/test/noheaders.c
Examining data/deheader-1.7/test/chown.c
Examining data/deheader-1.7/test/fstat.c
Examining data/deheader-1.7/test/fclose.c
Examining data/deheader-1.7/test/chdir.c
Examining data/deheader-1.7/test/bzero.c
Examining data/deheader-1.7/test/close.c
Examining data/deheader-1.7/test/bsort.c
Examining data/deheader-1.7/test/asinh.c
Examining data/deheader-1.7/test/cbrt.c
Examining data/deheader-1.7/test/calloc.c
Examining data/deheader-1.7/test/closedir.c
Examining data/deheader-1.7/test/crypt.c
Examining data/deheader-1.7/test/clearerr.c
Examining data/deheader-1.7/test/catclose.c
Examining data/deheader-1.7/test/catgets.c
Examining data/deheader-1.7/test/bsd_signal.c
Examining data/deheader-1.7/test/basename.c
Examining data/deheader-1.7/test/acos.c
FINAL RESULTS:
data/deheader-1.7/test/chmod.c:12:11: [5] (race) chmod:
This accepts filename arguments; if an attacker can move those files, a
race condition results. (CWE-362). Use fchmod( ) instead.
(void)chmod("/", 0);
data/deheader-1.7/test/chown.c:12:11: [5] (race) chown:
This accepts filename arguments; if an attacker can move those files, a
race condition results. (CWE-362). Use fchown( ) instead.
(void)chown("/", 0, 0);
data/deheader-1.7/test/access.c:16:11: [4] (race) access:
This usually indicates a security flaw. If an attacker can change anything
along the path between the call to access() and the file's actual use
(e.g., by moving files), the attacker can exploit the race condition
(CWE-362/CWE-367!). Set up the correct permissions (e.g., using setuid())
and try to open the file directly.
(void)access("/dev/null", 0);
data/deheader-1.7/test/bsort.c:36:12: [4] (buffer) scanf:
The scanf() family's %s operation, without a limit specification, permits
buffer overflows (CWE-120, CWE-20). Specify a limit to %s, or use a
different input function.
while (scanf("%s", node.string) != EOF) {
data/deheader-1.7/test/crypt.c:11:11: [4] (crypto) crypt:
The crypt functions use a poor one-way hashing algorithm; since they only
accept passwords of 8 characters or fewer and only a two-byte salt, they
are excessively vulnerable to dictionary attacks given today's faster
computing equipment (CWE-327). Use a different algorithm, such as SHA-256,
with a larger, non-repeating salt.
(void)crypt("foo", "salt");
data/deheader-1.7/test/chroot.c:11:11: [3] (misc) chroot:
chroot can be very helpful, but is hard to use correctly (CWE-250, CWE-22).
Make sure the program immediately chdir("/"), closes file descriptors, and
drops root privileges, and that all necessary files (and no more!) are in
the new root.
(void)chroot("/");
data/deheader-1.7/test/atoi.c:11:12: [2] (integer) atoi:
Unless checked, the resulting number can exceed the expected range
(CWE-190). If source untrusted, check both minimum and maximum, even if the
input had no minus sign (large numbers can roll over into negative number;
consider saving to an unsigned value if that is intended).
(void) atoi("2317");
data/deheader-1.7/test/atol.c:11:12: [2] (integer) atol:
Unless checked, the resulting number can exceed the expected range
(CWE-190). If source untrusted, check both minimum and maximum, even if the
input had no minus sign (large numbers can roll over into negative number;
consider saving to an unsigned value if that is intended).
(void) atol("2317");
data/deheader-1.7/test/bcopy.c:11:12: [2] (buffer) bcopy:
Does not check for buffer overflows when copying to destination (CWE-120).
Make sure destination can always hold the source data.
(void) bcopy(0, 0, 0);
data/deheader-1.7/test/bsort.c:33:5: [2] (buffer) char:
Statically-sized arrays can be improperly restricted, leading to potential
overflows or other issues (CWE-119!/CWE-120). Perform bounds checking, use
functions that limit length, or ensure that the size is larger than the
maximum possible length.
char str_space[20]; /* space to read string into */
data/deheader-1.7/test/advance.c:8:16: [1] (buffer) getc:
Check buffer boundaries if used in a loop including recursive loops
(CWE-120, CWE-20).
#define GETC() getc()
data/deheader-1.7/test/umask.c:13:5: [1] (access) umask:
Ensure that umask is given most restrictive possible setting (e.g., 066 or
077) (CWE-732).
umask(777);
ANALYSIS SUMMARY:
Hits = 12
Lines analyzed = 740 in approximately 0.07 seconds (9920 lines/second)
Physical Source Lines of Code (SLOC) = 334
Hits@level = [0] 2 [1] 2 [2] 4 [3] 1 [4] 3 [5] 2
Hits@level+ = [0+] 14 [1+] 12 [2+] 10 [3+] 6 [4+] 5 [5+] 2
Hits/KSLOC@level+ = [0+] 41.9162 [1+] 35.9281 [2+] 29.9401 [3+] 17.9641 [4+] 14.9701 [5+] 5.98802
Dot directories skipped = 1 (--followdotdir overrides)
Minimum risk level = 1
Not every hit is necessarily a security vulnerability.
There may be other security vulnerabilities; review your code!
See 'Secure Programming HOWTO'
(https://dwheeler.com/secure-programs) for more information.