nsPrintEngine::DoPrint leaks docTitleStr and docURLStr if !seqFrame

RESOLVED FIXED

Status

()

Core
Printing: Setup
--
minor
RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: timeless, Assigned: timeless)

Tracking

({coverity, memory-leak})

Trunk
coverity, memory-leak
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
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         }
(Assignee)

Updated

8 years ago
Keywords: coverity, mlk
(Assignee)

Comment 1

7 years ago
Created attachment 483784 [details] [diff] [review]
patch
Assignee: nobody → timeless
Status: NEW → ASSIGNED
Attachment #483784 - Flags: review?(roc)
Pushed http://hg.mozilla.org/mozilla-central/rev/8ab66adf43bb
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.