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/libid3tag-0.15.1b/id3tag.h Examining data/libid3tag-0.15.1b/version.c Examining data/libid3tag-0.15.1b/ucs4.c Examining data/libid3tag-0.15.1b/latin1.c Examining data/libid3tag-0.15.1b/utf8.c Examining data/libid3tag-0.15.1b/render.c Examining data/libid3tag-0.15.1b/field.c Examining data/libid3tag-0.15.1b/frametype.c Examining data/libid3tag-0.15.1b/compat.c Examining data/libid3tag-0.15.1b/genre.c Examining data/libid3tag-0.15.1b/frame.c Examining data/libid3tag-0.15.1b/crc.c Examining data/libid3tag-0.15.1b/util.c Examining data/libid3tag-0.15.1b/tag.c Examining data/libid3tag-0.15.1b/file.c Examining data/libid3tag-0.15.1b/version.h Examining data/libid3tag-0.15.1b/ucs4.h Examining data/libid3tag-0.15.1b/latin1.h Examining data/libid3tag-0.15.1b/utf16.h Examining data/libid3tag-0.15.1b/utf8.h Examining data/libid3tag-0.15.1b/parse.h Examining data/libid3tag-0.15.1b/render.h Examining data/libid3tag-0.15.1b/field.h Examining data/libid3tag-0.15.1b/genre.h Examining data/libid3tag-0.15.1b/frame.h Examining data/libid3tag-0.15.1b/crc.h Examining data/libid3tag-0.15.1b/util.h Examining data/libid3tag-0.15.1b/tag.h Examining data/libid3tag-0.15.1b/file.h Examining data/libid3tag-0.15.1b/global.h Examining data/libid3tag-0.15.1b/debug.c Examining data/libid3tag-0.15.1b/debug.h Examining data/libid3tag-0.15.1b/msvc++/config.h Examining data/libid3tag-0.15.1b/utf16.c Examining data/libid3tag-0.15.1b/parse.c Examining data/libid3tag-0.15.1b/frametype.h Examining data/libid3tag-0.15.1b/compat.h FINAL RESULTS: data/libid3tag-0.15.1b/field.c:672: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(field->immediate.value, language); data/libid3tag-0.15.1b/field.c:80:5: [2] (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). Risk is low because the source is a constant string. strcpy(field->immediate.value, "XXX"); data/libid3tag-0.15.1b/field.c:84:5: [2] (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). Risk is low because the source is a constant string. strcpy(field->immediate.value, "XXXX"); data/libid3tag-0.15.1b/field.c:726: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(mem, data, length); data/libid3tag-0.15.1b/file.c:435:12: [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). iofile = fopen(path, (mode == ID3_FILE_MODE_READWRITE) ? "r+b" : "rb"); data/libid3tag-0.15.1b/frame.c:219: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(mem, *ptr, length); data/libid3tag-0.15.1b/frame.c:272: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 xid[4]; data/libid3tag-0.15.1b/frame.c:419: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(mem, data, end - data); data/libid3tag-0.15.1b/id3tag.h:146: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 id[5]; data/libid3tag-0.15.1b/id3tag.h:231: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 value[9]; data/libid3tag-0.15.1b/parse.c:130: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(latin1, *ptr, length); data/libid3tag-0.15.1b/parse.c:194: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(data, *ptr, length); data/libid3tag-0.15.1b/render.c:106: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(*ptr, data, length); data/libid3tag-0.15.1b/render.c:126: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(*ptr, latin1, size); data/libid3tag-0.15.1b/tag.c:388: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 title[31], artist[31], album[31], year[5], comment[31]; data/libid3tag-0.15.1b/tag.c:402: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(title, &data[3], 30); data/libid3tag-0.15.1b/tag.c:403: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(artist, &data[33], 30); data/libid3tag-0.15.1b/tag.c:404: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(album, &data[63], 30); data/libid3tag-0.15.1b/tag.c:405: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(year, &data[93], 4); data/libid3tag-0.15.1b/tag.c:406: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(comment, &data[97], 30); data/libid3tag-0.15.1b/tag.c:454:7: [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(mem, ptr, size); data/libid3tag-0.15.1b/tag.c:759: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(buffer, data, 128); data/libid3tag-0.15.1b/field.c:669: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). if (strlen(language) != 3) data/libid3tag-0.15.1b/tag.c:230: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(id); data/libid3tag-0.15.1b/tag.c:238:13: [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(id); data/libid3tag-0.15.1b/tag.c:325: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). ptr = str + strlen(str); ANALYSIS SUMMARY: Hits = 26 Lines analyzed = 7484 in approximately 0.43 seconds (17544 lines/second) Physical Source Lines of Code (SLOC) = 4746 Hits@level = [0] 11 [1] 4 [2] 21 [3] 0 [4] 1 [5] 0 Hits@level+ = [0+] 37 [1+] 26 [2+] 22 [3+] 1 [4+] 1 [5+] 0 Hits/KSLOC@level+ = [0+] 7.79604 [1+] 5.4783 [2+] 4.63548 [3+] 0.210704 [4+] 0.210704 [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.