[Bug 55950] New: An attempt to print from 'wine iexplore' leads to a crash

[WineHQ Bugzilla]

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

Bug ID: 55950
           Summary: An attempt to print from 'wine iexplore' leads to a
                    crash
           Product: Wine
           Version: 8.21
          Hardware: x86-64
                OS: Linux
            Status: NEW
          Keywords: regression, source
          Severity: normal
          Priority: P2
         Component: wineps.drv
          Assignee: [e-mail]
          Reporter: [e-mail]
                CC: [e-mail]
   Regression SHA1: ca66b3eb8440db53fb110b559d29c6783be61e42
      Distribution: ---

Steps to reproduce:
1. wine iexplore http://google.com
2. File -> Print... (or press Print icon on the toolbar)
3. select Cups-PDF printer, Print to file, Pages from 1 To 1, press OK
4. enter file name (c:\aaa.ps), press OK

wine: Unhandled page fault on read access to 09A70001 at address 75781FB0
(thread 00d4), starting debugger...

Bisect points to
ca66b3eb8440db53fb110b559d29c6783be61e42 is the first bad commit
commit ca66b3eb8440db53fb110b559d29c6783be61e42
Author: Piotr Caban 
Date:   Wed Apr 26 19:26:21 2023 +0200

gdi32: Add support for creating EMF spool files.

Please note that printing has regressed several times, and this commit may
point to a different problem.

[Bug 55950] An attempt to print from 'wine iexplore' leads to a crash

[WineHQ Bugzilla]

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

--- Comment #1 from Piotr Caban  ---
The crash is caused by lack of error handling handling in get_vis_rectangles
helper (it may return incorrect rectangle in case of error that later leads to
a crash). I have prepared a patch for that but it reveals further problems.

Next issue is caused by incorrect ETO_PDY flag handling in EMFDC_ExtTextOut.
I'm currently working on a fix for that.

There are also some problems related to clipping but I didn't debug them yet.

[Bug 55950] An attempt to print from 'wine iexplore' leads to a crash

[WineHQ Bugzilla]

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

--- Comment #2 from Piotr Caban  ---
It should be fixed by patches from following merge requests:
https://gitlab.winehq.org/wine/wine/-/merge_requests/4543
https://gitlab.winehq.org/wine/wine/-/merge_requests/4549
https://gitlab.winehq.org/wine/wine/-/merge_requests/4550
https://gitlab.winehq.org/wine/wine/-/merge_requests/4563

[Bug 55950] An attempt to print from 'wine iexplore' leads to a crash

[WineHQ Bugzilla]

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

Dmitry Timoshkov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |ccbf1d25b68369a0ab2e72b32d4
                   |                            |c35c8dd7100bd
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #3 from Dmitry Timoshkov  ---
Hello Piotr, thank you very much for working on this.

The crash is fixed by ccbf1d25b68369a0ab2e72b32d4c35c8dd7100bd.

Remaining problems should be fixed by follow up pathces.

[Bug 55950] An attempt to print from 'wine iexplore' leads to a crash

[WineHQ Bugzilla]

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

Alexandre Julliard  changed:

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

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