From 1666d9aa81045d6ee23fade338fcde69ace2a4c1 Mon Sep 17 00:00:00 2001
From: amitkuma <amitkuma@redhat.com>
Date: Wed, 23 Aug 2017 22:44:34 +0530
Subject: [PATCH] test: Checking fd for negative before closing

Fixes the coverity issue:

** 1395317 Argument cannot be negative
161. negative_return_fn: Function open("bufferlist", 0) returns a
negative number.
162. var_assign: Assigning: signed variable fd = open.

Signed-off-by: Amit Kumar amitkuma@redhat.com
---
 src/test/bufferlist.cc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/test/bufferlist.cc b/src/test/bufferlist.cc
index 53d66a036c8..4d454ab7ccd 100644
--- a/src/test/bufferlist.cc
+++ b/src/test/bufferlist.cc
@@ -219,6 +219,7 @@ TEST(Buffer, constructors) {
     snprintf(cmd, sizeof(cmd), "echo ABC > %s", FILENAME);
     EXPECT_EQ(0, ::system(cmd));
     int fd = ::open(FILENAME, O_RDONLY);
+    assert (fd >= 0);
     bufferptr ptr(buffer::create_zero_copy(zc_len, fd, NULL));
     history_alloc_bytes += zc_len;
     history_alloc_num++;
-- 
2.39.5