[libvirt] [PATCH] vz: fix ref/unref of domain obj in vzDomainRevertToSnapshot

[Peter Krempa]

On Wed, Feb 28, 2018 at 10:36:19 +0100, Michal Privoznik wrote:
> On 02/28/2018 07:31 AM, Nikolay Shirokovskiy wrote:
> > ---
> >  src/vz/vz_driver.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/src/vz/vz_driver.c b/src/vz/vz_driver.c
> > index dab1ba8..18f4b4c 100644
> > --- a/src/vz/vz_driver.c
> > +++ b/src/vz/vz_driver.c
> > @@ -2732,7 +2732,7 @@ vzDomainRevertToSnapshot(virDomainSnapshotPtr snapshot, unsigned int flags)
> >  
> >      virCheckFlags(VIR_DOMAIN_SNAPSHOT_REVERT_PAUSED, -1);
> >  
> > -    if (!(dom = vzDomObjFromDomain(snapshot->domain)))
> > +    if (!(dom = vzDomObjFromDomainRef(snapshot->domain)))
> >          return -1;
> >  
> >      if (virDomainRevertToSnapshotEnsureACL(snapshot->domain->conn, dom->def) < 0)
> > 
> 
> Ooops, yes. Added SoB line, ACKed and pushed.

I think the idea of the SoB line is that you can't add it yourself. You
have to ask the author of the patch for the permission to do so.

Adding them just to make the push-hook happy defeats the statement of
the developer certificate of origin completely.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20180228/9e7bcc9d/attachment-0001.sig>