[PATCH] msi: Fix a typo.

[Dmitry Timoshkov]

This patch should fix the regresion introduced by baea371c3d4153bc4a718c07f46971d4d39aa9f3.

Signed-off-by: Dmitry Timoshkov <dmitry(a)baikal.ru>
---
 dlls/msi/package.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/msi/package.c b/dlls/msi/package.c
index 3374964078..8e9ae6dd90 100644
--- a/dlls/msi/package.c
+++ b/dlls/msi/package.c
@@ -1742,7 +1742,7 @@ MSIHANDLE WINAPI MsiGetActiveDatabase(MSIHANDLE hInstall)
     {
         __TRY
         {
-            handle = remote_GetActiveDatabase(hInstall);
+            handle = remote_GetActiveDatabase(remote);
             handle = alloc_msi_remote_handle(handle);
         }
         __EXCEPT(rpc_filter)
-- 
2.17.1

[Hans Leidekker]

Signed-off-by: Hans Leidekker <hans(a)codeweavers.com>

[Zebediah Figura]

On 18/09/18 00:59, Dmitry Timoshkov wrote:
> This patch should fix the regresion introduced by baea371c3d4153bc4a718c07f46971d4d39aa9f3.
>
> Signed-off-by: Dmitry Timoshkov <dmitry(a)baikal.ru>
> ---
>  dlls/msi/package.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/dlls/msi/package.c b/dlls/msi/package.c
> index 3374964078..8e9ae6dd90 100644
> --- a/dlls/msi/package.c
> +++ b/dlls/msi/package.c
> @@ -1742,7 +1742,7 @@ MSIHANDLE WINAPI MsiGetActiveDatabase(MSIHANDLE hInstall)
>      {
>          __TRY
>          {
> -            handle = remote_GetActiveDatabase(hInstall);
> +            handle = remote_GetActiveDatabase(remote);
>              handle = alloc_msi_remote_handle(handle);
>          }
>          __EXCEPT(rpc_filter)

*facepalm*

Thanks for finding that; I can't *believe* it got past the tests.

[Dmitry Timoshkov]

Zebediah Figura <z.figura12(a)gmail.com> wrote:
> *facepalm*
>
> Thanks for finding that;

You're welcome.
> I can't *believe* it got past the tests.

Probably it's the proper time to add some :)

-- 
Dmitry.

[Zebediah Figura]

On 18/09/18 09:29, Dmitry Timoshkov wrote:
> Zebediah Figura <z.figura12(a)gmail.com> wrote:
> > *facepalm*
> >
> > Thanks for finding that;
>
> You're welcome.
> > I can't *believe* it got past the tests.
>
> Probably it's the proper time to add some :)

That particular function does have tests; I suspect that the allocated
handle numbers just ended up being the same on the local and remote sides...