On 08/27/2018 04:08 AM, Michal Privoznik wrote: > This will help in future expansions of the code when it is be > harder to track if @newName and/or @newLockspace were already > allocated or not and thus whether it is safe to 'return' or we > need to 'goto error'. By using the 'cleanup' label those two > cases merge into a single one. > > Signed-off-by: Michal Privoznik <mpriv...@redhat.com> > --- > src/locking/lock_driver_lockd.c | 44 > ++++++++++++++++++++--------------------- > 1 file changed, 22 insertions(+), 22 deletions(-) > Reviewed-by: John Ferlan <jfer...@redhat.com> John -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
- [libvirt] [PATCH v3 00/28] Introduce metadata locking Michal Privoznik
- [libvirt] [PATCH v3 04/28] lock_driver_lockd: Don't ... Michal Privoznik
- [libvirt] [PATCH v3 03/28] lock_daemon: Fix some mem... Michal Privoznik
- [libvirt] [PATCH v3 15/28] locking: Introduce virLoc... Michal Privoznik
- [libvirt] [PATCH v3 08/28] virLockManagerLockDaemonA... Michal Privoznik
- Re: [libvirt] [PATCH v3 08/28] virLockManagerLoc... John Ferlan
- [libvirt] [PATCH v3 07/28] locking: Don't leak priva... Michal Privoznik
- [libvirt] [PATCH v3 06/28] virLockManagerSanlockAddR... Michal Privoznik
- [libvirt] [PATCH v3 12/28] _virLockManagerLockDaemon... Michal Privoznik
- Re: [libvirt] [PATCH v3 12/28] _virLockManagerLo... John Ferlan
- Re: [libvirt] [PATCH v3 12/28] _virLockManag... Michal Privoznik
- [libvirt] [PATCH v3 16/28] lock_driver: Introduce KE... Michal Privoznik
- Re: [libvirt] [PATCH v3 16/28] lock_driver: Intr... John Ferlan
- Re: [libvirt] [PATCH v3 16/28] lock_driver: ... Michal Privoznik