diff options
| author | Nandaja Varma <nvarma@redhat.com> | 2015-02-05 13:37:37 +0530 |
|---|---|---|
| committer | Vijay Bellur <vbellur@redhat.com> | 2015-03-30 00:42:49 -0700 |
| commit | b828932c1430bfc62d7c3c83fec8e4a79ca4dbe6 (patch) | |
| tree | cdb272fc9b37e8642adf1acb0c1fb15e8b0cb480 /xlators/storage/bd/src/bd.c | |
| parent | f2c1261123798403dd254926db4f2008de4b6d84 (diff) | |
bd: coverity fixes, removing logically dead code and correcting checks
Coverity CIDs:
1128910
1128911
1128913
1128912
1134020
Change-Id: I2d871723fbfe43f9ff6b3beba7a99b0d81d4aff5
BUG: 789278
Signed-off-by: Nandaja Varma <nvarma@redhat.com>
Reviewed-on: http://review.gluster.org/9588
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'xlators/storage/bd/src/bd.c')
| -rw-r--r-- | xlators/storage/bd/src/bd.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/xlators/storage/bd/src/bd.c b/xlators/storage/bd/src/bd.c index 6ea18e7787b..85372f072ed 100644 --- a/xlators/storage/bd/src/bd.c +++ b/xlators/storage/bd/src/bd.c @@ -693,7 +693,8 @@ out: GF_FREE (devpath); if (ret) { - close (_fd); + if (_fd >= 0) + close (_fd); GF_FREE (bd_fd); } |