Closed
Bug 413518
Opened 17 years ago
Closed 17 years ago
null-arg checks in nsMsgComposeService.cpp
Categories
(MailNews Core :: Backend, defect)
MailNews Core
Backend
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jminta, Assigned: jminta)
References
(Blocks 1 open bug)
Details
(Keywords: crash)
Attachments
(1 file)
|
835 bytes,
patch
|
standard8
:
review+
|
Details | Diff | Splinter Review |
A couple quick pointer checks needed here.
Attachment #298512 -
Flags: superreview?(dmose)
Attachment #298512 -
Flags: review?(bugzilla)
|
||
Comment 1•17 years ago
|
||
Comment on attachment 298512 [details] [diff] [review] patch v1 + NS_ENSURE_ARG_POINTER(aCmdLine); nsresult rv; nit: add blank line between these two lines. r=me.
Attachment #298512 -
Flags: review?(bugzilla) → review+
|
Assignee | |
Comment 2•17 years ago
|
||
Nits picked, checked in Checking in mailnews/compose/src/nsMsgComposeService.cpp; /cvsroot/mozilla/mailnews/compose/src/nsMsgComposeService.cpp,v <-- nsMsgComposeService.cpp new revision: 1.139; previous revision: 1.138 done
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Updated•16 years ago
|
Attachment #298512 -
Flags: superreview?(dmose)
|
||
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•