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

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.

On 02/28/2018 10:51 AM, Peter Krempa wrote: > 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. > Oops yes. Sorry.