Re: [PATCH] qemu: Add flags checking in DomainCoreDump

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

 



On Fri, Apr 22, 2011 at 09:54:03 -0600, Eric Blake wrote:
> On 04/22/2011 04:03 AM, Jiri Denemark wrote:
> > ---
> >  src/qemu/qemu_driver.c |    5 ++++-
> >  1 files changed, 4 insertions(+), 1 deletions(-)
> > 
> > diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
> > index f6e503a..f35616d 100644
> > --- a/src/qemu/qemu_driver.c
> > +++ b/src/qemu/qemu_driver.c
> > @@ -2345,7 +2345,8 @@ getCompressionType(struct qemud_driver *driver)
> >  
> >  static int qemudDomainCoreDump(virDomainPtr dom,
> >                                 const char *path,
> > -                               int flags ATTRIBUTE_UNUSED) {
> > +                               int flags)
> > +{
> >      struct qemud_driver *driver = dom->conn->privateData;
> >      virDomainObjPtr vm;
> >      int resume = 0, paused = 0;
> > @@ -2353,6 +2354,8 @@ static int qemudDomainCoreDump(virDomainPtr dom,
> >      virDomainEventPtr event = NULL;
> >      qemuDomainObjPrivatePtr priv;
> >  
> > +    virCheckFlags(VIR_DUMP_LIVE | VIR_DUMP_CRASH, -1);
> > +
> 
> ACK.

Pushed, thanks.

Jirka

--
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]