Closed Bug 119544 Opened 23 years ago Closed 23 years ago

Uninitialized memory read in nsBoxToBlockAdaptor::DoLayout

Categories

(Core :: Layout, defect)

x86
Windows 2000
defect
Not set
major

Tracking

()

VERIFIED DUPLICATE of bug 68265

People

(Reporter: stephend, Assigned: hyatt)

Details

Build ID: Latest win32 trunk running under Purify. Summary: Uninitialized memory read in nsBoxToBlockAdaptor::DoLayout Steps to Reproduce: 1. start mozilla 2. go to http://lxr.mozilla.org/seamonkey/source/layout/xul/base/src/nsBoxFrame.cpp 3. file -> send page 4. enter a recipient and click send 5. exit [W] UMR: Uninitialized memory read in nsBoxToBlockAdaptor::DoLayout (nsBoxLayoutState&) {1 occurrence} Reading 4 bytes from 0x0013a274 (4 bytes at 0x0013a274 uninitialized) Address 0x0013a274 points into a thread's stack Thread ID: 0x60c Error location nsBoxToBlockAdaptor::DoLayout(nsBoxLayoutState&) [nsBoxToBlockAdaptor.cpp:607] ourRect.height); if (currentSize) { => if (maxElementSize.width > currentSize->width) currentSize->width = maxElementSize.width; if (mCachedMaxElementHeight > currentSize->height) { nsBox::Layout(nsBoxLayoutState&) [nsBox.cpp:1050] { BeginLayout(aState); => DoLayout(aState); EndLayout(aState); nsScrollBoxFrame::DoLayout(nsBoxLayoutState&) [nsScrollBoxFrame.cpp:393] aState.SetLayoutFlags(flags); kid->SetBounds(aState, childRect); => kid->Layout(aState); kid->GetBounds(childRect); nsBox::Layout(nsBoxLayoutState&) [nsBox.cpp:1050] { BeginLayout(aState); => DoLayout(aState); EndLayout(aState); nsContainerBox::LayoutChildAt(nsBoxLayoutState&,nsIBox *,nsRect const&) [nsContainerBox.cpp:650] layout = PR_FALSE; if (layout || (oldRect.width != aRect.width || oldRect.height != aRect.height)) { => return aBox->Layout(aState); } return NS_OK; nsGfxScrollFrameInner::LayoutBox(nsBoxLayoutState&,nsIBox *,nsRect const&) [nsGfxScrollFrame.cpp:1066] nsresult nsGfxScrollFrameInner::LayoutBox(nsBoxLayoutState& aState, nsIBox* aBox, const nsRect& aRect) => { return mOuter->LayoutChildAt(aState, aBox, aRect); } ??? [ip=0x0013a110] nsGfxScrollFrameInner::Layout(nsBoxLayoutState&) [nsGfxScrollFrame.cpp:1171] mScrollAreaBox->GetClientRect(oldScrollAreaBounds); // layout our the scroll area => LayoutBox(aState, mScrollAreaBox, scrollAreaRect); // now look at the content area and see if we need scrollbars or not PRBool needsLayout = PR_FALSE; nsGfxScrollFrame::DoLayout(nsBoxLayoutState&) [nsGfxScrollFrame.cpp:1075] { PRUint32 flags = 0; aState.GetLayoutFlags(flags); => nsresult rv = mInner->Layout(aState); aState.SetLayoutFlags(flags); nsBox::DoLayout(aState); nsBox::Layout(nsBoxLayoutState&) [nsBox.cpp:1050] { BeginLayout(aState); => DoLayout(aState); EndLayout(aState);
Who owns this now that evaughan has left? jag?
-> hyatt
Assignee: attinasi → hyatt
*** This bug has been marked as a duplicate of 68265 ***
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
verified dup
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.