Re: [patch] bug in fs/isofs/inode.c (2.0.x & 2.1.x)

Heiko Ei\_feldt (heiko@colossus.escape.de)
Tue, 17 Nov 1998 11:00:41 +0100


Hi Cyrill and linux-kernel-hackers,

just read about your ISO-HS-patch, and I agree.
For this special case when both signatures (HS and ISO) are found,
we should favor ISO, and recognize High Sierra only, when no ISO
signature is found.
Since ISO came later, the case of having a HS cd, which
by chance has a valid ISO signature, is too improbable to be
encountered. In this really happens, explicit mount options should
be used.

Regarding the patch:
The comment
/* Just skip any volume descriptors we don't recognize */
should be moved behind the ISO detection. Otherwise it is fine.
And please add a comment, why this change had been done.

I agree with your fix. It should be included in the kernels.

ciao,
Heiko

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.rutgers.edu
Please read the FAQ at http://www.tux.org/lkml/