diff options
author | Umang Jain <email@uajain.com> | 2020-05-15 12:42:52 +0000 |
---|---|---|
committer | Kieran Bingham <kieran.bingham@ideasonboard.com> | 2020-08-13 11:05:58 +0100 |
commit | 02018a7d0222cefe8e6587deebe607917e20e2a1 (patch) | |
tree | 720b9fba5e69f69fb044e6be4fc502d7aaae7854 /test/ipc | |
parent | 6b88163343dc7aec6a6e5d3e964bba7b14f83dd6 (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>
Diffstat (limited to 'test/ipc')
-rw-r--r-- | test/ipc/unixsocket.cpp | 2 |
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; } |