Closed Bug 120 Opened 27 years ago Closed 24 years ago

Navigator crashes if external viewer is registered with WWW_RegisterViewer

Categories

(MozillaClassic Graveyard :: Windows FE, defect, P3)

1998-03-31
x86
Windows NT
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: stanb, Assigned: blythe)

Details

Created by Stan Bobrovskiy (stanb@real.com) on Wednesday, April 8, 1998 12:08:13 PM PDT Additional Details : After configuring an external viewer with WWW_RegisterViewer Navigator 4.04 and 5.0 crashes when you click on a link that belongs to that registered viewer. The crash is - Access Violation, using a pointer without checking if it is not NULL. The same thing happens if using OLE to register external viewer. The reason of the crash is a new piece of code in ns\cmd\winfe\cxsave.cpp in CSaveCX::CanCreate() method: // If this file exists, then we must attempt another temp file. if(-1 != _access(pLocalName, 0)) { int type = NET_URL_Type(pUrl->address); // bug 63751 for Mail/News attachment pUrl here is NULL, so simple change fixes the cresh: // If this file exists, then we must attempt another temp file. if(-1 != _access(pLocalName, 0)) { if(pUrl) { int type = NET_URL_Type(pUrl->address); // bug 63751 for Mail/News attachment ... } Updated by Garrett Arch Blythe (blythe@netscape.com) on Monday, August 10, 1998 3:14:06 PM PDT Additional Details : Don't know who bugzilla@mozilla.org would be, or why they would desire to fix the bug. Taking the bug over as clearly a windows issue. Updated by Garrett Arch Blythe (blythe@netscape.com) on Monday, August 10, 1998 3:57:40 PM PDT Additional Details : A fix has been checked into the mozilla code base.
Status: RESOLVED → VERIFIED
Old bug....old code base...marking Verified.
mid-air collision ? / bugzilla cleanup Reopening (current State: verified and no resolution)
Status: VERIFIED → REOPENED
fixed
Status: REOPENED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
verified
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.