[Bug 53133] New: uiautomationcore:uiautomation test_uia_prov_from_acc_propertiescrashes on Windows 8

[WineHQ Bugzilla]

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

Bug ID: 53133
           Summary: uiautomationcore:uiautomation
                    test_uia_prov_from_acc_propertiescrashes on Windows 8
           Product: Wine
           Version: unspecified
          Hardware: x86-64
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: uiautomationcore
          Assignee: [e-mail]
          Reporter: [e-mail]
      Distribution: ---

uiautomationcore:uiautomation - crashes on Windows 8

[Bug 53133] uiautomationcore:uiautomation - test_uia_prov_from_acc_properties() crashes on Windows 8

[WineHQ Bugzilla]

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

François Gouget  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |regression, source,
                   |                            |testcase
                 CC|                            |[e-mail]
    Regression SHA1|                            |12dfa7425437e5c8f5caab40b7f
                   |                            |09beb22d1eab5
            Summary|uiautomationcore:uiautomati |uiautomationcore:uiautomati
                   |on                          |on -
                   |test_uia_prov_from_acc_prop |test_uia_prov_from_acc_prop
                   |ertiescrashes on Windows 8  |erties() crashes on Windows
                   |                            |8

--- Comment #1 from François Gouget  ---
Sorry, I hit the wrong key. Here's the remainder of the report.

uiautomationcore:uiautomation - test_uia_prov_from_acc_properties() crashes on
Windows 8:

uiautomation.c:911: this is the last test seen before the exception
094c:uiautomation: unhandled exception c0000005 at AEC2D668

https://test.winehq.org/data/patterns.html#uiautomationcore:uiautomation

A bisect shows that the crash was introduced on 2022-06-10 by the commit below:

commit 12dfa7425437e5c8f5caab40b7f09beb22d1eab5
Author: Connor McAdams 
Date:   Mon May 23 12:32:53 2022 -0400

uiautomationcore: Implement get_HostRawElementProvider for MSAA providers.

Signed-off-by: Connor McAdams

[Bug 53133] uiautomationcore:uiautomation - test_uia_prov_from_acc_properties() crashes on Windows 8

[WineHQ Bugzilla]

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

--- Comment #2 from Connor McAdams  ---
I will try to look into this today and see if I can find anything.

[Bug 53133] uiautomationcore:uiautomation - test_uia_prov_from_acc_properties() crashes on Windows 8

[WineHQ Bugzilla]

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

--- Comment #3 from Connor McAdams  ---
Hm, so I don't seem to be able to reproduce the issue, but I can see a
potential problem, which is that I'm not releasing the
IRawElementProviderSimple returned from get_HostRawElementProvider in the last
two calls to that method. I'll send a patch for this, and hopefully it helps.

Not sure why it doesn't seem to be reproducing consistently.

[Bug 53133] uiautomationcore:uiautomation - test_uia_prov_from_acc_properties() crashes on Windows 8

[WineHQ Bugzilla]

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

--- Comment #4 from Connor McAdams  ---
This should be fixed by commit 350220af30ceb631b03e5872680326dd7cd97647 now.

[Bug 53133] uiautomationcore:uiautomation - test_uia_prov_from_acc_properties() crashes on Windows 8

[WineHQ Bugzilla]

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

François Gouget  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Fixed by SHA1|                            |350220af30ceb631b03e5872680
                   |                            |326dd7cd97647
             Status|NEW                         |RESOLVED

--- Comment #5 from François Gouget  ---
Thanks for the fix.

[Bug 53133] uiautomationcore:uiautomation - test_uia_prov_from_acc_properties() crashes on Windows 8

[WineHQ Bugzilla]

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

Alexandre Julliard  changed:

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

--- Comment #6 from Alexandre Julliard  ---
Closing bugs fixed in 7.13.