Closed Bug 604632 Opened 9 years ago Closed 9 years ago

nsPrintEngine::DoPrint leaks docTitleStr and docURLStr if !seqFrame

Categories

(Core :: Printing: Setup, defect, minor)

defect
Not set
minor

Tracking

()

RESOLVED FIXED

People

(Reporter: timeless, Assigned: timeless)

References

(Blocks 1 open bug)

Details

(Keywords: coverity, memory-leak)

Attachments

(1 file)

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         }
Keywords: coverity, mlk
Attached patch patchSplinter Review
Assignee: nobody → timeless
Status: NEW → ASSIGNED
Attachment #483784 - Flags: review?(roc)
Pushed http://hg.mozilla.org/mozilla-central/rev/8ab66adf43bb
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.