On Thu, Feb 25, 2016 at 09:03:12AM -0500, John Ferlan wrote: > The 'secretLoad' was essentially open coding virFileBuildPath. > > Adjust the logic to have the caller build the path and pass it. The net > sum of ignoring the virFileBuildPath failure is the same as before where > the failure to virAsprintf the path would have been ignored anyway in > the secretLoad error path. > > Signed-off-by: John Ferlan <jferlan@xxxxxxxxxx> > --- > src/secret/secret_driver.c | 29 ++++++++++++++--------------- > 1 file changed, 14 insertions(+), 15 deletions(-) ACK Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list