16312 - File op: fileRename of read-only file results in crash

Status: VERIFIED FIXED

(Core Graveyard :: Installer: XPInstall Engine, defect, P3)

Description

[Jimmy Lee]

Build: 10/12/99 SeaMonkey build

1. From your "Program" directory, create a subdirectory "filerename_other"
2. From "filerename_other", add any locked (read-only) text file named
   "readonly.txt"
2. Launch Seamonkey
3. From http://jimbob/trigger2.html, trigger
   http://jimbob/jars/f_fileop_filerename_readonly.xpi

jarSrc2 = "readonly.txt";

startInstall("Functional: f_fileop_filerename_readonly",
"f_fileop_filerename_readonly", "1.1.1.1", 0);
f = getFolder("Program", "filerename_other");
fileSource2 = f + jarSrc2;
fileTarget2 = "readonlyrenamed.txt";
err2 = fileRename(fileSource2, fileTarget2);
logComment("Path to fileSource2 = " + fileSource2);
logComment("fileRename returns = " + err2);

RESULT:
Crash after clicking OK button from XPInstall confirmation dialog.  Install.log
contains no information of this trigger.  I was able to get Windows to crash
before, but I cannot reproduce it anymore.

EXPECTED RESULT:
No crash.  File is not renamed.

NOTE:
From both Linux and Windows, it is possible to rename a read-only file from the
command line.  We are consistent.  The same cannot be said for Macintosh, so I
am expected similar behavior on behalf of XPInstall.

Comment 1

[Jimmy Lee]

Stack crawl is at http://jimbob/bugs/16312StdLog

Comment 2

[Sean Su]

it no longer crashes, and should return an error value instead.

Comment 3

[Jimmy Lee]

Build 1999-10-18-12-M11(MAC)

Error message is received as expected.  Marking Verified!

Comment 4

[leger]

Bulk move of XPInstall (component to be deleted) bugs to Installer: XPInstall
Engine