Remove warning from -Wmaybe-uninitialized

engines/ep/src/dcp/passive_stream.cc:734:9: warning: ‘ret’ may be used uninitialized [-Wmaybe-uninitialized]
734 |         if (ret != cb::engine_errc::no_memory) {
    |         ^~
engines/ep/src/dcp/passive_stream.cc:664:21: note: ‘ret’ was declared here
664 |     cb::engine_errc ret;
    |                     ^~~

protocol/mcbp/request_test.cc:49:77: warning: ‘blob’ may be used uninitialized [-Wmaybe-uninitialized]
 49 |     auto fe = encodeFrameInfo(request::FrameInfoId(32), {blob, sizeof(blob)});
    |                                                                             ^

Change-Id: I66c3e9052cc269b922daa3b4228db5f6bfbdb87d
Reviewed-on: https://review.couchbase.org/c/kv_engine/+/197127
Tested-by: Build Bot <[email protected]>
Reviewed-by: Jim Walker <[email protected]>

Author: trondn

engines/ep/src/dcp/passive_stream.cc

@@ -661,7 +661,7 @@ cb::engine_errc PassiveStream::processMessage(MutationConsumerMessage* message,
         message->getItem()->setCas();
     }
 
-    cb::engine_errc ret;
+    auto ret = cb::engine_errc::failed;
     DeleteSource deleteSource = DeleteSource::Explicit;
     bool switchComplete = false;
     switch (messageType) {

protocol/mcbp/request_test.cc

@@ -45,7 +45,7 @@ static std::vector<uint8_t> encodeFrameInfo(request::FrameInfoId id,
 }
 
 TEST(Request_ParseFrameExtras, ExtendedIdAndSize) {
-    uint8_t blob[37];
+    uint8_t blob[37] = {};
     auto fe = encodeFrameInfo(request::FrameInfoId(32), {blob, sizeof(blob)});
     std::vector<uint8_t> packet(1024);
     RequestBuilder builder({packet.data(), packet.size()});