From 0c70e44630734760fd36e0c770a33fb0e74b42a4 Mon Sep 17 00:00:00 2001
From: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Date: Fri, 10 May 2013 13:07:24 +0200
Subject: [PATCH] libcephfs/test.cc: add assert for result of ceph_getxattr()

Check result of ceph_getxattr() before pass it as parameter to
strncmp(). Make sure it's not negative.

CID 739411 Argument cannot be negative (CWE-687)

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
---
 src/test/libcephfs/test.cc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/test/libcephfs/test.cc b/src/test/libcephfs/test.cc
index 6022fc5e06a..7ecf536fb99 100644
--- a/src/test/libcephfs/test.cc
+++ b/src/test/libcephfs/test.cc
@@ -436,6 +436,7 @@ TEST(LibCephFS, Xattrs) {
 
     char gxattrv[128];
     int alen = ceph_getxattr(cmount, test_xattr_file, p, (void *) gxattrv, 128);
+    ASSERT_LT(alen, 0);
     sprintf(xattrv, "testxattr%c", i);
     ASSERT_TRUE(!strncmp(xattrv, gxattrv, alen));