The attached file (gzip-compressed to save

Project:JNode Core
Category:bug report
Status:patch (code needs review)

The attached file (gzip-compressed to save space) is an 100MB NTFS image created on Windows XP.

Unless I'm using the API wrong, the following unit test should pass for this image:

public void testTrivial() throws Exception {
Device device = new FileDevice(getDataFile("diskimages/test.ntfs"), "r");
FSBlockDeviceAPI blockDeviceAPI = device.getAPI(FSBlockDeviceAPI.class);
byte[] firstSector = new byte[blockDeviceAPI.getSectorSize()];, ByteBuffer.wrap(firstSector));
assertTrue("NTFS driver should support an NTFS disk image",
new NTFSFileSystemType().supports(null, firstSector, blockDeviceAPI));

issues_1234.02 KB

Whoops. Left a bit of our

Whoops. Left a bit of our framework in that one. Smiling

You can replace this:

With this:
new File("test.ntfs")

Side effect ?


Are you sure this patch does'nt have any side effect on other filesystem types (Ext2, jfat, ...) detection.

I would like the advice of other developers before committing this patch (if it's possible)


Fabien L.

It's safe

I'm checking 8 characters worth of magic number instead of 4 characters worth now. So this patch actually decreases the chance of another filesystem mistakenly being identified as NTFS, while allowing for user error in creating the partition table (extremely common) and permitting for disk images which aren't an image of the entire disk.

Status check

Did this go in yet?


Status:active» patch (code needs review)

Here's a patch which is more forgiving -- allows the image to work as long as the magic number is present, ignoring anything related to partition tables.