[Bug 13394] gtk-dev-2.12.9-win32-2.exe crashes on installation

wine-bugs at winehq.org wine-bugs at winehq.org
Mon Apr 13 04:24:51 CDT 2009


--- Comment #22 from Lionel Debroux <lionel_debroux at yahoo.fr>  2009-04-13 04:24:43 ---
Created an attachment (id=20414)
 --> (http://bugs.winehq.org/attachment.cgi?id=20414)
Check the result of a HeapAlloc() against NULL

Nikolay Sivov sent the patch he posted here to wine-patches:

James Hawkins replied that the functionality should be moved to copy_files():

Nikolay modified his patch to move the code to copy_files() and decide
whether to use the current directory upon flTo->feFiles == NULL:
That looks reasonable to me, because of
ZeroMemory(&flTo, sizeof(FILE_LIST));
in SHFileOperationW, and parse_file_list returning an error if its szFiles
argument is an empty string, before allocating flList (flTo, in this case)

Is there something wrong with Nikolay's second try (e.g. a failure on native
Windows hosts) ?

While checking Nikolay's patch, I found one place where the result of a
HeapAlloc invocation is not checked against NULL. I'm attaching a patch for it.

Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
Do not reply to this email, post in Bugzilla using the
above URL to reply.
------- You are receiving this mail because: -------
You are watching all bug changes.

More information about the wine-bugs mailing list