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/csstidy-1.4/csstidy/background.cpp Examining data/csstidy-1.4/csstidy/background.hpp Examining data/csstidy-1.4/csstidy/conversions.cpp Examining data/csstidy-1.4/csstidy/conversions.hpp Examining data/csstidy-1.4/csstidy/cssopt.cpp Examining data/csstidy-1.4/csstidy/cssopt.hpp Examining data/csstidy-1.4/csstidy/csspp_globals.cpp Examining data/csstidy-1.4/csstidy/csspp_globals.hpp Examining data/csstidy-1.4/csstidy/csspp_private.h Examining data/csstidy-1.4/csstidy/csstidy.cpp Examining data/csstidy-1.4/csstidy/csstidy.hpp Examining data/csstidy-1.4/csstidy/datastruct.hpp Examining data/csstidy-1.4/csstidy/file_functions.cpp Examining data/csstidy-1.4/csstidy/file_functions.hpp Examining data/csstidy-1.4/csstidy/important.cpp Examining data/csstidy-1.4/csstidy/important.hpp Examining data/csstidy-1.4/csstidy/main.cpp Examining data/csstidy-1.4/csstidy/misc.hpp Examining data/csstidy-1.4/csstidy/parse_css.cpp Examining data/csstidy-1.4/csstidy/parse_css.hpp Examining data/csstidy-1.4/csstidy/prepare.cpp Examining data/csstidy-1.4/csstidy/prepare.hpp Examining data/csstidy-1.4/csstidy/trim.cpp Examining data/csstidy-1.4/csstidy/trim.hpp Examining data/csstidy-1.4/csstidy/umap.cpp Examining data/csstidy-1.4/csstidy/umap.hpp Examining data/csstidy-1.4/csstidy/print_css.cpp Examining data/csstidy-1.4/csstidy/misc.cpp FINAL RESULTS: data/csstidy-1.4/csstidy/cssopt.cpp:169:38: [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). color_tmp[i] = f2str(round(255 * atoi(color_tmp[i].c_str())/100,0)); data/csstidy-1.4/csstidy/cssopt.cpp:171:7: [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). if(atoi(color_tmp[i].c_str()) > 255) color_tmp[i] = 255; data/csstidy-1.4/csstidy/cssopt.cpp:177:7: [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). if(atoi(color_tmp[i].c_str()) < 16) data/csstidy-1.4/csstidy/cssopt.cpp:179:27: [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). color += "0" + dechex(atoi(color_tmp[i].c_str())); data/csstidy-1.4/csstidy/cssopt.cpp:183:21: [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). color += dechex(atoi(color_tmp[i].c_str())); data/csstidy-1.4/csstidy/print_css.cpp:87:15: [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). file_output.open(filename.c_str(),ios::binary); data/csstidy-1.4/csstidy/misc.cpp:125:21: [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). for(int i = 0; i < strlen(haystack); ++i) data/csstidy-1.4/csstidy/misc.cpp:190:8: [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). while(strlen > 0) data/csstidy-1.4/csstidy/misc.cpp:196:6: [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). --strlen; ++strpos; data/csstidy-1.4/csstidy/misc.cpp:199:5: [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). --strlen; ++strpos; data/csstidy-1.4/csstidy/misc.cpp:208:6: [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). --strlen; ++strpos; ANALYSIS SUMMARY: Hits = 11 Lines analyzed = 3475 in approximately 0.15 seconds (23901 lines/second) Physical Source Lines of Code (SLOC) = 2587 Hits@level = [0] 0 [1] 5 [2] 6 [3] 0 [4] 0 [5] 0 Hits@level+ = [0+] 11 [1+] 11 [2+] 6 [3+] 0 [4+] 0 [5+] 0 Hits/KSLOC@level+ = [0+] 4.25203 [1+] 4.25203 [2+] 2.31929 [3+] 0 [4+] 0 [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.