diff options
| author | M. Mohan Kumar <mohan@in.ibm.com> | 2013-11-15 18:23:29 +0530 |
|---|---|---|
| committer | Vijay Bellur <vbellur@redhat.com> | 2013-11-19 09:58:55 -0800 |
| commit | 56d0124224259ccfc4c8b1cc33939902ed480d23 (patch) | |
| tree | 35e31f55fabe106e0107b683380a885a4bf7adaf /xlators/storage/bd | |
| parent | 1ef8a597db1ead482612f2f0bcc212d9a1349ccb (diff) | |
BD fixes for coverity scan
BUG: 1028672
Change-Id: I2e7889fb113cedd2d5928b210149d3fd7b8b22ab
Signed-off-by: M. Mohan Kumar <mohan@in.ibm.com>
Reviewed-on: http://review.gluster.org/6292
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'xlators/storage/bd')
| -rw-r--r-- | xlators/storage/bd/src/bd.c | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/xlators/storage/bd/src/bd.c b/xlators/storage/bd/src/bd.c index 405474c587d..1eb5cd15838 100644 --- a/xlators/storage/bd/src/bd.c +++ b/xlators/storage/bd/src/bd.c @@ -1102,7 +1102,7 @@ out: if (local->fd) BD_STACK_UNWIND (fsetxattr, frame, op_ret, op_errno, NULL); else - BD_STACK_UNWIND (setxattr, frame, op_errno, op_errno, NULL); + BD_STACK_UNWIND (setxattr, frame, op_ret, op_errno, NULL); return 0; } @@ -2328,11 +2328,15 @@ init (xlator_t *this) return 0; error: - GF_FREE (_private->vg); - if (_private->handle) - lvm_quit (_private->handle); + if (_private) { + GF_FREE (_private->vg); + if (_private->handle) + lvm_quit (_private->handle); + GF_FREE (_private); + } + mem_pool_destroy (this->local_pool); - GF_FREE (_private); + return -1; } |