[libvirt] [PATCH] virsh: report errors in virshInit()

Daniel P. Berrange berrange at redhat.com
Thu Apr 20 09:14:59 UTC 2017


On Wed, Apr 19, 2017 at 08:10:22PM -0400, John Ferlan wrote:
> 
> 
> On 04/19/2017 06:43 AM, Daniel P. Berrange wrote:
> > There are several functions in virshInit which can fail, especially
> > when running win32 builds under WINE. Currently virsh just exits
> > without reporting what error happened.
> > 
> > Signed-off-by: Daniel P. Berrange <berrange at redhat.com>
> > ---
> >  tools/virsh.c | 12 +++++++++---
> >  1 file changed, 9 insertions(+), 3 deletions(-)
> > 
> 
> Why not a "goto error;" instead where error: does the
> vshReportError(ctl); return false;
> 
> ACK either way

I realize I accidentally pushed this patch already yesterday when
pushing the RBD build breaker fix.


Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|




More information about the libvir-list mailing list