http://bugs.winehq.org/show_bug.cgi?id=16331
Summary: inetcomm: possible null pointer dereferences Product: Wine Version: CVS/GIT Platform: PC-x86-64 OS/Version: Windows XP Status: UNCONFIRMED Severity: enhancement Priority: P2 Component: -unknown AssignedTo: wine-bugs@winehq.org ReportedBy: grschneider@gmail.com
Created an attachment (id=17635) --> (http://bugs.winehq.org/attachment.cgi?id=17635) suggested patch
Three possible NULL pointer dereferences in dlls\inetcomm\smtptransport.c, fixed by doing the parameter check first and using the checked variable afterwards.
http://bugs.winehq.org/show_bug.cgi?id=16331
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |patch
http://bugs.winehq.org/show_bug.cgi?id=16331
Vitaliy Margolen vitaliy@kievinfo.com changed:
What |Removed |Added ---------------------------------------------------------------------------- OS/Version|Windows XP |other Version|CVS/GIT |1.1.9
--- Comment #1 from Vitaliy Margolen vitaliy@kievinfo.com 2008-12-04 14:06:35 --- Please submit patches to wine-patches@winehq.org mailing list. They are not being picked up from bugzilla.
http://bugs.winehq.org/show_bug.cgi?id=16331
Gregor Schneider grschneider@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution| |INVALID
--- Comment #2 from Gregor Schneider grschneider@gmail.com 2008-12-04 14:34:19 --- Patch submitted to wine-patches list, thanks for the information. Closing as invalid.
http://bugs.winehq.org/show_bug.cgi?id=16331
James Hawkins truiken@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |UNCONFIRMED Resolution|INVALID |
--- Comment #3 from James Hawkins truiken@gmail.com 2008-12-04 15:21:08 --- Why did you mark this as invalid?
http://bugs.winehq.org/show_bug.cgi?id=16331
--- Comment #4 from Vitaliy Margolen vitaliy@kievinfo.com 2008-12-04 20:07:21 --- Gregor, the bug will be closed when patch is accepted and committed to Wine tree.
http://bugs.winehq.org/show_bug.cgi?id=16331
--- Comment #5 from Dmitry Timoshkov dmitry@codeweavers.com 2008-12-04 22:23:21 --- (In reply to comment #0)
Three possible NULL pointer dereferences in dlls\inetcomm\smtptransport.c, fixed by doing the parameter check first and using the checked variable afterwards.
Did you test under Windows that it really doesn't crash and checks for parameters being NULL?
http://bugs.winehq.org/show_bug.cgi?id=16331
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution| |FIXED
--- Comment #6 from Austin English austinenglish@gmail.com 2009-06-08 12:31:02 --- This was fixed by e7f4f3b69cbf9a5e248fbf696f11c281f9f20a70. Apparently coverity agreed about it being dangerous ;-).
http://bugs.winehq.org/show_bug.cgi?id=16331
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #7 from Alexandre Julliard julliard@winehq.org 2009-06-19 11:02:55 --- Closing bugs fixed in 1.1.24.
-
wine-bugs＠winehq.org