Re: [PATCH 8/8] Always clear out the last_error in virshReportError.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 07/17/10 - 06:08:17PM, Matthias Bolte wrote:
> 2010/7/8 Chris Lalancette <clalance@xxxxxxxxxx>:
> > Otherwise you can get bogus "unknown error" printouts on
> > subsequent commands.
> >
> > Signed-off-by: Chris Lalancette <clalance@xxxxxxxxxx>
> > ---
> >  tools/virsh.c |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/tools/virsh.c b/tools/virsh.c
> > index 6a22071..6b7580b 100644
> > --- a/tools/virsh.c
> > +++ b/tools/virsh.c
> > @@ -390,7 +390,7 @@ virshReportError(void)
> >          * no error was ever raised, so just ignore */
> >         last_error = virSaveLastError();
> >         if (!last_error || last_error->code == VIR_ERR_OK)
> > -            return;
> > +            goto out;
> >     }
> >
> >     if (last_error->code == VIR_ERR_OK) {
> > --
> > 1.7.1.1
> 
> ACK.

Thanks, I also pushed this one.

--
Chris Lalancette

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]