diff options
author | Umang Jain <email@uajain.com> | 2020-05-15 12:42:53 +0000 |
---|---|---|
committer | Kieran Bingham <kieran.bingham@ideasonboard.com> | 2020-06-04 11:02:38 +0100 |
commit | 3706b716eba66e77c27ab365f54e6b13f145fea8 (patch) | |
tree | 1c60316e76dc51324d140d4fedc15434de086754 | |
parent | 5bb6607933f28d2484b68034f871cd58074ae695 (diff) |
test: log: log_api: Close open fds on error paths
Reported-by: Coverity CID=279091
Signed-off-by: Umang Jain <email@uajain.com>
Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
-rw-r--r-- | test/log/log_api.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/test/log/log_api.cpp b/test/log/log_api.cpp index 6f3248a7..ae3c607a 100644 --- a/test/log/log_api.cpp +++ b/test/log/log_api.cpp @@ -87,6 +87,7 @@ protected: if (logSetFile(path) < 0) { cerr << "Failed to set log file" << endl; + close(fd); return TestFail; } @@ -97,6 +98,7 @@ protected: lseek(fd, 0, SEEK_SET); if (read(fd, buf, sizeof(buf)) < 0) { cerr << "Failed to read tmp log file" << endl; + close(fd); return TestFail; } close(fd); |