Ext2FileSystemType currently looks in the
| Project: | JNode Core |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Jump to:
Description
Ext2FileSystemType currently looks in the partition table and assumes that if the type is Linux (0x83), it must be Ext2.
Because of this, if the filesystem happens to be some *other* Linux filesystem (e.g. ReiserFS) the type is incorrectly identified.
The magic number check alone should be sufficient. If it isn't, more checks should be done based on the contents of the filesystem itself, rather than by trusting external metadata.
Attached patch fixes the problem.
| Attachment | Size |
|---|---|
| issues_39 | 1.24 KB |
- Login to post comments
#1
Thanks for the patch, I commited a modified version to keep the partition table check since I didn't want to change functionality at that point. We need to talk about these issues first.
#2