[Bug 35612] New: jscript:activex fails if the user does not have sufficient privileges

https://bugs.winehq.org/show_bug.cgi?id=35612

Bug ID: 35612
           Summary: jscript:activex fails if the user does not have
                    sufficient privileges
           Product: Wine
           Version: 1.7.12
          Hardware: x86
                OS: Windows
            Status: NEW
          Severity: normal
          Priority: P2
         Component: jscript
          Assignee: [e-mail]
          Reporter: [e-mail]
    Classification: Unclassified

If the user does not have elevated privileges, then we get 81 test failures.
This can be seen in the fg-win7u64-1spie9-adm, fg-win7u64-spie9-usr VM results
and also happens on my Windows 8.1 Acer laptop (fg-acer64-w81-t32 and    
fg-acer64-w81-t64). See:

http://test.winehq.org/data/tests/jscript:activex.html

Mostly it seems to be the IActiveScriptParse::ParseScriptText() calls that are
failing with all the other failures being natural consequences of that. So I'm
only showing the ParseScriptText failures below:

activex.c:828: Test failed: ParseScriptText failed: 800a01ad
activex.c:955: Test failed: ParseScriptText failed: 800a01ad
activex.c:1006: Test failed: ParseScriptText failed: 800a01ad
activex.c:1023: Test failed: ParseScriptText failed: 800a01ad
activex.c:1056: Test failed: ParseScriptText failed: 800a01ad

I suspect the issue is that without elevated privileges we cannot
register/install our Wine.Test ActiveX object. I would expect to get failures a
bit earlier though.

Probably the solution is to include the is_process_limited() function and skip
the tests at some point if it returns TRUE.

[Bug 35612] jscript:activex fails if the user does not have sufficient privileges

https://bugs.winehq.org/show_bug.cgi?id=35612

--- Comment #1 from François Gouget  ---
vbscript:createobj seems to have a very similar problem:

createobj.c:787: Test failed: ParseScriptText failed: 800a01ad
createobj.c:877: Test failed: ParseScriptText failed: 800a01ad
createobj.c:931: Test failed: ParseScriptText failed: 800a01ad
createobj.c:948: Test failed: ParseScriptText failed: 800a01ad

[Bug 35612] jscript:activex fails if the user does not have sufficient privileges

https://bugs.winehq.org/show_bug.cgi?id=35612

Jacek Caban  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #2 from Jacek Caban  ---
We should skip tests if register_activex fails, but we weren't checking its
result. I've sent a patch:

http://source.winehq.org/patches/data/102561

[Bug 35612] jscript:activex fails if the user does not have sufficient privileges

https://bugs.winehq.org/show_bug.cgi?id=35612

Jacek Caban  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |cd22e8c223bc3cd55ff86aa4da0
                   |                            |3c3d8b6fb3e41
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #3 from Jacek Caban  ---
Fixed in Git:

http://source.winehq.org/git/wine.git/commit/cd22e8c223bc3cd55ff86aa4da03c3d...

I also sent similar patches for vbscript and wscript:

http://source.winehq.org/patches/data/102582
http://source.winehq.org/patches/data/102583

[Bug 35612] jscript:activex fails if the user does not have sufficient privileges

https://bugs.winehq.org/show_bug.cgi?id=35612

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #4 from Alexandre Julliard  ---
Closing bugs fixed in 1.7.13.