[libvirt] [PATCH] locking: remove redundant codes
ajia at redhat.com
Thu Sep 22 01:50:03 UTC 2011
On 09/22/2011 09:35 AM, Eric Blake wrote:
> On 09/21/2011 07:15 PM, ajia at redhat.com wrote:
>> From: Alex Jia<ajia at redhat.com>
>> * src/locking/lock_driver_sanlock.c: remove redundant codes in
>> label from virLockManagerSanlockSetupLockspace, in fact, the codes
>> 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:
Agree, my patch is incomplete, it's okay for me.
More information about the libvir-list