]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
test: objectstore: chain_xattr: fix wrong memset usage to fill buf 14277/head
authorWeibing Zhang <atheism.zhang@gmail.com>
Sat, 1 Apr 2017 05:24:42 +0000 (13:24 +0800)
committerWeibing Zhang <atheism.zhang@gmail.com>
Sat, 1 Apr 2017 05:24:42 +0000 (13:24 +0800)
    Current usage:
        memset(buf, sizeof(buf), 0x1F);
    should be:
        memset(buf, 0x1F, sizeof(buf));
    to fill the buffer with '0x1F'.

Signed-off-by: Weibing Zhang <zhangweibing@unitedstack.com>
src/test/objectstore/chain_xattr.cc

index 34654295d770f4e95d509c43ecc2961400963347..9f98ed2934df13e60ac2888d3d319a4ff1ab4e24 100644 (file)
@@ -349,7 +349,7 @@ TEST(chain_xattr, fskip_chain_cleanup_and_ensure_single_attr)
 
   std::size_t existing_xattrs = get_xattrs(fd).size();
   char buf[800];
-  memset(buf, sizeof(buf), 0x1F);
+  memset(buf, 0x1F, sizeof(buf));
   // set chunked without either
   {
     std::size_t r = chain_fsetxattr(fd, name, buf, sizeof(buf));
@@ -394,7 +394,7 @@ TEST(chain_xattr, skip_chain_cleanup_and_ensure_single_attr)
   ::close(fd);
 
   char buf[3000];
-  memset(buf, sizeof(buf), 0x1F);
+  memset(buf, 0x1F, sizeof(buf));
   // set chunked without either
   {
     std::size_t r = chain_setxattr(file, name, buf, sizeof(buf));