mirror of
https://git.proxmox.com/git/mirror_zfs.git
synced 2024-12-26 03:09:34 +03:00
Fix coverity defects: CID 150953, 147603, 147610
coverity scan CID:150953,type: uninitialized scalar variable coverity scan CID:147603,type: Resource leak coverity scan CID:147610,type: Resource leak Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov> Signed-off-by: luozhengzheng <luo.zhengzheng@zte.com.cn> Closes #5209
This commit is contained in:
parent
5cc78dc812
commit
e2c292bbfc
@ -6120,8 +6120,11 @@ share_mount(int op, int argc, char **argv)
|
||||
start_progress_timer();
|
||||
get_all_datasets(&dslist, &count, verbose);
|
||||
|
||||
if (count == 0)
|
||||
if (count == 0) {
|
||||
if (options != NULL)
|
||||
free(options);
|
||||
return (0);
|
||||
}
|
||||
|
||||
qsort(dslist, count, sizeof (void *), libzfs_dataset_cmp);
|
||||
|
||||
@ -6153,8 +6156,11 @@ share_mount(int op, int argc, char **argv)
|
||||
*/
|
||||
|
||||
/* Reopen MNTTAB to prevent reading stale data from open file */
|
||||
if (freopen(MNTTAB, "r", mnttab_file) == NULL)
|
||||
if (freopen(MNTTAB, "r", mnttab_file) == NULL) {
|
||||
if (options != NULL)
|
||||
free(options);
|
||||
return (ENOENT);
|
||||
}
|
||||
|
||||
while (getmntent(mnttab_file, &entry) == 0) {
|
||||
if (strcmp(entry.mnt_fstype, MNTTYPE_ZFS) != 0 ||
|
||||
@ -6184,6 +6190,9 @@ share_mount(int op, int argc, char **argv)
|
||||
}
|
||||
}
|
||||
|
||||
if (options != NULL)
|
||||
free(options);
|
||||
|
||||
return (ret);
|
||||
}
|
||||
|
||||
|
@ -599,8 +599,8 @@ cv_broadcast(kcondvar_t *cv)
|
||||
int
|
||||
vn_open(char *path, int x1, int flags, int mode, vnode_t **vpp, int x2, int x3)
|
||||
{
|
||||
int fd;
|
||||
int dump_fd;
|
||||
int fd = -1;
|
||||
int dump_fd = -1;
|
||||
vnode_t *vp;
|
||||
int old_umask = 0;
|
||||
char *realpath;
|
||||
@ -698,6 +698,8 @@ vn_open(char *path, int x1, int flags, int mode, vnode_t **vpp, int x2, int x3)
|
||||
if (fstat64_blk(fd, &st) == -1) {
|
||||
err = errno;
|
||||
close(fd);
|
||||
if (dump_fd != -1)
|
||||
close(dump_fd);
|
||||
return (err);
|
||||
}
|
||||
|
||||
|
@ -1282,7 +1282,7 @@ aes_common_init_ctx(aes_ctx_t *aes_ctx, crypto_spi_ctx_template_t *template,
|
||||
{
|
||||
int rv = CRYPTO_SUCCESS;
|
||||
void *keysched;
|
||||
size_t size;
|
||||
size_t size = 0;
|
||||
|
||||
if (template == NULL) {
|
||||
if ((keysched = aes_alloc_keysched(&size, kmflag)) == NULL)
|
||||
|
Loading…
Reference in New Issue
Block a user