[0.4.9][APPWIZ] Don't hung up in Gecko download cancellation #3049 CORE-14538
authorJoachim Henze <Joachim.Henze@reactos.org>
Sun, 17 Jan 2021 21:45:00 +0000 (22:45 +0100)
committerJoachim Henze <Joachim.Henze@reactos.org>
Sun, 17 Jan 2021 21:45:00 +0000 (22:45 +0100)
commit7bbe9f94baa071843274da46f7425f1d4cca2417
treec4fc7dac215c2c7e654daca192509c92c8c45f72
parent11b2653a65c897d69af706c9877c51604c7cfa62
[0.4.9][APPWIZ] Don't hung up in Gecko download cancellation #3049 CORE-14538

Do not call EndDialog outside the dialog procedure.

fix picked from commit 0.4.15-dev-511-g 90456e54a42604909bf2d93cb74abbbf6574268f

This hang/crash in 2nd stage setup did *not* affect our official carrier-releaser isos that
we host on sourceforge, because those come bundled with Gecko already and therefore always
skipped the download.
Most users were safe in the past therefore.
Still everyone that would like to build an iso by himself could potentially run
into those crashes, and it did not only happen upon user-driven cancellation,
but also often crashed upon network failure / timeout.
A very important fix therefore.
For some unknown reason the likelihood of this crash gradually increased from
0.4.7 where it happened almost never, until 0.4.14 where it crashed much more likely.
Not well understood, maybe depends on memory layout or something similar.
dll/cpl/appwiz/addons.c