Closed Bug 201412 Opened 22 years ago Closed 22 years ago

Uninstaller crashes removing pathtoexe entry

Categories

(Core Graveyard :: Installer: GRE, defect, P1)

Product:
Core Graveyard
Component:
Installer: GRE
Platform:
x86
Windows 2000
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: carosendahl, Assigned: ssu0262)

Details

(Keywords: topembed+, Whiteboard: [adt2])

Attachments

(2 files, 3 obsolete files)

Test Case
1.03 KB, application/x-zip-compressed
Details
patch v1.3
8.36 KB, patch
jag+mozilla
: review+
jag+mozilla
: superreview+
jag+mozilla
: approval1.4+
Details | Diff | Splinter Review
1. Pull down two different versions of the GRE installer 2. Unzip the attached test case and set the environment variables correctly 3. Run setenv.bat and the 030GREInstall.bat expected result: C:\foo.txt pathtoexe entry is removed actual result: crash.
Keywords: nsbeta1, topembed
Priority: -- → P1
Attached file Test Case —
Discussed in edt bug triage. Plussing.
Keywords: topembed → topembed+
adt: nsbeta1+/adt2
Keywords: nsbeta1 → nsbeta1+
Whiteboard: [adt2]
Attached patch patch v1.0 (obsolete) — — Splinter Review
Attachment #123929 - Flags: review?(sgehani)
Attachment #123929 - Flags: review?(sgehani) → review-
Attached patch patch v1.1 (obsolete) — — Splinter Review
got r=sgehani for this patch.
Attachment #123929 - Attachment is obsolete: true
Comment on attachment 124030 [details] [diff] [review] patch v1.1 adding r=sgehani seeking rs=
Attachment #124030 - Flags: superreview?(jaggernaut)
Attachment #124030 - Flags: review+
Attachment #124030 - Flags: superreview?(jaggernaut) → superreview?(sspitzer)
Comment on attachment 124030 [details] [diff] [review] patch v1.1 rs=sspitzer a=sspitzer for 1.4 final sorry for the delay, ssu.
Attachment #124030 - Flags: superreview?(sspitzer)
Attachment #124030 - Flags: superreview+
Attachment #124030 - Flags: approval1.4+
Attached patch patch v1.2 (obsolete) — — Splinter Review
patch updated after talk with jag.
Attachment #124030 - Attachment is obsolete: true
Attached patch patch v1.3 — — Splinter Review
one minor change
Attachment #124045 - Attachment is obsolete: true
Comment on attachment 124046 [details] [diff] [review] patch v1.3 sr=jag, carrying forward r= and a= since nothing really changed.
Attachment #124046 - Flags: superreview+
Attachment #124046 - Flags: review+
Attachment #124046 - Flags: approval1.4+
patch was checked in on 05/23/03 resolving bug as fixed.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Verified using the 20030613 1.4 build. No longer crashes and correctly removes pathtoexe entries. Ran test case in the bug along with a couple of others related to this area.
Status: RESOLVED → VERIFIED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: