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)
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.