mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2024-09-08 16:54:49 +00:00
storage_backend_fs: Don't free a part of a structure on error
As the storage pool sources are stored in a list of structs, the pointer returned by virStoragePoolSourceListNewSource() shouldn't be freed as it points in the middle of a memory block. This combined with a regression that takes the error path every time on caused a double-free abort on the src struct in question.
This commit is contained in:
parent
73f4b30882
commit
ab9c72ae9e
@ -218,10 +218,8 @@ virStorageBackendFileSystemNetFindPoolSourcesFunc(virStoragePoolObjPtr pool ATTR
|
||||
}
|
||||
src->format = VIR_STORAGE_POOL_NETFS_NFS;
|
||||
|
||||
src = NULL;
|
||||
ret = 0;
|
||||
cleanup:
|
||||
virStoragePoolSourceFree(src);
|
||||
return ret;
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user