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/clipit-1.4.4+git20190202/src/clipit-i18n.h Examining data/clipit-1.4.4+git20190202/src/daemon.c Examining data/clipit-1.4.4+git20190202/src/daemon.h Examining data/clipit-1.4.4+git20190202/src/eggaccelerators.c Examining data/clipit-1.4.4+git20190202/src/eggaccelerators.h Examining data/clipit-1.4.4+git20190202/src/history.c Examining data/clipit-1.4.4+git20190202/src/history.h Examining data/clipit-1.4.4+git20190202/src/keybinder.c Examining data/clipit-1.4.4+git20190202/src/keybinder.h Examining data/clipit-1.4.4+git20190202/src/main.c Examining data/clipit-1.4.4+git20190202/src/main.h Examining data/clipit-1.4.4+git20190202/src/manage.c Examining data/clipit-1.4.4+git20190202/src/manage.h Examining data/clipit-1.4.4+git20190202/src/preferences.c Examining data/clipit-1.4.4+git20190202/src/preferences.h Examining data/clipit-1.4.4+git20190202/src/utils.c Examining data/clipit-1.4.4+git20190202/src/utils.h FINAL RESULTS: data/clipit-1.4.4+git20190202/src/eggaccelerators.c:404:7: [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 (accelerator + l, text_release); data/clipit-1.4.4+git20190202/src/eggaccelerators.c:409:7: [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 (accelerator + l, text_shift); data/clipit-1.4.4+git20190202/src/eggaccelerators.c:414:7: [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 (accelerator + l, text_control); data/clipit-1.4.4+git20190202/src/eggaccelerators.c:419:7: [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 (accelerator + l, text_mod1); data/clipit-1.4.4+git20190202/src/eggaccelerators.c:424:7: [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 (accelerator + l, text_mod2); data/clipit-1.4.4+git20190202/src/eggaccelerators.c:429:7: [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 (accelerator + l, text_mod3); data/clipit-1.4.4+git20190202/src/eggaccelerators.c:434:7: [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 (accelerator + l, text_mod4); data/clipit-1.4.4+git20190202/src/eggaccelerators.c:439:7: [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 (accelerator + l, text_mod5); data/clipit-1.4.4+git20190202/src/eggaccelerators.c:444:7: [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 (accelerator + l, text_meta); data/clipit-1.4.4+git20190202/src/eggaccelerators.c:449:7: [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 (accelerator + l, text_hyper); data/clipit-1.4.4+git20190202/src/eggaccelerators.c:454:7: [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 (accelerator + l, text_super); data/clipit-1.4.4+git20190202/src/eggaccelerators.c:458:3: [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 (accelerator + l, keyval_name); data/clipit-1.4.4+git20190202/src/history.c:62:24: [2] (misc) fopen: 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 *excludes_file = fopen(path, "rb"); data/clipit-1.4.4+git20190202/src/history.c:101:23: [2] (misc) fopen: 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 *history_file = fopen(history_path, "rb"); data/clipit-1.4.4+git20190202/src/history.c:113:4: [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 extra_data[64]; data/clipit-1.4.4+git20190202/src/history.c:177:24: [2] (misc) fopen: 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 *history_file = fopen(history_path, "wb"); data/clipit-1.4.4+git20190202/src/main.c:504:24: [2] (misc) fopen: 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* actions_file = fopen(path, "rb"); data/clipit-1.4.4+git20190202/src/preferences.c:220:28: [2] (misc) fopen: 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 *history_file = fopen(history_path, "wb"); data/clipit-1.4.4+git20190202/src/preferences.c:315:24: [2] (misc) fopen: 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* actions_file = fopen(path, "rb"); data/clipit-1.4.4+git20190202/src/preferences.c:356:24: [2] (misc) fopen: 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* actions_file = fopen(path, "wb"); data/clipit-1.4.4+git20190202/src/preferences.c:587:25: [2] (misc) fopen: 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* excludes_file = fopen(path, "rb"); data/clipit-1.4.4+git20190202/src/preferences.c:622:25: [2] (misc) fopen: 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* excludes_file = fopen(path, "wb"); data/clipit-1.4.4+git20190202/src/eggaccelerators.c:224:9: [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). len = strlen (accelerator); data/clipit-1.4.4+git20190202/src/eggaccelerators.c:396: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). l += strlen (keyval_name); data/clipit-1.4.4+git20190202/src/history.c:227: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). if(strlen(item) > ENTRY_MAX_SIZE) data/clipit-1.4.4+git20190202/src/main.c:153:34: [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 (prefs.exclude_windows && strlen(prefs.exclude_windows) > 0) data/clipit-1.4.4+git20190202/src/main.c:609: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). int end = start + strlen(input_buffer); ANALYSIS SUMMARY: Hits = 27 Lines analyzed = 4958 in approximately 0.16 seconds (31503 lines/second) Physical Source Lines of Code (SLOC) = 3564 Hits@level = [0] 26 [1] 5 [2] 10 [3] 0 [4] 12 [5] 0 Hits@level+ = [0+] 53 [1+] 27 [2+] 22 [3+] 12 [4+] 12 [5+] 0 Hits/KSLOC@level+ = [0+] 14.8709 [1+] 7.57576 [2+] 6.17284 [3+] 3.367 [4+] 3.367 [5+] 0 Dot directories skipped = 2 (--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.