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)
|
9.28 KB,
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
•