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/animals-201207131226/Animal.h Examining data/animals-201207131226/Question.h Examining data/animals-201207131226/Record.h Examining data/animals-201207131226/db++-stuff.h Examining data/animals-201207131226/db2++-stuff.cc Examining data/animals-201207131226/db2++-stuff.h Examining data/animals-201207131226/db4++-stuff.cc Examining data/animals-201207131226/db4++-stuff.h Examining data/animals-201207131226/db4.0++-stuff.cc Examining data/animals-201207131226/db4.1++-stuff.cc Examining data/animals-201207131226/db4.1++-stuff.h Examining data/animals-201207131226/gdbm-stuff.cc Examining data/animals-201207131226/gdbm-stuff.h Examining data/animals-201207131226/main.cc Examining data/animals-201207131226/main.h Examining data/animals-201207131226/str-conversions.cc Examining data/animals-201207131226/str-conversions.h Examining data/animals-201207131226/test-str-cvt.cc Examining data/animals-201207131226/test.cc Examining data/animals-201207131226/tut.cc Examining data/animals-201207131226/util.cc Examining data/animals-201207131226/util.h FINAL RESULTS: data/animals-201207131226/gdbm-stuff.cc:59:5: [4] (buffer) strcpy: Does not check for buffer overflows when copying to destination [MS-banned] (CWE-120). Consider using snprintf, strcpy_s, or strlcpy (warning: strncpy easily misused). strcpy(tmp, orig.c_str()); data/animals-201207131226/gdbm-stuff.cc:71:5: [4] (buffer) strcpy: Does not check for buffer overflows when copying to destination [MS-banned] (CWE-120). Consider using snprintf, strcpy_s, or strlcpy (warning: strncpy easily misused). strcpy(tmp, orig); data/animals-201207131226/str-conversions.cc:51:5: [4] (buffer) strcpy: Does not check for buffer overflows when copying to destination [MS-banned] (CWE-120). Consider using snprintf, strcpy_s, or strlcpy (warning: strncpy easily misused). strcpy(tmp, orig.c_str()); data/animals-201207131226/str-conversions.cc:63:5: [4] (buffer) strcpy: Does not check for buffer overflows when copying to destination [MS-banned] (CWE-120). Consider using snprintf, strcpy_s, or strlcpy (warning: strncpy easily misused). strcpy(tmp, orig); data/animals-201207131226/str-conversions.cc:81:5: [4] (buffer) strcpy: Does not check for buffer overflows when copying to destination [MS-banned] (CWE-120). Consider using snprintf, strcpy_s, or strlcpy (warning: strncpy easily misused). strcpy(tmp, theStr); data/animals-201207131226/db2++-stuff.cc:118:24: [2] (misc) open: Check when opening files - can an attacker redirect it (via symlinks), force the opening of special file type (e.g., device files), move things around to create a race condition, control its ancestors, or change its contents? (CWE-362). createBad = Db::open data/animals-201207131226/db2++-stuff.cc:186:18: [2] (misc) open: Check when opening files - can an attacker redirect it (via symlinks), force the opening of special file type (e.g., device files), move things around to create a race condition, control its ancestors, or change its contents? (CWE-362). openBad = Db::open data/animals-201207131226/db4++-stuff.cc:122:29: [2] (misc) open: Check when opening files - can an attacker redirect it (via symlinks), force the opening of special file type (e.g., device files), move things around to create a race condition, control its ancestors, or change its contents? (CWE-362). createBad = theFile->open data/animals-201207131226/db4++-stuff.cc:191:23: [2] (misc) open: Check when opening files - can an attacker redirect it (via symlinks), force the opening of special file type (e.g., device files), move things around to create a race condition, control its ancestors, or change its contents? (CWE-362). openBad = theFile->open data/animals-201207131226/db4.0++-stuff.cc:120:29: [2] (misc) open: Check when opening files - can an attacker redirect it (via symlinks), force the opening of special file type (e.g., device files), move things around to create a race condition, control its ancestors, or change its contents? (CWE-362). createBad = theFile->open data/animals-201207131226/db4.0++-stuff.cc:186:23: [2] (misc) open: Check when opening files - can an attacker redirect it (via symlinks), force the opening of special file type (e.g., device files), move things around to create a race condition, control its ancestors, or change its contents? (CWE-362). openBad = theFile->open data/animals-201207131226/db4.1++-stuff.cc:120:29: [2] (misc) open: Check when opening files - can an attacker redirect it (via symlinks), force the opening of special file type (e.g., device files), move things around to create a race condition, control its ancestors, or change its contents? (CWE-362). createBad = theFile->open data/animals-201207131226/db4.1++-stuff.cc:187:23: [2] (misc) open: Check when opening files - can an attacker redirect it (via symlinks), force the opening of special file type (e.g., device files), move things around to create a race condition, control its ancestors, or change its contents? (CWE-362). openBad = theFile->open data/animals-201207131226/gdbm-stuff.cc:87:5: [2] (buffer) memcpy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. memcpy(tmp, orig.str(), stringSize); data/animals-201207131226/test.cc:39:3: [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 ret[3]; data/animals-201207131226/tut.cc:18:17: [2] (misc) open: Check when opening files - can an attacker redirect it (via symlinks), force the opening of special file type (e.g., device files), move things around to create a race condition, control its ancestors, or change its contents? (CWE-362). openBad = Db::open(DATABASE, DB_BTREE, DB_CREATE, 0600, NULL, NULL, &dbp); data/animals-201207131226/db2++-stuff.cc:42:16: [1] (buffer) strlen: Does not handle strings that are not \0-terminated; if given one it may perform an over-read (it could cause a crash if unprotected) (CWE-126). key.set_size(strlen(theKey.c_str()) + 1); data/animals-201207131226/db2++-stuff.cc:45:18: [1] (buffer) strlen: Does not handle strings that are not \0-terminated; if given one it may perform an over-read (it could cause a crash if unprotected) (CWE-126). value.set_size(strlen(theValue.c_str()) + 1); data/animals-201207131226/db2++-stuff.cc:71:16: [1] (buffer) strlen: Does not handle strings that are not \0-terminated; if given one it may perform an over-read (it could cause a crash if unprotected) (CWE-126). key.set_size(strlen(theKey.c_str()) + 1); data/animals-201207131226/db2++-stuff.cc:114:22: [1] (access) umask: Ensure that umask is given most restrictive possible setting (e.g., 066 or 077) (CWE-732). mode_t saveMode = umask(0); data/animals-201207131226/db2++-stuff.cc:136:4: [1] (access) umask: Ensure that umask is given most restrictive possible setting (e.g., 066 or 077) (CWE-732). umask(saveMode); data/animals-201207131226/db4++-stuff.cc:45:16: [1] (buffer) strlen: Does not handle strings that are not \0-terminated; if given one it may perform an over-read (it could cause a crash if unprotected) (CWE-126). key.set_size(strlen(theKey.c_str()) + 1); data/animals-201207131226/db4++-stuff.cc:48:18: [1] (buffer) strlen: Does not handle strings that are not \0-terminated; if given one it may perform an over-read (it could cause a crash if unprotected) (CWE-126). value.set_size(strlen(theValue.c_str()) + 1); data/animals-201207131226/db4++-stuff.cc:74:16: [1] (buffer) strlen: Does not handle strings that are not \0-terminated; if given one it may perform an over-read (it could cause a crash if unprotected) (CWE-126). key.set_size(strlen(theKey.c_str()) + 1); data/animals-201207131226/db4++-stuff.cc:117:22: [1] (access) umask: Ensure that umask is given most restrictive possible setting (e.g., 066 or 077) (CWE-732). mode_t saveMode = umask(0); data/animals-201207131226/db4++-stuff.cc:140:4: [1] (access) umask: Ensure that umask is given most restrictive possible setting (e.g., 066 or 077) (CWE-732). umask(saveMode); data/animals-201207131226/db4.0++-stuff.cc:43:16: [1] (buffer) strlen: Does not handle strings that are not \0-terminated; if given one it may perform an over-read (it could cause a crash if unprotected) (CWE-126). key.set_size(strlen(theKey.c_str()) + 1); data/animals-201207131226/db4.0++-stuff.cc:46:18: [1] (buffer) strlen: Does not handle strings that are not \0-terminated; if given one it may perform an over-read (it could cause a crash if unprotected) (CWE-126). value.set_size(strlen(theValue.c_str()) + 1); data/animals-201207131226/db4.0++-stuff.cc:72:16: [1] (buffer) strlen: Does not handle strings that are not \0-terminated; if given one it may perform an over-read (it could cause a crash if unprotected) (CWE-126). key.set_size(strlen(theKey.c_str()) + 1); data/animals-201207131226/db4.0++-stuff.cc:115:22: [1] (access) umask: Ensure that umask is given most restrictive possible setting (e.g., 066 or 077) (CWE-732). mode_t saveMode = umask(0); data/animals-201207131226/db4.0++-stuff.cc:135:4: [1] (access) umask: Ensure that umask is given most restrictive possible setting (e.g., 066 or 077) (CWE-732). umask(saveMode); data/animals-201207131226/db4.1++-stuff.cc:43:16: [1] (buffer) strlen: Does not handle strings that are not \0-terminated; if given one it may perform an over-read (it could cause a crash if unprotected) (CWE-126). key.set_size(strlen(theKey.c_str()) + 1); data/animals-201207131226/db4.1++-stuff.cc:46:18: [1] (buffer) strlen: Does not handle strings that are not \0-terminated; if given one it may perform an over-read (it could cause a crash if unprotected) (CWE-126). value.set_size(strlen(theValue.c_str()) + 1); data/animals-201207131226/db4.1++-stuff.cc:72:16: [1] (buffer) strlen: Does not handle strings that are not \0-terminated; if given one it may perform an over-read (it could cause a crash if unprotected) (CWE-126). key.set_size(strlen(theKey.c_str()) + 1); data/animals-201207131226/db4.1++-stuff.cc:115:22: [1] (access) umask: Ensure that umask is given most restrictive possible setting (e.g., 066 or 077) (CWE-732). mode_t saveMode = umask(0); data/animals-201207131226/db4.1++-stuff.cc:136:4: [1] (access) umask: Ensure that umask is given most restrictive possible setting (e.g., 066 or 077) (CWE-732). umask(saveMode); data/animals-201207131226/gdbm-stuff.cc:67:24: [1] (buffer) strlen: Does not handle strings that are not \0-terminated; if given one it may perform an over-read (it could cause a crash if unprotected) (CWE-126). char *tmp = new char[strlen(orig) + 1]; data/animals-201207131226/gdbm-stuff.cc:107:22: [1] (access) umask: Ensure that umask is given most restrictive possible setting (e.g., 066 or 077) (CWE-732). mode_t saveMode = umask(0); data/animals-201207131226/gdbm-stuff.cc:118:4: [1] (access) umask: Ensure that umask is given most restrictive possible setting (e.g., 066 or 077) (CWE-732). umask(saveMode); data/animals-201207131226/gdbm-stuff.cc:223:19: [1] (buffer) strlen: Does not handle strings that are not \0-terminated; if given one it may perform an over-read (it could cause a crash if unprotected) (CWE-126). content.dsize = strlen(content.dptr) + 1; data/animals-201207131226/gdbm-stuff.cc:258:15: [1] (buffer) strlen: Does not handle strings that are not \0-terminated; if given one it may perform an over-read (it could cause a crash if unprotected) (CWE-126). key.dsize = strlen(key.dptr) + 1; data/animals-201207131226/str-conversions.cc:59:24: [1] (buffer) strlen: Does not handle strings that are not \0-terminated; if given one it may perform an over-read (it could cause a crash if unprotected) (CWE-126). char *tmp = new char[strlen(orig) + 1]; data/animals-201207131226/str-conversions.cc:76:20: [1] (buffer) strlen: Does not handle strings that are not \0-terminated; if given one it may perform an over-read (it could cause a crash if unprotected) (CWE-126). int stringSize = strlen(theStr); ANALYSIS SUMMARY: Hits = 43 Lines analyzed = 3285 in approximately 0.08 seconds (43389 lines/second) Physical Source Lines of Code (SLOC) = 1809 Hits@level = [0] 9 [1] 27 [2] 11 [3] 0 [4] 5 [5] 0 Hits@level+ = [0+] 52 [1+] 43 [2+] 16 [3+] 5 [4+] 5 [5+] 0 Hits/KSLOC@level+ = [0+] 28.7452 [1+] 23.77 [2+] 8.84467 [3+] 2.76396 [4+] 2.76396 [5+] 0 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.