22 Sep
2011
22 Sep
'11
1:35 a.m.
On 09/21/2011 07:15 PM, ajia@redhat.com wrote:
From: Alex Jia<ajia@redhat.com>
* src/locking/lock_driver_sanlock.c: remove redundant codes in error_unlink label from virLockManagerSanlockSetupLockspace, in fact, the codes make sure 'path' is non-null before getting to error_unlink label, so 'if (path)' is redundant, removing it in here and will also silence coverity.
I'd rather go with my more comprehensive patch, since it also fixes a mem leak: https://www.redhat.com/archives/libvir-list/2011-September/msg00865.html -- Eric Blake eblake@redhat.com +1-801-349-2682 Libvirt virtualization library http://libvirt.org