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/zita-lrx-0.1.2/source/jclient.h
Examining data/zita-lrx-0.1.2/source/jclient.cc
Examining data/zita-lrx-0.1.2/source/sstring.cc
Examining data/zita-lrx-0.1.2/source/global.h
Examining data/zita-lrx-0.1.2/source/delayproc.cc
Examining data/zita-lrx-0.1.2/source/delayproc.h
Examining data/zita-lrx-0.1.2/source/lr4filt.cc
Examining data/zita-lrx-0.1.2/source/lrxconf.h
Examining data/zita-lrx-0.1.2/source/sstring.h
Examining data/zita-lrx-0.1.2/source/lr4filt.h
Examining data/zita-lrx-0.1.2/source/lrxproc.h
Examining data/zita-lrx-0.1.2/source/lrxconf.cc
Examining data/zita-lrx-0.1.2/source/zita-lrx.cc
Examining data/zita-lrx-0.1.2/source/lrxproc.cc
FINAL RESULTS:
data/zita-lrx-0.1.2/source/lrxconf.cc:123: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 (_filename, file);
data/zita-lrx-0.1.2/source/lrxconf.cc:170:15: [4] (buffer) sscanf:
The scanf() family's %s operation, without a limit specification, permits
buffer overflows (CWE-120, CWE-20). Specify a limit to %s, or use a
different input function.
else if (sscanf (q, "%d%s%n", &i1, s1, &n) != 2) stat = ARGS;
data/zita-lrx-0.1.2/source/lrxconf.cc:184:8: [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).
else strcpy (_chann [i1 - 1]._label, s1);
data/zita-lrx-0.1.2/source/lrxconf.cc:255:15: [4] (buffer) sscanf:
The scanf() family's %s operation, without a limit specification, permits
buffer overflows (CWE-120, CWE-20). Specify a limit to %s, or use a
different input function.
else if (sscanf (q, "%d%s%n", &i1, s1, &n) != 2) stat = ARGS;
data/zita-lrx-0.1.2/source/lrxconf.cc:269:8: [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).
else strcpy (_fband [i1 - 1]._label, s1);
data/zita-lrx-0.1.2/source/zita-lrx.cc:55:17: [3] (buffer) getopt:
Some older implementations do not protect against internal buffer overflows
(CWE-120, CWE-20). Check implementation on installation, or limit the size
of all string inputs.
while ((k = getopt (ac, av, (char *) clopt)) != -1)
data/zita-lrx-0.1.2/source/delayproc.h:80:17: [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 (p, q, n * sizeof (float));
data/zita-lrx-0.1.2/source/delayproc.h:92:17: [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 (p, q, n * sizeof (float));
data/zita-lrx-0.1.2/source/jclient.cc:141: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 s [64];
data/zita-lrx-0.1.2/source/jclient.cc:155: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 s [64];
data/zita-lrx-0.1.2/source/jclient.cc:182: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 s [64];
data/zita-lrx-0.1.2/source/lrxconf.cc:78:2: [2] (buffer) sprintf:
Does not check for buffer overflows (CWE-120). Use sprintf_s, snprintf, or
vsnprintf. Risk is low because the source has a constant maximum length.
sprintf (_chann [j]._label, "%d", j + 1);
data/zita-lrx-0.1.2/source/lrxconf.cc:82:2: [2] (buffer) sprintf:
Does not check for buffer overflows (CWE-120). Use sprintf_s, snprintf, or
vsnprintf. Risk is low because the source has a constant maximum length.
sprintf (_fband [j]._label, "%d", j + 1);
data/zita-lrx-0.1.2/source/lrxconf.cc:117: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 buff [1024];
data/zita-lrx-0.1.2/source/lrxconf.cc:121: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 s1 [64];
data/zita-lrx-0.1.2/source/lrxconf.cc:125:13: [2] (buffer) strcat:
Does not check for buffer overflows when concatenating to destination
[MS-banned] (CWE-120). Consider using strcat_s, strncat, strlcat, or
snprintf (warning: strncat is easily misused). Risk is low because the
source is a constant string.
if (!p) strcat (_filename, ".zlrx");
data/zita-lrx-0.1.2/source/lrxconf.cc:126:16: [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).
if (! (F = fopen (_filename, "r")))
data/zita-lrx-0.1.2/source/lrxconf.h:35: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 _label [16];
data/zita-lrx-0.1.2/source/lrxconf.h:38: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 *_dest [MAXBAND];
data/zita-lrx-0.1.2/source/lrxconf.h:46: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 _label [16];
data/zita-lrx-0.1.2/source/lrxconf.h:84: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 _filename [1024];
data/zita-lrx-0.1.2/source/lrxproc.cc:177:3: [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 (q [0], D->rd_ptr (), n * sizeof (float));
data/zita-lrx-0.1.2/source/lrxproc.cc:232:2: [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 (outa, A->rd_ptr (), n * sizeof (float));
data/zita-lrx-0.1.2/source/lrxproc.cc:238:2: [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 (outb, B->rd_ptr (), n * sizeof (float));
data/zita-lrx-0.1.2/source/lrxconf.cc:179:12: [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).
else if (strlen (s1) > 15)
data/zita-lrx-0.1.2/source/lrxconf.cc:264:12: [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).
else if (strlen (s1) > 15)
ANALYSIS SUMMARY:
Hits = 26
Lines analyzed = 1782 in approximately 0.05 seconds (36685 lines/second)
Physical Source Lines of Code (SLOC) = 1182
Hits@level = [0] 58 [1] 2 [2] 18 [3] 1 [4] 5 [5] 0
Hits@level+ = [0+] 84 [1+] 26 [2+] 24 [3+] 6 [4+] 5 [5+] 0
Hits/KSLOC@level+ = [0+] 71.066 [1+] 21.9966 [2+] 20.3046 [3+] 5.07614 [4+] 4.23012 [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.