HFS+, HFSX: check the version field to reduce false positive

This commit is contained in:
Christophe Grenier 2009-01-10 15:51:30 +01:00
parent dbba3af27b
commit 38ce287026
2 changed files with 31 additions and 30 deletions

View file

@ -85,32 +85,29 @@ int recover_HFSP(disk_t *disk_car, const struct hfsp_vh *vh,partition_t *partiti
int test_HFSP(disk_t *disk_car, const struct hfsp_vh *vh,partition_t *partition,const int verbose, const int dump_ind) int test_HFSP(disk_t *disk_car, const struct hfsp_vh *vh,partition_t *partition,const int verbose, const int dump_ind)
{ {
if (vh->signature==be16(HFSP_VOLHEAD_SIG) && be32(vh->blocksize)%512==0 && be32(vh->blocksize)!=0 && be32(vh->free_blocks)<=be32(vh->total_blocks)) if (!(be32(vh->blocksize)%512==0 && be32(vh->blocksize)!=0 && be32(vh->free_blocks)<=be32(vh->total_blocks)))
return 1;
/* http://developer.apple.com/technotes/tn/tn1150.html */
if (be16(vh->version)==4 && vh->signature==be16(HFSP_VOLHEAD_SIG))
{ {
partition->upart_type=UP_HFSP;
if(verbose>0 || dump_ind!=0) if(verbose>0 || dump_ind!=0)
{ {
log_info("\nHFS+ magic value at %u/%u/%u\n", offset2cylinder(disk_car,partition->part_offset),offset2head(disk_car,partition->part_offset),offset2sector(disk_car,partition->part_offset)); log_info("\nHFS+ magic value at %u/%u/%u\n", offset2cylinder(disk_car,partition->part_offset),offset2head(disk_car,partition->part_offset),offset2sector(disk_car,partition->part_offset));
} }
if(dump_ind!=0)
{
/* There is a little offset ... */
dump_log(vh,DEFAULT_SECTOR_SIZE);
} }
if(verbose>1) else if (be16(vh->version)==5 && vh->signature==be16(HFSX_VOLHEAD_SIG))
{
log_info("blocksize %u\n",(unsigned) be32(vh->blocksize));
log_info("total_blocks %u\n",(unsigned) be32(vh->total_blocks));
log_info("free_blocks %u\n",(unsigned) be32(vh->free_blocks));
}
partition->upart_type=UP_HFSP;
return 0;
}
if (vh->signature==be16(HFSX_VOLHEAD_SIG) && be32(vh->blocksize)%512==0 && be32(vh->blocksize)!=0 && be32(vh->free_blocks)<=be32(vh->total_blocks))
{ {
partition->upart_type=UP_HFSX;
if(verbose>0 || dump_ind!=0) if(verbose>0 || dump_ind!=0)
{ {
log_info("\nHFSX magic value at %u/%u/%u\n", offset2cylinder(disk_car,partition->part_offset),offset2head(disk_car,partition->part_offset),offset2sector(disk_car,partition->part_offset)); log_info("\nHFSX magic value at %u/%u/%u\n", offset2cylinder(disk_car,partition->part_offset),offset2head(disk_car,partition->part_offset),offset2sector(disk_car,partition->part_offset));
} }
}
else
{
return 1;
}
if(dump_ind!=0) if(dump_ind!=0)
{ {
/* There is a little offset ... */ /* There is a little offset ... */
@ -122,10 +119,7 @@ int test_HFSP(disk_t *disk_car, const struct hfsp_vh *vh,partition_t *partition,
log_info("total_blocks %u\n",(unsigned) be32(vh->total_blocks)); log_info("total_blocks %u\n",(unsigned) be32(vh->total_blocks));
log_info("free_blocks %u\n",(unsigned) be32(vh->free_blocks)); log_info("free_blocks %u\n",(unsigned) be32(vh->free_blocks));
} }
partition->upart_type=UP_HFSX;
return 0; return 0;
}
return 1;
} }
static int set_HFSP_info(partition_t *partition, const struct hfsp_vh *vh) static int set_HFSP_info(partition_t *partition, const struct hfsp_vh *vh)

View file

@ -25,6 +25,10 @@
*/ */
#ifndef _HFSP_H #ifndef _HFSP_H
#define _HFSP_H #define _HFSP_H
#ifdef __cplusplus
extern "C" {
#endif
#define HFSP_BOOT_SECTOR_SIZE 512 #define HFSP_BOOT_SECTOR_SIZE 512
#define HFSP_BLOCKSZ 512 /* A sector for Apple is always 512 bytes */ #define HFSP_BLOCKSZ 512 /* A sector for Apple is always 512 bytes */
#define HFSP_BLOCKSZ_BITS 9 /* 1<<9 == 512 */ #define HFSP_BLOCKSZ_BITS 9 /* 1<<9 == 512 */
@ -77,7 +81,7 @@ typedef struct {
*/ */
typedef struct hfsp_vh { typedef struct hfsp_vh {
uint16_t signature; // must be HFSPLUS_VOLHEAD_SIG 'H+' uint16_t signature; // must be HFSPLUS_VOLHEAD_SIG 'H+'
uint16_t version; // currently 4, ignored uint16_t version; // 4 for HFS+, 5 for HFSX
uint32_t attributes; // See bit constants below uint32_t attributes; // See bit constants below
uint32_t last_mount_vers; uint32_t last_mount_vers;
// Use a registered creator code here (See libhfsp.h) // Use a registered creator code here (See libhfsp.h)
@ -148,4 +152,7 @@ int check_HFSP(disk_t *disk_car,partition_t *partition,const int verbose);
int test_HFSP(disk_t *disk_car, const struct hfsp_vh *vh,partition_t *partition,const int verbose, const int dump_ind); int test_HFSP(disk_t *disk_car, const struct hfsp_vh *vh,partition_t *partition,const int verbose, const int dump_ind);
int recover_HFSP(disk_t *disk_car, const struct hfsp_vh *vh,partition_t *partition,const int verbose, const int dump_ind, const int backup); int recover_HFSP(disk_t *disk_car, const struct hfsp_vh *vh,partition_t *partition,const int verbose, const int dump_ind, const int backup);
#ifdef __cplusplus
} /* closing brace for extern "C" */
#endif
#endif #endif