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/pd-lua-0.7.3/pdlua.c FINAL RESULTS: data/pd-lua-0.7.3/pdlua.c:39:9: [4] (format) snprintf: If format strings can be influenced by an attacker, they can be exploited, and note that sprintf variations do not always \0-terminate (CWE-134). Use a constant for the format specification. #define snprintf _snprintf data/pd-lua-0.7.3/pdlua.c:39:18: [4] (format) _snprintf: If format strings can be influenced by an attacker, they can be exploited, and note that sprintf variations do not always \0-terminate (CWE-134). Use a constant for the format specification. #define snprintf _snprintf data/pd-lua-0.7.3/pdlua.c:499:17: [4] (format) printf: If format strings can be influenced by an attacker, they can be exploited (CWE-134). Use a constant for the format specification. printf(lua_toboolean(L, i) ? "true" : "false"); data/pd-lua-0.7.3/pdlua.c:542:13: [4] (buffer) sprintf: Does not check for buffer overflows (CWE-120). Use sprintf_s, snprintf, or vsnprintf. sprintf(pathname, "%s", name); data/pd-lua-0.7.3/pdlua.c:558:13: [4] (buffer) sprintf: Does not check for buffer overflows (CWE-120). Use sprintf_s, snprintf, or vsnprintf. sprintf(pathname, "%s/%s", path, name); data/pd-lua-0.7.3/pdlua.c:1671:5: [4] (buffer) sprintf: Does not check for buffer overflows (CWE-120). Use sprintf_s, snprintf, or vsnprintf. sprintf(pd_lua_path, "%s/pd.lua", pdlua_proxyinlet_class->c_externdir->s_name); /* the full path to pd.lua */ data/pd-lua-0.7.3/pdlua.c:102: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 buffer[MAXPDSTRING]; /**< Buffer to read into. */ data/pd-lua-0.7.3/pdlua.c:521: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 pathname[FILENAME_MAX]; data/pd-lua-0.7.3/pdlua.c:1381: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 buf[MAXPDSTRING]; data/pd-lua-0.7.3/pdlua.c:1568: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 dirbuf[MAXPDSTRING]; data/pd-lua-0.7.3/pdlua.c:1589: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 dirbuf[MAXPDSTRING]; data/pd-lua-0.7.3/pdlua.c:1618: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 pd_lua_path[MAXPDSTRING]; data/pd-lua-0.7.3/pdlua.c:1624: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 compiled[MAXPDSTRING]; data/pd-lua-0.7.3/pdlua.c:1625: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 luaversionStr[MAXPDSTRING]; data/pd-lua-0.7.3/pdlua.c:1673:10: [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). fd = open(pd_lua_path, O_RDONLY); data/pd-lua-0.7.3/pdlua.c:36:9: [1] (buffer) read: Check buffer boundaries if used in a loop including recursive loops (CWE-120, CWE-20). #define read _read data/pd-lua-0.7.3/pdlua.c:261:9: [1] (buffer) read: Check buffer boundaries if used in a loop including recursive loops (CWE-120, CWE-20). s = read(r->fd, r->buffer, MAXPDSTRING-2); data/pd-lua-0.7.3/pdlua.c:539: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). if (name[strlen(name)-1] == 'x') data/pd-lua-0.7.3/pdlua.c:989:29: [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). if (strlen(s) != sl) pd_error(o, "lua: warning: symbol munged (contains \\0 in body)"); data/pd-lua-0.7.3/pdlua.c:1067:29: [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). if (strlen(s) != sl) pd_error(o, "lua: warning: symbol munged (contains \\0 in body)"); data/pd-lua-0.7.3/pdlua.c:1119:17: [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). if (strlen(receivename) != receivenamel) error("lua: warning: symbol munged (contains \\0 in body)"); data/pd-lua-0.7.3/pdlua.c:1126:25: [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). if (strlen(selname) != selnamel) error("lua: warning: symbol munged (contains \\0 in body)"); ANALYSIS SUMMARY: Hits = 22 Lines analyzed = 1722 in approximately 0.06 seconds (29189 lines/second) Physical Source Lines of Code (SLOC) = 1331 Hits@level = [0] 7 [1] 7 [2] 9 [3] 0 [4] 6 [5] 0 Hits@level+ = [0+] 29 [1+] 22 [2+] 15 [3+] 6 [4+] 6 [5+] 0 Hits/KSLOC@level+ = [0+] 21.7881 [1+] 16.5289 [2+] 11.2697 [3+] 4.50789 [4+] 4.50789 [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.