snbmodules
Unit Tests
No unit tests written.
Errors in Documentation Links
docs/Configuration.md
- [404] https://github.com/DUNE-DAQ/snbmodules/tree/leo-initial-merge/snbconfig Rejected status code: 404 Not Found
docs/TechnicalDocumentation.md
- [404] https://github.com/DUNE-DAQ/daqconf/tree/ljoly/snbmodules-config Rejected status code: 404 Not Found
docs/Transfer.md
- [404] https://github.com/DUNE-DAQ/snbmodules/blob/leo-initial-merge/snbconfig/commands/command_generator.sh Rejected status code: 404 Not Found
docs/Transform.md
- [ERROR] file:///home/runner/work/snbmodules/snbmodules/docs/snbmodules/config/snbmodules/simple-transform-test.data.xml File not found. Check if file exists and path is correct
- [ERROR] file:///home/runner/work/snbmodules/snbmodules/docs/snbmodules/scripts/generate_simple_transform File not found. Check if file exists and path is correct
Clang Formatting
| File | Status |
|---|---|
| include/snbmodules/bookkeeper.hpp | ✅ Already formatted |
| include/snbmodules/common/errors_declaration.hpp | ✅ Already formatted |
| include/snbmodules/common/notification_enum.hpp | ✅ Already formatted |
| include/snbmodules/common/protocols_enum.hpp | ✅ Already formatted |
| include/snbmodules/common/status_enum.hpp | ✅ Already formatted |
| include/snbmodules/group_metadata.hpp | ✅ Already formatted |
| include/snbmodules/interfaces/transfer_interface_RClone.hpp | ✅ Already formatted |
| include/snbmodules/interfaces/transfer_interface_SCP.hpp | ✅ Already formatted |
| include/snbmodules/interfaces/transfer_interface_abstract.hpp | ✅ Already formatted |
| include/snbmodules/interfaces/transfer_interface_bittorrent.hpp | ✅ Already formatted |
| include/snbmodules/iomanager_wrapper.hpp | ✅ Already formatted |
| include/snbmodules/ip_format.hpp | ✅ Already formatted |
| include/snbmodules/metadata_abstract.hpp | ✅ Already formatted |
| include/snbmodules/notification_interface.hpp | ✅ Already formatted |
| include/snbmodules/readout/FileReaderBase.hpp | ❌ Needs formatting |
| include/snbmodules/readout/FileSourceConcept.hpp | ❌ Needs formatting |
| include/snbmodules/readout/FileSourceModel.hpp | ❌ Needs formatting |
| include/snbmodules/readout/SNBDataHandlingModel.hpp | ✅ Already formatted |
| include/snbmodules/readout/SNBRequestHandlerModel.hpp | ✅ Already formatted |
| include/snbmodules/readout/detail/FileReaderBase.hxx | ❌ Needs formatting |
| include/snbmodules/readout/detail/FileSourceModel.hxx | ❌ Needs formatting |
| include/snbmodules/readout/detail/SNBDataHandlingModel.hxx | ❌ Needs formatting |
| include/snbmodules/readout/detail/SNBRequestHandlerModel.hxx | ✅ Already formatted |
| include/snbmodules/transfer_client.hpp | ✅ Already formatted |
| include/snbmodules/transfer_metadata.hpp | ✅ Already formatted |
| include/snbmodules/transfer_session.hpp | ✅ Already formatted |
| plugins/SNBDataHandlerModule.cpp | ✅ Already formatted |
| plugins/SNBDataHandlerModule.hpp | ✅ Already formatted |
| plugins/SNBFileReaderModule.cpp | ❌ Needs formatting |
| plugins/SNBFileReaderModule.hpp | ✅ Already formatted |
| plugins/SNBFileTransfer.cpp | ✅ Already formatted |
| plugins/SNBFileTransfer.hpp | ✅ Already formatted |
| plugins/SNBTransferBookkeeper.cpp | ✅ Already formatted |
| plugins/SNBTransferBookkeeper.hpp | ✅ Already formatted |
| src/bookkeeper/bookkeeper.cpp | ✅ Already formatted |
| src/client/transfer_client.cpp | ✅ Already formatted |
| src/client/transfer_session.cpp | ✅ Already formatted |
| src/common/group_metadata.cpp | ✅ Already formatted |
| src/common/iomanager_wrapper.cpp | ✅ Already formatted |
| src/common/notification_interface.cpp | ✅ Already formatted |
| src/common/transfer_interface_bittorrent.cpp | ✅ Already formatted |
| src/common/transfer_metadata.cpp | ✅ Already formatted |
| test/apps/snb_bittorrent_full_test.cxx | ✅ Already formatted |
| test/apps/snb_bookkeeper_app.cxx | ✅ Already formatted |
| test/apps/snb_client_available_test.cxx | ✅ Already formatted |
| test/apps/snb_client_test.cxx | ✅ Already formatted |
| test/apps/snb_group_metadata_save_load.cxx | ✅ Already formatted |
| test/apps/snb_ip_format_test.cxx | ✅ Already formatted |
| test/apps/snb_notification_test.cxx | ✅ Already formatted |
| test/apps/snb_rclone_full_test.cxx | ✅ Already formatted |
| test/apps/snb_session_test.cxx | ✅ Already formatted |
| test/apps/snb_transfer_client_app.cxx | ✅ Already formatted |
| test/apps/snb_transfer_metadata_save_load.cxx | ✅ Already formatted |
| unittest/Placeholder_test.cxx | ✅ Already formatted |
include/snbmodules/bookkeeper.hpp
Errors
- Line 116: Add #include
for shared_ptr<>
include/snbmodules/group_metadata.hpp
Errors
- Line 88: Use of Run Time Type Information not allowed unless this code is meant to test other code
- Line 98: Use of Run Time Type Information not allowed unless this code is meant to test other code
- Line 158: Add #include
for shared_ptr<>
include/snbmodules/interfaces/transfer_interface_RClone.hpp
Errors
- Line 524: Missing author name in TODO comment; it should appear on the same line as the TODO, as Firstname Lastname
- Line 524: Missing email address in TODO comment; it should appear on the same line as the TODO
include/snbmodules/iomanager_wrapper.hpp
Errors
- Line 65: Add #include
for vector<>
include/snbmodules/notification_interface.hpp
Errors
- Line 138: A catch-all-exceptions construct was found in a file which doesn't contain int main(); this can only be used to wrap main()
- Line 150: A catch-all-exceptions construct was found in a file which doesn't contain int main(); this can only be used to wrap main()
include/snbmodules/readout/FileReaderBase.hpp
Errors
- Line 12: #ifndef header guard has wrong style, please use: SNBMODULES_INCLUDE_SNBMODULES_READOUT_FILEREADERBASE_HPP_
- Line 93: #endif line should be "#endif // SNBMODULES_INCLUDE_SNBMODULES_READOUT_FILEREADERBASE_HPP_"
include/snbmodules/readout/FileSourceConcept.hpp
Errors
- Line 9: #ifndef header guard has wrong style, please use: SNBMODULES_INCLUDE_SNBMODULES_READOUT_FILESOURCECONCEPT_HPP_
- Line 49: #endif line should be "#endif // SNBMODULES_INCLUDE_SNBMODULES_READOUT_FILESOURCECONCEPT_HPP_"
include/snbmodules/readout/FileSourceModel.hpp
Errors
- Line 8: #ifndef header guard has wrong style, please use: SNBMODULES_INCLUDE_SNBMODULES_READOUT_FILESOURCEMODEL_HPP_
- Line 125: #endif line should be "#endif // SNBMODULES_INCLUDE_SNBMODULES_READOUT_FILESOURCEMODEL_HPP_"
- Line 28: Include the directory when naming .hh files
- Line 78: "virtual" is redundant since function is already declared as "override"
- Line 93: An unsigned integer appears to be used here.
include/snbmodules/readout/SNBDataHandlingModel.hpp
Errors
- Line 34: "appmodel/DataHandlerModule.hpp" already included at sourcecode/snbmodules/include/snbmodules/readout/SNBDataHandlingModel.hpp:13
- Line 156: Use of reinterpret_cast can be dangerous. If there's no way to avoid its use add " // NOLINT" to the end of this line so this script will ignore it next time
- Line 159: "virtual" is redundant since function is already declared as "override"
- Line 171: An unsigned integer appears to be used here.
- Line 172: An unsigned integer appears to be used here.
- Line 173: An unsigned integer appears to be used here.
include/snbmodules/readout/SNBRequestHandlerModel.hpp
Errors
- Line 8: #ifndef header guard has wrong style, please use: SNBMODULES_INCLUDE_SNBMODULES_READOUT_SNBREQUESTHANDLERMODEL_HPP_
- Line 316: #endif line should be "#endif // SNBMODULES_INCLUDE_SNBMODULES_READOUT_SNBREQUESTHANDLERMODEL_HPP_"
- Line 73: An unsigned integer appears to be used here.
- Line 145: "virtual" is redundant since function is already declared as "override"
- Line 215: An unsigned integer appears to be used here.
- Line 216: An unsigned integer appears to be used here.
- Line 223: "virtual" is redundant since function is already declared as "override"
- Line 264: An unsigned integer appears to be used here.
- Line 267: An unsigned integer appears to be used here.
- Line 273: An unsigned integer appears to be used here.
- Line 264: Add #include
for set<>
include/snbmodules/transfer_client.hpp
Errors
- Line 146: Add #include
for vector<> - Line 178: Add #include
for shared_ptr<>
include/snbmodules/transfer_metadata.hpp
Errors
- Line 49: Use of Run Time Type Information not allowed unless this code is meant to test other code
- Line 62: Use of Run Time Type Information not allowed unless this code is meant to test other code
- Line 75: An unsigned integer appears to be used here.
- Line 80: An unsigned integer appears to be used here.
- Line 174: An unsigned integer appears to be used here.
- Line 180: An unsigned integer appears to be used here.
- Line 260: An unsigned integer appears to be used here.
- Line 261: An unsigned integer appears to be used here.
- Line 317: An unsigned integer appears to be used here.
- Line 320: An unsigned integer appears to be used here.
include/snbmodules/transfer_session.hpp
Errors
- Line 105: Add #include
plugins/SNBDataHandlerModule.cpp
Errors
- Line 187: Small and focused functions are preferred: SNBDataHandlerModule::create_readout() has 86 non-comment lines (error triggered by exceeding 80 lines) (disable this by putting "// NOLINT" after the function's closing brace.)
Warnings
- Line 58: using declarations in the global namespace in headers are prohibited
- Line 59: using declarations in the global namespace in headers are prohibited
- Line 60: using declarations in the global namespace in headers are prohibited
- Line 61: using declarations in the global namespace in headers are prohibited
- Line 63: nested namespaces can be concatenated
- Line 102: 'init' overrides a member function but is not marked 'override'
- Line 105: 'conf' overrides a member function but is not marked 'override'
- Line 108: 'scrap' overrides a member function but is not marked 'override'
- Line 116: 'start' overrides a member function but is not marked 'override'
- Line 119: 'stop' overrides a member function but is not marked 'override'
- Line 141: 'run_timesync' overrides a member function but is not marked 'override'
- Line 144: 'run_consume' overrides a member function but is not marked 'override'
- Line 156: do not use reinterpret_cast
- Line 54: using declarations in the global namespace in headers are prohibited
- Line 55: using declarations in the global namespace in headers are prohibited
- Line 56: using declarations in the global namespace in headers are prohibited
- Line 58: nested namespaces can be concatenated
- Line 127: 'conf' overrides a member function but is not marked 'override'
- Line 133: 'start' overrides a member function but is not marked 'override'
- Line 136: 'stop' overrides a member function but is not marked 'override'
- Line 148: default arguments on virtual or override methods are prohibited
- Line 193: do not use pointer arithmetic
- Line 209: 'cleanup' overrides a member function but is not marked 'override'
- Line 8: nested namespaces can be concatenated
- Line 372: std::move of the variable 'timesyncmsg_copy' of the trivially-copyable type 'dfmessages::TimeSync' has no effect
- Line 3: nested namespaces can be concatenated
- Line 18: narrowing conversion from 'uint32_t' (aka 'unsigned int') to signed type 'int' is implementation-defined
- Line 22: narrowing conversion from 'uint32_t' (aka 'unsigned int') to signed type 'int' is implementation-defined
- Line 92: loop variable is copied but only used as const reference; consider making it a const reference
- Line 136: redundant return statement at the end of a function with a void return type
- Line 154: default arguments on virtual or override methods are prohibited
- Line 198: narrowing conversion from 'rep' (aka 'long') to signed type '__int_type' (aka 'int') is implementation-defined
- Line 200: narrowing conversion from 'rep' (aka 'long') to signed type '__int_type' (aka 'int') is implementation-defined
- Line 202: narrowing conversion from 'rep' (aka 'long') to signed type '__int_type' (aka 'int') is implementation-defined
- Line 319: narrowing conversion from 'unsigned int' to signed type '__int_type' (aka 'int') is implementation-defined
- Line 407: C-style casts are discouraged; use static_cast
- Line 411: variable 'elements_handled' set but not used
- Line 17: nested namespaces can be concatenated
plugins/SNBDataHandlerModule.hpp
Errors
- Line 48: Namespace should be terminated with '// namespace snbmodules '
- Line 41: Add #include
for shared_ptr<>
plugins/SNBFileReaderModule.cpp
Warnings
- Line 43: nested namespaces can be concatenated
- Line 22: nested namespaces can be concatenated
- Line 37: using declarations in the global namespace in headers are prohibited
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 40: nested namespaces can be concatenated
- Line 58: 'set_sender' overrides a member function but is not marked 'override'
- Line 60: 'conf' overrides a member function but is not marked 'override'
- Line 61: 'scrap' overrides a member function but is not marked 'override'
- Line 71: 'start' overrides a member function but is not marked 'override'
- Line 72: 'stop' overrides a member function but is not marked 'override'
- Line 6: nested namespaces can be concatenated
- Line 9: function 'FileReaderBase' defined in a header file; function definitions in header files can lead to ODR violations
- Line 17: function 'init' defined in a header file; function definitions in header files can lead to ODR violations
- Line 49: function 'do_conf' defined in a header file; function definitions in header files can lead to ODR violations
- Line 95: function 'do_scrap' defined in a header file; function definitions in header files can lead to ODR violations
- Line 110: function 'do_start' defined in a header file; function definitions in header files can lead to ODR violations
- Line 126: function 'do_stop' defined in a header file; function definitions in header files can lead to ODR violations
- Line 7: using declarations in the global namespace in headers are prohibited
- Line 8: using declarations in the global namespace in headers are prohibited
- Line 9: using declarations in the global namespace in headers are prohibited
- Line 10: using declarations in the global namespace in headers are prohibited
- Line 11: using declarations in the global namespace in headers are prohibited
- Line 13: nested namespaces can be concatenated
- Line 92: the variable 'raw_dt' is copy-constructed from a const reference but is only used as const reference; consider making it a const reference
- Line 23: nested namespaces can be concatenated
plugins/SNBFileReaderModule.hpp
Errors
- Line 8: #ifndef header guard has wrong style, please use: SNBMODULES_PLUGINS_SNBFILEREADERMODULE_HPP_
- Line 53: #endif line should be "#endif // SNBMODULES_PLUGINS_SNBFILEREADERMODULE_HPP_"
- Line 46: Add #include
for shared_ptr<>
plugins/SNBFileTransfer.cpp
Warnings
- Line 91: Call to function 'strcpy' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcpy'. CWE-119
- Line 95: unnecessary temporary object created while calling emplace_back
plugins/SNBTransferBookkeeper.cpp
Warnings
- Line 91: Call to function 'strcpy' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcpy'. CWE-119
- Line 95: unnecessary temporary object created while calling emplace_back
src/bookkeeper/bookkeeper.cpp
Errors
- Line 146: An unsigned integer appears to be used here.
- Line 217: Small and focused functions are preferred: Bookkeeper::input_action() has 112 non-comment lines (error triggered by exceeding 80 lines) (disable this by putting "// NOLINT" after the function's closing brace.)
- Line 421: Use of Run Time Type Information not allowed unless this code is meant to test other code
- Line 422: Use of Run Time Type Information not allowed unless this code is meant to test other code
- Line 425: Use of Run Time Type Information not allowed unless this code is meant to test other code
- Line 430: Use of Run Time Type Information not allowed unless this code is meant to test other code
- Line 518: Use of Run Time Type Information not allowed unless this code is meant to test other code
- Line 519: Use of Run Time Type Information not allowed unless this code is meant to test other code
- Line 523: Small and focused functions are preferred: Bookkeeper::display_information() has 86 non-comment lines (error triggered by exceeding 80 lines) (disable this by putting "// NOLINT" after the function's closing brace.)
- Line 533: Add #include
for shared_ptr<>
Warnings
- Line 77: Call to virtual method 'GroupMetadata::from_string' during construction bypasses virtual dispatch
- Line 91: Call to function 'strcpy' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcpy'. CWE-119
- Line 95: unnecessary temporary object created while calling emplace_back
- Line 123: Call to virtual method 'TransferMetadata::from_string' during construction bypasses virtual dispatch
- Line 157: loop variable is copied but only used as const reference; consider making it a const reference
- Line 400: passing result of std::move() as a const reference argument; no move will actually happen
src/client/transfer_client.cpp
Errors
- Line 251: Missing date in TODO comment; it should appear on same line as the TODO, preferably in a form like "Apr-14-2020"
- Line 251: Missing author name in TODO comment; it should appear on the same line as the TODO, as Firstname Lastname
- Line 251: Missing email address in TODO comment; it should appear on the same line as the TODO
- Line 451: Small and focused functions are preferred: TransferClient::action_on_receive_notification() has 128 non-comment lines (error triggered by exceeding 80 lines) (disable this by putting "// NOLINT" after the function's closing brace.)
Warnings
- Line 77: Call to virtual method 'GroupMetadata::from_string' during construction bypasses virtual dispatch
- Line 91: Call to function 'strcpy' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcpy'. CWE-119
- Line 95: unnecessary temporary object created while calling emplace_back
- Line 121: Call to virtual method 'TransferMetadata::load_metadata_from_meta_file' during construction bypasses virtual dispatch
- Line 123: Call to virtual method 'TransferMetadata::from_string' during construction bypasses virtual dispatch
- Line 153: passing result of std::move() as a const reference argument; no move will actually happen
- Line 171: loop variable is copied but only used as const reference; consider making it a const reference
- Line 265: passing result of std::move() as a const reference argument; no move will actually happen
src/client/transfer_session.cpp
Errors
- Line 55: Use of Run Time Type Information not allowed unless this code is meant to test other code
- Line 124: Missing date in TODO comment; it should appear on same line as the TODO, preferably in a form like "Apr-14-2020"
- Line 124: Missing author name in TODO comment; it should appear on the same line as the TODO, as Firstname Lastname
- Line 124: Missing email address in TODO comment; it should appear on the same line as the TODO
Warnings
- Line 91: Call to function 'strcpy' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcpy'. CWE-119
- Line 95: unnecessary temporary object created while calling emplace_back
- Line 32: passing result of std::move() as a const reference argument; no move will actually happen
- Line 57: loop variable is copied but only used as const reference; consider making it a const reference
- Line 153: loop variable is copied but only used as const reference; consider making it a const reference
- Line 397: loop variable is copied but only used as const reference; consider making it a const reference
- Line 409: loop variable is copied but only used as const reference; consider making it a const reference
- Line 425: loop variable is copied but only used as const reference; consider making it a const reference
- Line 443: loop variable is copied but only used as const reference; consider making it a const reference
- Line 459: loop variable is copied but only used as const reference; consider making it a const reference
src/common/group_metadata.cpp
Errors
- Line 36: Add #include
for shared_ptr<>
Warnings
- Line 91: Call to function 'strcpy' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcpy'. CWE-119
- Line 95: unnecessary temporary object created while calling emplace_back
- Line 26: loop variable is copied but only used as const reference; consider making it a const reference
src/common/iomanager_wrapper.cpp
Warnings
- Line 91: Call to function 'strcpy' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcpy'. CWE-119
- Line 95: unnecessary temporary object created while calling emplace_back
src/common/notification_interface.cpp
Warnings
- Line 91: Call to function 'strcpy' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcpy'. CWE-119
- Line 95: unnecessary temporary object created while calling emplace_back
src/common/transfer_interface_bittorrent.cpp
Errors
- Line 212: An unsigned integer appears to be used here.
- Line 389: Missing date in TODO comment; it should appear on same line as the TODO, preferably in a form like "Apr-14-2020"
- Line 389: Missing author name in TODO comment; it should appear on the same line as the TODO, as Firstname Lastname
- Line 389: Missing email address in TODO comment; it should appear on the same line as the TODO
- Line 390: "cerr" should not be used for output in DUNE DAQ software.
- Line 391: Small and focused functions are preferred: TransferInterfaceBittorrent::do_work() has 155 non-comment lines (error triggered by exceeding 80 lines) (disable this by putting "// NOLINT" after the function's closing brace.)
- Line 697: Small and focused functions are preferred: TransferInterfaceBittorrent::set_settings() has 121 non-comment lines (error triggered by exceeding 80 lines) (disable this by putting "// NOLINT" after the function's closing brace.)
- Line 798: "cerr" should not be used for output in DUNE DAQ software.
- Line 817: "cerr" should not be used for output in DUNE DAQ software.
- Line 819: "cerr" should not be used for output in DUNE DAQ software.
- Line 846: "cerr" should not be used for output in DUNE DAQ software.
- Line 926: Missing date in TODO comment; it should appear on same line as the TODO, preferably in a form like "Apr-14-2020"
- Line 926: Missing author name in TODO comment; it should appear on the same line as the TODO, as Firstname Lastname
- Line 926: Missing email address in TODO comment; it should appear on the same line as the TODO
- Line 997: Missing date in TODO comment; it should appear on same line as the TODO, preferably in a form like "Apr-14-2020"
- Line 997: Missing author name in TODO comment; it should appear on the same line as the TODO, as Firstname Lastname
- Line 997: Missing email address in TODO comment; it should appear on the same line as the TODO
Warnings
- Line 91: Call to function 'strcpy' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcpy'. CWE-119
- Line 95: unnecessary temporary object created while calling emplace_back
- Line 81: Value stored to 'm_done' is never read
src/common/transfer_metadata.cpp
Warnings
- Line 91: Call to function 'strcpy' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcpy'. CWE-119
- Line 95: unnecessary temporary object created while calling emplace_back
test/apps/snb_bittorrent_full_test.cxx
Warnings
- Line 95: unnecessary temporary object created while calling emplace_back
test/apps/snb_bookkeeper_app.cxx
Warnings
- Line 95: unnecessary temporary object created while calling emplace_back
test/apps/snb_client_available_test.cxx
Errors
- Line 41: Add #include
for make_shared<>
Warnings
- Line 91: Call to function 'strcpy' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcpy'. CWE-119
- Line 95: unnecessary temporary object created while calling emplace_back
test/apps/snb_client_test.cxx
Warnings
- Line 91: Call to function 'strcpy' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcpy'. CWE-119
- Line 95: unnecessary temporary object created while calling emplace_back
test/apps/snb_group_metadata_save_load.cxx
Warnings
- Line 75: Call to virtual method 'GroupMetadata::load_metadata_from_meta_file' during construction bypasses virtual dispatch
- Line 91: Call to function 'strcpy' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcpy'. CWE-119
- Line 95: unnecessary temporary object created while calling emplace_back
test/apps/snb_ip_format_test.cxx
Warnings
- Line 91: Call to function 'strcpy' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcpy'. CWE-119
- Line 95: unnecessary temporary object created while calling emplace_back
test/apps/snb_notification_test.cxx
Errors
- Line 55: If an else has a brace on one side, it should have it on both
Warnings
- Line 95: unnecessary temporary object created while calling emplace_back
test/apps/snb_rclone_full_test.cxx
Warnings
- Line 95: unnecessary temporary object created while calling emplace_back
test/apps/snb_session_test.cxx
Errors
- Line 51: Add #include
for make_shared<>
Warnings
- Line 91: Call to function 'strcpy' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcpy'. CWE-119
- Line 95: unnecessary temporary object created while calling emplace_back
test/apps/snb_transfer_client_app.cxx
Warnings
- Line 95: unnecessary temporary object created while calling emplace_back
test/apps/snb_transfer_metadata_save_load.cxx
Warnings
- Line 91: Call to function 'strcpy' is insecure as it does not provide bounding of the memory buffer. Replace unbounded copy functions with analogous functions that support length arguments such as 'strlcpy'. CWE-119
- Line 95: unnecessary temporary object created while calling emplace_back
- Line 121: Call to virtual method 'TransferMetadata::load_metadata_from_meta_file' during construction bypasses virtual dispatch