On 01/09/13 15:54, John Ferlan wrote: > --- > src/locking/lock_driver_sanlock.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/src/locking/lock_driver_sanlock.c b/src/locking/lock_driver_sanlock.c > index d06fa66..df6bee9 100644 > --- a/src/locking/lock_driver_sanlock.c > +++ b/src/locking/lock_driver_sanlock.c > @@ -235,6 +235,7 @@ static int virLockManagerSanlockSetupLockspace(void) > path); > goto error; > } > + VIR_FREE(dir); > > if (driver->group != (gid_t) -1) > perms |= 0060; That does fix the 'dir' leak, but it seems we still leak 'path' on non-error path this way. -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list