summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorUmang Jain <email@uajain.com>2020-05-15 12:42:52 +0000
committerKieran Bingham <kieran.bingham@ideasonboard.com>2020-08-13 11:05:58 +0100
commit02018a7d0222cefe8e6587deebe607917e20e2a1 (patch)
tree720b9fba5e69f69fb044e6be4fc502d7aaae7854
parent6b88163343dc7aec6a6e5d3e964bba7b14f83dd6 (diff)
test: ipc: unixsocket: Close open fds on error paths
Reported-by: Coverity CID=279099 Signed-off-by: Umang Jain <email@uajain.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com> Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
-rw-r--r--test/ipc/unixsocket.cpp2
1 files changed, 2 insertions, 0 deletions
diff --git a/test/ipc/unixsocket.cpp b/test/ipc/unixsocket.cpp
index 4487d27b..9cd07ce8 100644
--- a/test/ipc/unixsocket.cpp
+++ b/test/ipc/unixsocket.cpp
@@ -146,6 +146,7 @@ private:
if (num < 0) {
cerr << "Read failed" << endl;
+ close(outfd);
stop(-EIO);
return;
} else if (!num)
@@ -153,6 +154,7 @@ private:
if (write(outfd, buf, num) < 0) {
cerr << "Write failed" << endl;
+ close(outfd);
stop(-EIO);
return;
}