diff options
| author | cao <[email protected]> | 2016-09-21 08:45:45 +0800 |
|---|---|---|
| committer | Brian Behlendorf <[email protected]> | 2016-09-20 17:45:45 -0700 |
| commit | 884385a0b25dc36b622e5e427bf29281c3dbca06 (patch) | |
| tree | d5f09f9cc5ecc0fa9e381292c943877dc983a2b5 /cmd/zinject | |
| parent | d2a6f65cfb65ec3079f843542a156704264260be (diff) | |
Fix coverity defects
Fix coverity defects:
coverity scan CID:147623, Type: Resource leak.
coverity scan CID:147622, Type: Resource leak.
reason: zpool_open zhp, but not zpool_close zhp. so resource leak.
coverity scan CID:147621, Type: Resource fd leak.
coverity scan CID:147620, Type: Resource fd leak.
reason: do_write do_read open file fd,but exception not close fd.
delete unuse definition DMU_OS_IS_L2COMPRESSIBLE.
Reviewed-by: Brian Behlendorf <[email protected]>
Signed-off-by: cao.xuewen <[email protected]>
Closes #5137
Diffstat (limited to 'cmd/zinject')
| -rw-r--r-- | cmd/zinject/translate.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/cmd/zinject/translate.c b/cmd/zinject/translate.c index adace72ff..1aef074d3 100644 --- a/cmd/zinject/translate.c +++ b/cmd/zinject/translate.c @@ -474,6 +474,7 @@ translate_device(const char *pool, const char *device, err_type_t label_type, if (tgt == NULL) { (void) fprintf(stderr, "cannot find device '%s' in " "pool '%s'\n", device, pool); + zpool_close(zhp); return (-1); } @@ -515,5 +516,6 @@ translate_device(const char *pool, const char *device, err_type_t label_type, record->zi_end = record->zi_start + VDEV_PAD_SIZE - 1; break; } + zpool_close(zhp); return (0); } |