163 - [PP]NSPR: pathnames that start with /dev, /bin, /etc are not allowed on the Mac

Status: VERIFIED FIXED

(NSPR :: NSPR, defect, P2)

Description

[Hisakuni Fujimoto]

Created by Hisakuni Fujimoto ([e-mail]) on Monday, April 13, 1998 12:07:40 AM PDT
Additional Details :
In function ConvertUnixPathToMacPath() of ./ns/nsprpub/pr/
src/md/mac/macio.c.

both 03-31, 04-08 version.

If one of HDD volumes name starts with some specified
strings (dev, bin, etc) , display debug message "Unable to
translate Unix file path %s to Mac path" and immediately
exit.

patch for my Macintosh environment only bellow...

--- macio.c.orig	Mon Mar 30 18:15:06 1998
+++ macio.c	Thu Apr  9 16:42:46 1998
@@ -1258,3 +1258,3 @@

-		else if (strncmp(unixPath, "/dev", strlen("/dev")) =
= 0) {
+		else if (strcmp(unixPath, "/dev") == 0) {
 			dprintf("Unable to translate Unix file path %s
to Mac path\n", unixPath);
@@ -1731,3 +1731,3 @@

-		else if (strncmp(unixPath, "/dev", strlen("/dev")) =
= 0) {
+		else if (strcmp(unixPath, "/dev") == 0) {
 			dprintf("Unable to translate Unix file path %s
to Mac path\n", unixPath);


Updated by Mike Pinkerton ([e-mail]) on Monday, April 13, 1998 12:20:40 PM PDT
Additional Details :
The fix we are implementing is to remove the lines that check for /dev /bin /etc
because they are pointless. I should be able to have a volume called "dev" if I
want to.
Updated by Wan-Teh Chang ([e-mail]) on Friday, April 24, 1998 12:49:58 PM PDT
Additional Details :
Assigned bug to myself.

This has been fixed in ns/nsprpub/pr/src/md/mac/macio.c,
revision 3.2, as Mike Pinkerton outlined above.

Closed the bug.
Updated by Wan-Teh Chang ([e-mail]) on Friday, April 24, 1998 12:52:20 PM PDT
Additional Details :
Made the "Summary Description" more descriptive.

Comment 1

[Terry Weissman]

NSPR now has its own Bugzilla product.  Moving this bug to the NSPR product.

Comment 2

[Jesse Ruderman]

[mass change] these bugs are closed but have no resolution.  reopening...

Comment 3

[Jesse Ruderman]

marking as fixed.

Comment 4

[Brant Gurganus]

VERIFIED due to no changes in over a year