Closed Bug 413518 Opened 15 years ago Closed 15 years ago

null-arg checks in nsMsgComposeService.cpp

Categories

(MailNews Core :: Backend, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jminta, Assigned: jminta)

References

(Blocks 1 open bug)

Details

(Keywords: crash)

Attachments

(1 file)

Attached patch patch v1Splinter Review
A couple quick pointer checks needed here.
Attachment #298512 - Flags: superreview?(dmose)
Attachment #298512 - Flags: review?(bugzilla)
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+
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: 15 years ago
Resolution: --- → FIXED
Attachment #298512 - Flags: superreview?(dmose)
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.