Closed
Bug 604632
Opened 14 years ago
Closed 14 years ago
nsPrintEngine::DoPrint leaks docTitleStr and docURLStr if !seqFrame
Categories
(Core :: Printing: Setup, defect)
Core
Printing: Setup
Tracking
()
RESOLVED
FIXED
People
(Reporter: timeless, Assigned: timeless)
References
(Blocks 1 open bug)
Details
(Keywords: coverity, memory-leak)
Attachments
(1 file)
(deleted),
patch
|
roc
:
review+
roc
:
approval2.0+
|
Details | Diff | Splinter Review |
2256 nsPrintEngine::DoPrint(nsPrintObject * aPO) 2257 { 2309 if (mPrt->mPrintSettings) { 2310 PRUnichar * docTitleStr = nsnull; 2311 PRUnichar * docURLStr = nsnull; 2312 2313 GetDisplayTitleAndURL(aPO, &docTitleStr, &docURLStr, eDocTitleDefBlank); 2390 nsIFrame * seqFrame = do_QueryFrame(pageSequence); 2391 if (!seqFrame) { 2392 SetIsPrinting(PR_FALSE); 2393 return NS_ERROR_FAILURE; 2394 }
Attachment #483784 -
Flags: review?(roc)
Attachment #483784 -
Flags: review+
Attachment #483784 -
Flags: approval2.0+
Comment 2•14 years ago
|
||
Pushed http://hg.mozilla.org/mozilla-central/rev/8ab66adf43bb
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Blocks: coverity-analysis
You need to log in
before you can comment on or make changes to this bug.
Description
•