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/orthanc-dicomweb-1.3+dfsg/Plugin/Configuration.cpp Examining data/orthanc-dicomweb-1.3+dfsg/Plugin/Configuration.h Examining data/orthanc-dicomweb-1.3+dfsg/Plugin/DicomWebClient.cpp Examining data/orthanc-dicomweb-1.3+dfsg/Plugin/DicomWebClient.h Examining data/orthanc-dicomweb-1.3+dfsg/Plugin/DicomWebFormatter.cpp Examining data/orthanc-dicomweb-1.3+dfsg/Plugin/DicomWebFormatter.h Examining data/orthanc-dicomweb-1.3+dfsg/Plugin/DicomWebServers.cpp Examining data/orthanc-dicomweb-1.3+dfsg/Plugin/DicomWebServers.h Examining data/orthanc-dicomweb-1.3+dfsg/Plugin/Plugin.cpp Examining data/orthanc-dicomweb-1.3+dfsg/Plugin/QidoRs.cpp Examining data/orthanc-dicomweb-1.3+dfsg/Plugin/QidoRs.h Examining data/orthanc-dicomweb-1.3+dfsg/Plugin/StowRs.cpp Examining data/orthanc-dicomweb-1.3+dfsg/Plugin/StowRs.h Examining data/orthanc-dicomweb-1.3+dfsg/Plugin/WadoRs.cpp Examining data/orthanc-dicomweb-1.3+dfsg/Plugin/WadoRs.h Examining data/orthanc-dicomweb-1.3+dfsg/Plugin/WadoRsRetrieveFrames.cpp Examining data/orthanc-dicomweb-1.3+dfsg/Plugin/WadoRsRetrieveRendered.cpp Examining data/orthanc-dicomweb-1.3+dfsg/Plugin/WadoUri.cpp Examining data/orthanc-dicomweb-1.3+dfsg/Plugin/WadoUri.h Examining data/orthanc-dicomweb-1.3+dfsg/Resources/Graveyard/MinorityReport.cpp Examining data/orthanc-dicomweb-1.3+dfsg/Resources/Orthanc/Plugins/OrthancPluginCppWrapper.cpp Examining data/orthanc-dicomweb-1.3+dfsg/Resources/Orthanc/Plugins/OrthancPluginCppWrapper.h Examining data/orthanc-dicomweb-1.3+dfsg/Resources/Orthanc/Plugins/OrthancPluginException.h Examining data/orthanc-dicomweb-1.3+dfsg/Resources/Orthanc/Sdk-1.7.0/orthanc/OrthancCPlugin.h Examining data/orthanc-dicomweb-1.3+dfsg/UnitTestsSources/UnitTestsMain.cpp FINAL RESULTS: data/orthanc-dicomweb-1.3+dfsg/Plugin/Plugin.cpp:469:7: [4] (buffer) sprintf: Does not check for buffer overflows (CWE-120). Use sprintf_s, snprintf, or vsnprintf. sprintf(info, "Your version of Orthanc (%s) must be above %d.%d.%d to run this plugin", data/orthanc-dicomweb-1.3+dfsg/Resources/Orthanc/Plugins/OrthancPluginCppWrapper.cpp:1431:7: [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. sscanf data/orthanc-dicomweb-1.3+dfsg/Resources/Orthanc/Sdk-1.7.0/orthanc/OrthancCPlugin.h:1782:7: [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. sscanf data/orthanc-dicomweb-1.3+dfsg/Plugin/DicomWebFormatter.cpp:36: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 buf[16]; data/orthanc-dicomweb-1.3+dfsg/Plugin/DicomWebFormatter.cpp:37:5: [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(buf, "%04x%04x", group, element); data/orthanc-dicomweb-1.3+dfsg/Plugin/Plugin.cpp:468:7: [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 info[1024]; data/orthanc-dicomweb-1.3+dfsg/Resources/Orthanc/Plugins/OrthancPluginCppWrapper.cpp:137: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(buffer_.data, buffer, size); data/orthanc-dicomweb-1.3+dfsg/Resources/Orthanc/Plugins/OrthancPluginCppWrapper.cpp:2637:13: [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(&target[pos], (*it)->c_str(), s); data/orthanc-dicomweb-1.3+dfsg/Resources/Orthanc/Sdk-1.7.0/orthanc/OrthancCPlugin.h:3028:33: [1] (buffer) read: Check buffer boundaries if used in a loop including recursive loops (CWE-120, CWE-20). OrthancPluginStorageRead read; data/orthanc-dicomweb-1.3+dfsg/Resources/Orthanc/Sdk-1.7.0/orthanc/OrthancCPlugin.h:3050:33: [1] (buffer) read: Check buffer boundaries if used in a loop including recursive loops (CWE-120, CWE-20). OrthancPluginStorageRead read, data/orthanc-dicomweb-1.3+dfsg/Resources/Orthanc/Sdk-1.7.0/orthanc/OrthancCPlugin.h:3055:19: [1] (buffer) read: Check buffer boundaries if used in a loop including recursive loops (CWE-120, CWE-20). params.read = read; ANALYSIS SUMMARY: Hits = 11 Lines analyzed = 21063 in approximately 0.48 seconds (44298 lines/second) Physical Source Lines of Code (SLOC) = 13657 Hits@level = [0] 3 [1] 3 [2] 5 [3] 0 [4] 3 [5] 0 Hits@level+ = [0+] 14 [1+] 11 [2+] 8 [3+] 3 [4+] 3 [5+] 0 Hits/KSLOC@level+ = [0+] 1.02512 [1+] 0.805448 [2+] 0.58578 [3+] 0.219668 [4+] 0.219668 [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.