Crash due to CreateBackup() calling CopyToNative with nsnull

VERIFIED FIXED

Status

Core Graveyard
File Handling
--
critical
VERIFIED FIXED
16 years ago
a year ago

People

(Reporter: jesup, Assigned: Bill Law)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

16 years ago
Trunk build, fresh pull today, FreeBSD 4.3

nsSafeSaveFile::CreateBackup() calls CopyToNative with a 1st parameter of
nsnull.  That's "newParent" in io/nsLocalFileUnix.cpp's CopyToNative(), and
there's no check for a null newParent before using it.

Win32 (nsLocalFileWin.cpp) ends up checking for/handling a null newParent.

Crashes randomly and often!

Comment 1

16 years ago
I think bnesse backed himself out after tbox went orange.

Comment 2

16 years ago
I landed sfrasers patch for bug 164190 on the trunk. It was backed out shortly
thereafter due to orangeness. This is fixed.
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
rs vrfy.
Status: RESOLVED → VERIFIED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.