[Bug 16331] New: inetcomm: possible null pointer dereferences
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(a)winehq.org ReportedBy: grschneider(a)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. -- 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.
http://bugs.winehq.org/show_bug.cgi?id=16331 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |patch -- 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.
http://bugs.winehq.org/show_bug.cgi?id=16331 Vitaliy Margolen <vitaliy(a)kievinfo.com> changed: What |Removed |Added ---------------------------------------------------------------------------- OS/Version|Windows XP |other Version|CVS/GIT |1.1.9 --- Comment #1 from Vitaliy Margolen <vitaliy(a)kievinfo.com> 2008-12-04 14:06:35 --- Please submit patches to wine-patches(a)winehq.org mailing list. They are not being picked up from bugzilla. -- 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.
http://bugs.winehq.org/show_bug.cgi?id=16331 Gregor Schneider <grschneider(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution| |INVALID --- Comment #2 from Gregor Schneider <grschneider(a)gmail.com> 2008-12-04 14:34:19 --- Patch submitted to wine-patches list, thanks for the information. Closing as invalid. -- 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.
http://bugs.winehq.org/show_bug.cgi?id=16331 James Hawkins <truiken(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |UNCONFIRMED Resolution|INVALID | --- Comment #3 from James Hawkins <truiken(a)gmail.com> 2008-12-04 15:21:08 --- Why did you mark this as invalid? -- 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.
http://bugs.winehq.org/show_bug.cgi?id=16331 --- Comment #4 from Vitaliy Margolen <vitaliy(a)kievinfo.com> 2008-12-04 20:07:21 --- Gregor, the bug will be closed when patch is accepted and committed to Wine tree. -- 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.
http://bugs.winehq.org/show_bug.cgi?id=16331 --- Comment #5 from Dmitry Timoshkov <dmitry(a)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? -- 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.
http://bugs.winehq.org/show_bug.cgi?id=16331 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution| |FIXED --- Comment #6 from Austin English <austinenglish(a)gmail.com> 2009-06-08 12:31:02 --- This was fixed by e7f4f3b69cbf9a5e248fbf696f11c281f9f20a70. Apparently coverity agreed about it being dangerous ;-). -- 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.
http://bugs.winehq.org/show_bug.cgi?id=16331 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #7 from Alexandre Julliard <julliard(a)winehq.org> 2009-06-19 11:02:55 --- Closing bugs fixed in 1.1.24. -- 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.
participants (1)
-
wine-bugs＠winehq.org