Start VSS Provider after install

[Qemu-devel] [PATCH qemu-ga] Start VSS Provider after install

Posted by Sameeh Jubran 7 years, 1 month ago

Signed-off-by: Sameeh Jubran <sameeh@daynix.com>
---
 qga/vss-win32/install.cpp | 1 +
 1 file changed, 1 insertion(+)

diff --git a/qga/vss-win32/install.cpp b/qga/vss-win32/install.cpp
index f4160a3..7e38332 100644
--- a/qga/vss-win32/install.cpp
+++ b/qga/vss-win32/install.cpp
@@ -307,6 +307,7 @@ STDAPI COMRegister(void)
     chk(put_Value(pObj, L"User", L"SYSTEM"));
     chk(pUsersInRole->SaveChanges(&n));
 
+    chk(pCatalog->StartApplication(_bstr_t(QGA_PROVIDER_LNAME)));
 out:
     if (unregisterOnFailure && FAILED(hr)) {
         COMUnregister();
-- 
2.9.3

Re: [Qemu-devel] [PATCH qemu-ga] Start VSS Provider after install

Posted by Michael Roth 7 years, 1 month ago

Quoting Sameeh Jubran (2017-03-21 07:03:26)
> Signed-off-by: Sameeh Jubran <sameeh@daynix.com>

What happens without this patch? Fresh installs don't report the
fsfreeze interface as available?

> ---
>  qga/vss-win32/install.cpp | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/qga/vss-win32/install.cpp b/qga/vss-win32/install.cpp
> index f4160a3..7e38332 100644
> --- a/qga/vss-win32/install.cpp
> +++ b/qga/vss-win32/install.cpp
> @@ -307,6 +307,7 @@ STDAPI COMRegister(void)
>      chk(put_Value(pObj, L"User", L"SYSTEM"));
>      chk(pUsersInRole->SaveChanges(&n));
> 
> +    chk(pCatalog->StartApplication(_bstr_t(QGA_PROVIDER_LNAME)));
>  out:
>      if (unregisterOnFailure && FAILED(hr)) {
>          COMUnregister();
> -- 
> 2.9.3
>

Re: [Qemu-devel] [PATCH qemu-ga] Start VSS Provider after install

Posted by Sameeh Jubran 7 years, 1 month ago

On Tue, Mar 21, 2017 at 6:15 PM, Michael Roth <mdroth@linux.vnet.ibm.com>
wrote:

> Quoting Sameeh Jubran (2017-03-21 07:03:26)
> > Signed-off-by: Sameeh Jubran <sameeh@daynix.com>
>
> What happens without this patch? Fresh installs don't report the
> fsfreeze interface as available?
>
 It solves a bug (https://bugzilla.redhat.com/show_bug.cgi?id=1218937) that
was reported in bugzilla, however
 thinking about it again this patch is not needed as the service should
only be running when it is needed.

>
> > ---
> >  qga/vss-win32/install.cpp | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/qga/vss-win32/install.cpp b/qga/vss-win32/install.cpp
> > index f4160a3..7e38332 100644
> > --- a/qga/vss-win32/install.cpp
> > +++ b/qga/vss-win32/install.cpp
> > @@ -307,6 +307,7 @@ STDAPI COMRegister(void)
> >      chk(put_Value(pObj, L"User", L"SYSTEM"));
> >      chk(pUsersInRole->SaveChanges(&n));
> >
> > +    chk(pCatalog->StartApplication(_bstr_t(QGA_PROVIDER_LNAME)));
> >  out:
> >      if (unregisterOnFailure && FAILED(hr)) {
> >          COMUnregister();
> > --
> > 2.9.3
> >
>
>


-- 
Respectfully,
*Sameeh Jubran*
*Linkedin <https://il.linkedin.com/pub/sameeh-jubran/87/747/a8a>*
*Software Engineer @ Daynix <http://www.daynix.com>.*