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/gnome-themes-extra-3.28/themes/Adwaita/gtk-2.0/adwaita_engine.c Examining data/gnome-themes-extra-3.28/themes/HighContrast/create-highcontrast.c FINAL RESULTS: data/gnome-themes-extra-3.28/themes/HighContrast/create-highcontrast.c:34:7: [4] (buffer) sprintf: Does not check for buffer overflows (CWE-120). Use sprintf_s, snprintf, or vsnprintf. sprintf (buf + (ptr - str), "%s%s", new_substr, ptr + strlen (substr)); data/gnome-themes-extra-3.28/themes/HighContrast/create-highcontrast.c:43:7: [4] (buffer) sprintf: Does not check for buffer overflows (CWE-120). Use sprintf_s, snprintf, or vsnprintf. sprintf (buf2 + (ptr - str), "%s%s", new_substr, ptr + strlen (substr)); data/gnome-themes-extra-3.28/themes/HighContrast/create-highcontrast.c:21:10: [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. static char buf[4096]; data/gnome-themes-extra-3.28/themes/HighContrast/create-highcontrast.c:38:14: [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. static char buf2[4096]; data/gnome-themes-extra-3.28/themes/HighContrast/create-highcontrast.c:29:3: [1] (buffer) strncpy: Easily used incorrectly; doesn't always \0-terminate or check for invalid pointers [MS-banned] (CWE-120). strncpy (buf, str, ptr - str); data/gnome-themes-extra-3.28/themes/HighContrast/create-highcontrast.c:32:7: [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 (substr) >=strlen (new_substr)) data/gnome-themes-extra-3.28/themes/HighContrast/create-highcontrast.c:32: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 (substr) >=strlen (new_substr)) data/gnome-themes-extra-3.28/themes/HighContrast/create-highcontrast.c:34:61: [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). sprintf (buf + (ptr - str), "%s%s", new_substr, ptr + strlen (substr)); data/gnome-themes-extra-3.28/themes/HighContrast/create-highcontrast.c:40:7: [1] (buffer) strncpy: Easily used incorrectly; doesn't always \0-terminate or check for invalid pointers [MS-banned] (CWE-120). strncpy (buf2, str, ptr - str + strlen (substr)); data/gnome-themes-extra-3.28/themes/HighContrast/create-highcontrast.c:40:39: [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). strncpy (buf2, str, ptr - str + strlen (substr)); data/gnome-themes-extra-3.28/themes/HighContrast/create-highcontrast.c:41: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). buf2[ptr - str + strlen (substr)] = '\0'; data/gnome-themes-extra-3.28/themes/HighContrast/create-highcontrast.c:43:62: [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). sprintf (buf2 + (ptr - str), "%s%s", new_substr, ptr + strlen (substr)); data/gnome-themes-extra-3.28/themes/HighContrast/create-highcontrast.c:44:7: [1] (buffer) strncpy: Easily used incorrectly; doesn't always \0-terminate or check for invalid pointers [MS-banned] (CWE-120). strncpy (buf, buf2, strlen (buf)); data/gnome-themes-extra-3.28/themes/HighContrast/create-highcontrast.c:44:27: [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). strncpy (buf, buf2, strlen (buf)); ANALYSIS SUMMARY: Hits = 14 Lines analyzed = 637 in approximately 5.64 seconds (113 lines/second) Physical Source Lines of Code (SLOC) = 484 Hits@level = [0] 0 [1] 10 [2] 2 [3] 0 [4] 2 [5] 0 Hits@level+ = [0+] 14 [1+] 14 [2+] 4 [3+] 2 [4+] 2 [5+] 0 Hits/KSLOC@level+ = [0+] 28.9256 [1+] 28.9256 [2+] 8.26446 [3+] 4.13223 [4+] 4.13223 [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.