102171 - [WFM]The page http://www.cnn.com does not print, because of uninitialized variable.

Status: VERIFIED WORKSFORME

(Core :: Printing: Output, defect)

Description

[amardare]

From Bugzilla Helper:
User-Agent: Mozilla/4.0 (compatible; MSIE 5.5; Windows 98)
BuildID:    

Some pages ( ex: http://www.cnn.com ) do not print on our platform (QNX Rtp). I 
was debugging this and I think this is because of an uninitialized variable on 
the stack. Go to mozilla/content/base/src/nsDocumentViewer.cpp line:3650
The variable donePrinting it's not initialized and is also not modified in the 
DoPrint function. On windows and other platforms printing is ok because it 
happens to be a nonzero value on the stack.
I think it should be initialized with PR_TRUE ( or make sure it's set in the 
DoPrint function ).

Reproducible: Always
Steps to Reproduce:
1. Read the description please.
2.
3.

Comment 1

[sujay]

I just downloaded the 9/28 branch build
and this worksforme...I can print cnn.com w/o any problems

Comment 2

[sujay]

Asa, can you try this on the trunk ?

Comment 3

[sujay]

I tried in 9/28 trunk build also...worksforme on win98

Comment 4

[kinmoz]

This is not fixed yet, I just looked in the code. There are still cases in 
DoPrint() where we can leave the method without setting the aDonePrinting as the 
reporter pointed out.

Comment 5

[rods (gone)]

This prints fine with taday's tip build, taking bug

Comment 6

[rods (gone)]

re-open if still a problem

Comment 7

[sujay]

marking verified....please REOPEN if you still see this...