Fix issues reported by SVACE/Coverity

Change-Id: Ide8e99b306cc623f5c1a46ff5a0373f7484870c1

diff --git a/srcs/ctap_message_processor.h b/srcs/ctap_message_processor.h
index 812f807f7b24ede2b62bd0b59a290546b5719110..dcf0d849e7511ce69eed7b1c4ba277e7ccccd7a6 100644 (file)
--- a/srcs/ctap_message_processor.h
+++ b/srcs/ctap_message_processor.h
@@ -47,7 +47,7 @@ public:
     // May be called from the other thread.
     virtual void Cancel() = 0;
 
-    ~ICtapMessageProcessor() = default;
+    virtual ~ICtapMessageProcessor() = default;
 };
 
 class CtapMessageProcessor : public ICtapMessageProcessor {

diff --git a/srcs/message.h b/srcs/message.h
index 3e3ac7c7af36b1f471da5942669d2eb3468adeaf..3fbba44a9d742947c3edbbb3165d357bc8e21821 100644 (file)
--- a/srcs/message.h
+++ b/srcs/message.h
@@ -168,7 +168,7 @@ public:
         };
 
         BufferView m_rpIdHash;
-        uint8_t m_flags;
+        uint8_t m_flags = 0;
         std::optional<AttestationData> m_attestationData;
     } m_authData;
 

diff --git a/tests/cbor_tests.cpp b/tests/cbor_tests.cpp
index 77ad9efd10961eae3e4e70736e2a1375d71203ce..edb3b1a9e24ad4128dbd051ad0159af1681b7766 100644 (file)
--- a/tests/cbor_tests.cpp
+++ b/tests/cbor_tests.cpp
@@ -266,7 +266,7 @@ TEST(CborEncoding, EmptyBuffers)
     ASSERT_NO_THROW(map.AppendByteStringAt(0x02, Buffer{}));
     wauthn_const_buffer_s buffer;
     buffer.data = nullptr;
-    buffer.data = 0;
+    buffer.size = 0;
     ASSERT_NO_THROW(map.AppendByteStringAt(0x03, buffer));
     buffer.size = 10;
     ASSERT_THROW(map.AppendByteStringAt(0x04, buffer), Exception::InvalidParam);

diff --git a/tests/man_tests.cpp b/tests/man_tests.cpp
index 9599afdca1908d1ecab839351b9b352bc2a82b48..d6dd6ae990b4bfb9c0d6147066316f9a18e70138 100644 (file)
--- a/tests/man_tests.cpp
+++ b/tests/man_tests.cpp
@@ -138,7 +138,7 @@ void DisplayQRCallback(const char *qr_contents, void *data)
     if (testContents->path.empty() || !qr_contents) {
         std::cout << "qrcode_callback failed" << std::endl;
         std::cout << "QRcode path: " << testContents->path << std::endl;
-        std::cout << "qr_contents: " << qr_contents << std::endl;
+        std::cout << "qr_contents: " << (qr_contents ? qr_contents : "null") << std::endl;
         testContents->status = false;
         return;
     }
@@ -347,7 +347,7 @@ int main(int argc, char *argv[])
         ret = 1;
     }
 
-    if (system("killall /usr/apps/org.tizen.image-viewer_common/bin/image-viewer")) {
+    if (system("/usr/bin/killall /usr/apps/org.tizen.image-viewer_common/bin/image-viewer")) {
         std::cout << "Cannot close org.tizen.image-viewer\n";
         ret = 1;
     }

diff --git a/tests/tunnel/auto_tests.cpp b/tests/tunnel/auto_tests.cpp
index a001e19335476c4660b9bff7705a27d37e990b65..0d9889f31ff133dd6bd7121933388341b61ff6b4 100644 (file)
--- a/tests/tunnel/auto_tests.cpp
+++ b/tests/tunnel/auto_tests.cpp
@@ -563,7 +563,7 @@ TEST(TunnelMockedTests, InjectedEvents)
         }
 
     private:
-        const std::vector<EventInfo> *m_injectionsMap;
+        const std::vector<EventInfo> *m_injectionsMap = nullptr;
         size_t m_injectionsMapNextEventIdx = 0;
         int m_eventNo = 0;
     };