Don't keep copy of backup header

The primary and backup headers may be fully reconstructed from each other
(this is the purpose of having a backup header in the first place).  So
there is no point in saving a copy of the backup header.
This commit is contained in:
Tom Marshall 2015-04-18 09:39:27 -07:00
parent 58c4e79708
commit 9df07a3ec1
2 changed files with 92 additions and 114 deletions

203
gpt.c
View File

@ -143,13 +143,12 @@ int gpt_open(struct gpt *gpt, const char *pathname)
uint64_t next_lba; uint64_t next_lba;
uint32_t n; uint32_t n;
struct stat st; struct stat st;
int primary_header_valid = 0; struct gpt_header backup_header;
int backup_header_valid = 0;
memset(gpt, 0, sizeof(struct gpt)); memset(gpt, 0, sizeof(struct gpt));
gpt->pathname = strdup(pathname); gpt->pathname = strdup(pathname);
fd = open(gpt->pathname, O_RDNOLY); fd = open(gpt->pathname, O_RDONLY);
if (fd < 0) { if (fd < 0) {
perror("open"); perror("open");
return -1; return -1;
@ -199,7 +198,12 @@ int gpt_open(struct gpt *gpt, const char *pathname)
gpt->header.ptbl_count > GPT_MAX_PARTITIONS || gpt->header.ptbl_count > GPT_MAX_PARTITIONS ||
gpt->header.ptbl_entry_size > gpt->lbsize) { gpt->header.ptbl_entry_size > gpt->lbsize) {
fprintf(stderr, "W: bad primary gpt\n"); fprintf(stderr, "W: bad primary gpt\n");
goto primary_header_out; }
gpt->pad_idx = gpt_part_find(gpt, "pad");
if (gpt->pad_idx == GPT_PART_INVALID) {
fprintf(stderr, "no pad found\n");
close(fd);
return -1;
} }
#else #else
if (gpt->header.current_lba != 1 || if (gpt->header.current_lba != 1 ||
@ -213,142 +217,121 @@ int gpt_open(struct gpt *gpt, const char *pathname)
gpt->header.ptbl_count > GPT_MAX_PARTITIONS || gpt->header.ptbl_count > GPT_MAX_PARTITIONS ||
gpt->header.ptbl_entry_size > gpt->lbsize) { gpt->header.ptbl_entry_size > gpt->lbsize) {
fprintf(stderr, "W: bad primary gpt\n"); fprintf(stderr, "W: bad primary gpt\n");
goto primary_header_out;
} }
#endif #endif
primary_header_valid = 1;
primary_header_out:
/* Validate backup GPT for block devices */ next_lba = gpt->header.first_usable_lba;
if (S_ISBLK(st.st_mode) &&
gpt->header.backup_lba > 2 && off = lseek64(fd, gpt->header.ptbl_lba * gpt->lbsize, SEEK_SET);
if (off < 0) {
perror("lseek\n");
close(fd);
return -1;
}
calc_crc = 0;
for (n = 0; n < gpt->header.ptbl_count; ++n) {
gpt->partitions[n] = (struct gpt_partition *)malloc(sizeof(struct gpt_partition));
memset(gpt->partitions[n], 0, sizeof(struct gpt_partition));
rc = read(fd, gpt->partitions[n], gpt->header.ptbl_entry_size);
if (rc < 0 || (uint32_t)rc != gpt->header.ptbl_entry_size) {
fprintf(stderr, "failed to read partition entry %u\n", n);
close(fd);
return -1;
}
calc_crc = crc32(calc_crc, gpt->partitions[n], gpt->header.ptbl_entry_size);
if (gpt->partitions[n]->first_lba == 0 && gpt->partitions[n]->last_lba == 0) {
continue;
}
if (gpt->partitions[n]->first_lba < next_lba ||
gpt->partitions[n]->last_lba < gpt->partitions[n]->first_lba ||
gpt->partitions[n]->last_lba > gpt->header.last_usable_lba) {
fprintf(stderr, "bad lba in partition entry %u\n", n);
close(fd);
return -1;
}
gpt->last_used_idx = n;
}
if (gpt->header.ptbl_crc != calc_crc) {
fprintf(stderr, "W: bad ptbl crc\n");
}
/* Validate backup GPT */
if (gpt->header.backup_lba > 2 &&
gpt->header.backup_lba < gpt->lblen) { gpt->header.backup_lba < gpt->lblen) {
off = lseek64(fd, gpt->header.backup_lba * gpt->lbsize, SEEK_SET); off = lseek64(fd, gpt->header.backup_lba * gpt->lbsize, SEEK_SET);
if (off < 0) { if (off < 0) {
fprintf(stderr, "bad backup seek\n"); fprintf(stderr, "bad backup seek\n");
goto backup_header_out; goto out;
} }
rc = read(fd, buf, gpt->lbsize); rc = read(fd, buf, gpt->lbsize);
if (rc != (ssize_t)gpt->lbsize) { if (rc != (ssize_t)gpt->lbsize) {
fprintf(stderr, "bad backup read\n"); fprintf(stderr, "bad backup read\n");
goto backup_header_out; goto out;
} }
memcpy(&gpt->backup_header, buf, sizeof(struct gpt_header)); memcpy(&backup_header, buf, sizeof(struct gpt_header));
if (gpt_header_is_valid(&gpt->backup_header, gpt->lbsize) != 0) { if (gpt_header_is_valid(&backup_header, gpt->lbsize) != 0) {
fprintf(stderr, "bad backup header\n"); fprintf(stderr, "bad backup header\n");
goto backup_header_out;
} }
#if defined(ANDROID) && defined(QCOM) #if defined(ANDROID) && defined(QCOM)
if (gpt->backup_header.current_lba != gpt->header.backup_lba || if (backup_header.current_lba != gpt->header.backup_lba ||
gpt->backup_header.backup_lba != 1 || backup_header.backup_lba != 1 ||
gpt->backup_header.first_usable_lba != gpt->header.first_usable_lba || backup_header.first_usable_lba != gpt->header.first_usable_lba ||
gpt->backup_header.last_usable_lba != gpt->header.last_usable_lba || backup_header.last_usable_lba != gpt->header.last_usable_lba ||
memcmp(gpt->backup_header.disk_guid, gpt->header.disk_guid, 16) != 0 || memcmp(backup_header.disk_guid, gpt->header.disk_guid, 16) != 0 ||
gpt->backup_header.ptbl_lba >= gpt->lblen || backup_header.ptbl_lba >= gpt->lblen ||
gpt->backup_header.ptbl_count != gpt->header.ptbl_count || backup_header.ptbl_count != gpt->header.ptbl_count ||
gpt->backup_header.ptbl_entry_size != gpt->header.ptbl_entry_size) { backup_header.ptbl_entry_size != gpt->header.ptbl_entry_size) {
fprintf(stderr, "W: bad backup gpt\n"); fprintf(stderr, "W: bad backup gpt\n");
goto backup_header_out; goto out;
} }
#else #else
if (gpt->backup_header.current_lba != gpt->header.backup_lba || if (backup_header.current_lba != gpt->header.backup_lba ||
gpt->backup_header.backup_lba != 1 || backup_header.backup_lba != 1 ||
gpt->backup_header.first_usable_lba != gpt->header.first_usable_lba || backup_header.first_usable_lba != gpt->header.first_usable_lba ||
gpt->backup_header.last_usable_lba != gpt->header.last_usable_lba || backup_header.last_usable_lba != gpt->header.last_usable_lba ||
memcmp(gpt->backup_header.disk_guid, gpt->header.disk_guid, 16) != 0 || memcmp(backup_header.disk_guid, gpt->header.disk_guid, 16) != 0 ||
gpt->backup_header.ptbl_lba >= gpt->lblen || backup_header.ptbl_lba >= gpt->lblen ||
gpt->backup_header.ptbl_count != gpt->header.ptbl_count || backup_header.ptbl_count != gpt->header.ptbl_count ||
gpt->backup_header.ptbl_entry_size != gpt->header.ptbl_entry_size) { backup_header.ptbl_entry_size != gpt->header.ptbl_entry_size) {
fprintf(stderr, "W: bad backup gpt\n"); fprintf(stderr, "W: bad backup gpt\n");
goto backup_header_out; goto out;
} }
#endif #endif
backup_header_valid = 1;
}
backup_header_out:
next_lba = gpt->header.first_usable_lba;
if (primary_header_valid) {
off = lseek64(fd, gpt->header.ptbl_lba * gpt->lbsize, SEEK_SET);
if (off < 0) {
perror("lseek\n");
close(fd);
return -1;
}
calc_crc = 0;
for (n = 0; n < gpt->header.ptbl_count; ++n) {
gpt->partitions[n] = (struct gpt_partition *)malloc(sizeof(struct gpt_partition));
memset(gpt->partitions[n], 0, sizeof(struct gpt_partition));
rc = read(fd, gpt->partitions[n], gpt->header.ptbl_entry_size);
if (rc < 0 || (uint32_t)rc != gpt->header.ptbl_entry_size) {
fprintf(stderr, "failed to read partition entry %u\n", n);
close(fd);
return -1;
}
calc_crc = crc32(calc_crc, gpt->partitions[n], gpt->header.ptbl_entry_size);
if (gpt->partitions[n]->first_lba == 0 && gpt->partitions[n]->last_lba == 0) {
continue;
}
if (gpt->partitions[n]->first_lba < next_lba ||
gpt->partitions[n]->last_lba < gpt->partitions[n]->first_lba ||
gpt->partitions[n]->last_lba > gpt->header.last_usable_lba) {
fprintf(stderr, "bad lba in partition entry %u\n", n);
close(fd);
return -1;
}
gpt->last_used_idx = n;
}
if (gpt->header.ptbl_crc != calc_crc) {
fprintf(stderr, "bad ptbl crc\n");
close(fd);
return -1;
}
} }
if (backup_header_valid) { int warned = 0;
int warned = 0; off = lseek64(fd, backup_header.ptbl_lba * gpt->lbsize, SEEK_SET);
off = lseek64(fd, gpt->backup_header.ptbl_lba * gpt->lbsize, SEEK_SET); if (off < 0) {
if (off < 0) { perror("lseek\n");
perror("lseek\n");
close(fd);
return -1;
}
calc_crc = 0;
for (n = 0; n < gpt->backup_header.ptbl_count; ++n) {
struct gpt_partition backup_part;
rc = read(fd, &backup_part, gpt->backup_header.ptbl_entry_size);
if (rc < 0 || (uint32_t)rc != gpt->backup_header.ptbl_entry_size) {
fprintf(stderr, "failed to read backup partition entry %u\n", n);
close(fd);
return -1;
}
if (memcmp(gpt->partitions[n], &backup_part, sizeof(struct gpt_partition)) != 0) {
if (!warned) {
fprintf(stderr, "mismatched backup partition entry %u\n", n);
warned = 1;
}
}
}
}
#if defined(ANDROID) && defined(QCOM)
gpt->pad_idx = gpt_part_find(gpt, "pad");
if (gpt->pad_idx == GPT_PART_INVALID) {
fprintf(stderr, "no pad found\n");
close(fd); close(fd);
return -1; return -1;
} }
#endif calc_crc = 0;
for (n = 0; n < backup_header.ptbl_count; ++n) {
struct gpt_partition backup_part;
rc = read(fd, &backup_part, backup_header.ptbl_entry_size);
if (rc < 0 || (uint32_t)rc != backup_header.ptbl_entry_size) {
fprintf(stderr, "failed to read backup partition entry %u\n", n);
close(fd);
return -1;
}
if (memcmp(gpt->partitions[n], &backup_part, sizeof(struct gpt_partition)) != 0) {
if (!warned) {
fprintf(stderr, "mismatched backup partition entry %u\n", n);
warned = 1;
}
}
}
out:
close(fd); close(fd);
return 0; return 0;
} }
int gpt_write(const struct gpt *gpt) int gpt_write(struct gpt *gpt)
{ {
int fd; int fd;
int rc; int rc;
@ -461,10 +444,6 @@ void gpt_show(const struct gpt *gpt)
gpt_header_show("Primary GPT", &gpt->header); gpt_header_show("Primary GPT", &gpt->header);
if (gpt->backup_header.size != 0) {
gpt_header_show("Backup GPT", &gpt->backup_header);
}
printf("Partition table: count=%u\n", (unsigned int)gpt->last_used_idx); printf("Partition table: count=%u\n", (unsigned int)gpt->last_used_idx);
for (n = 0; n < gpt->header.ptbl_count; ++n) { for (n = 0; n < gpt->header.ptbl_count; ++n) {
if (gpt->partitions[n]->first_lba == 0 && gpt->partitions[n]->last_lba == 0) { if (gpt->partitions[n]->first_lba == 0 && gpt->partitions[n]->last_lba == 0) {

3
gpt.h
View File

@ -51,7 +51,6 @@ struct gpt
uint32_t lbsize; uint32_t lbsize;
uint32_t lblen; uint32_t lblen;
struct gpt_header header; struct gpt_header header;
struct gpt_header backup_header;
#if defined(ANDROID) && defined(QCOM) #if defined(ANDROID) && defined(QCOM)
uint32_t pad_idx; uint32_t pad_idx;
#endif #endif
@ -60,7 +59,7 @@ struct gpt
}; };
int gpt_open(struct gpt *gpt, const char *pathname); int gpt_open(struct gpt *gpt, const char *pathname);
int gpt_write(const struct gpt *gpt); int gpt_write(struct gpt *gpt);
int gpt_close(struct gpt *gpt); int gpt_close(struct gpt *gpt);
void gpt_show(const struct gpt *gpt); void gpt_show(const struct gpt *gpt);