Re: [libvirt] [PATCH] locking: remove redundant codes

On 09/21/2011 07:15 PM, ajia(a)redhat.com wrote: > From: Alex Jia<ajia(a)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