From f0c5cc80659586578dc1f7f9a2d201302f137624 Mon Sep 17 00:00:00 2001 From: Radoslaw Zarzynski Date: Wed, 2 Mar 2022 15:14:29 +0000 Subject: [PATCH] test/cls_lua: don't append uninitialized memory to bufferlist. MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit This is intended to fix the following compiler's warnings: ``` [1115/1786] Building CXX object src/test/cls_lua/CMakeFiles/ceph_test_cls_lua.dir/test_cls_lua.cc.o ../src/test/cls_lua/test_cls_lua.cc: In member function ‘virtual void ClsLua_Stat_Test::TestBody()’: ../src/test/cls_lua/test_cls_lua.cc:685:12: warning: ‘buf’ may be used uninitialized [-Wmaybe-uninitialized] 685 | bl.append(buf, sizeof(buf)); | ~~~~~~~~~^~~~~~~~~~~~~~~~~~ In file included from ../src/include/encoding.h:38, from ../src/include/uuid.h:9, from ../src/include/types.h:21, from ../src/test/cls_lua/test_cls_lua.cc:3: ../src/include/buffer.h:1124:10: note: by argument 2 of type ‘const char*’ to ‘void ceph::buffer::v15_2_0::list::append(const char*, unsigned int)’ declared here 1124 | void append(const char *data, unsigned len); | ^~~~~~ ../src/test/cls_lua/test_cls_lua.cc:683:8: note: ‘buf’ declared here 683 | char buf[1024]; | ^~~ ../src/test/cls_lua/test_cls_lua.cc: In member function ‘virtual void ClsLua_WriteFull_Test::TestBody()’: ../src/test/cls_lua/test_cls_lua.cc:893:14: warning: ‘buf’ may be used uninitialized [-Wmaybe-uninitialized] 893 | blin.append(buf, sizeof(buf)); | ~~~~~~~~~~~^~~~~~~~~~~~~~~~~~ In file included from ../src/include/encoding.h:38, from ../src/include/uuid.h:9, from ../src/include/types.h:21, from ../src/test/cls_lua/test_cls_lua.cc:3: ../src/include/buffer.h:1124:10: note: by argument 2 of type ‘const char*’ to ‘void ceph::buffer::v15_2_0::list::append(const char*, unsigned int)’ declared here 1124 | void append(const char *data, unsigned len); | ^~~~~~ ../src/test/cls_lua/test_cls_lua.cc:891:8: note: ‘buf’ declared here 891 | char buf[1024]; ``` Signed-off-by: Radoslaw Zarzynski --- src/test/cls_lua/test_cls_lua.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/test/cls_lua/test_cls_lua.cc b/src/test/cls_lua/test_cls_lua.cc index 516b4c9ae00dd..a7d09aba46406 100644 --- a/src/test/cls_lua/test_cls_lua.cc +++ b/src/test/cls_lua/test_cls_lua.cc @@ -680,7 +680,7 @@ TEST_F(ClsLua, Remove) { TEST_F(ClsLua, Stat) { /* build object and stat */ - char buf[1024]; + char buf[1024] = {}; bufferlist bl; bl.append(buf, sizeof(buf)); ASSERT_EQ(0, ioctx.write_full(oid, bl)); @@ -888,7 +888,7 @@ TEST_F(ClsLua, SetXattr) { TEST_F(ClsLua, WriteFull) { // write some data - char buf[1024]; + char buf[1024] = {}; bufferlist blin; blin.append(buf, sizeof(buf)); ASSERT_EQ(0, ioctx.write(oid, blin, blin.length(), 0)); -- 2.39.5