vbatts/sbsigntools
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

image: warn about potential checksum differences

Browse source 
Signed-off-by: Jeremy Kerr <jeremy.kerr@canonical.com>
This commit is contained in:
Jeremy Kerr 2012-04-23 17:42:45 +08:00
parent d8eadfcc24
commit da5568e8ff
1 changed files with 14 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
image.c
View file

@ -173,10 +173,11 @@ static void set_region_from_range(struct region *region, void *start, void *end)
int image_find_regions(struct image *image) int image_find_regions(struct image *image)
{ {
struct region *regions; struct region *regions;
int i, gap_warn;
uint32_t align; uint32_t align;
size_t bytes; size_t bytes;
int i;
gap_warn = 0;
align = *(uint32_t *)image->aouthdr->FileAlignment; align = *(uint32_t *)image->aouthdr->FileAlignment;
/* now we know where the checksum and cert table data is, we can /* now we know where the checksum and cert table data is, we can
@ -238,8 +239,20 @@ int image_find_regions(struct image *image)
regions[i + 3].name = talloc_strndup(image->checksum_regions, regions[i + 3].name = talloc_strndup(image->checksum_regions,
image->scnhdr[i].s_name, 8); image->scnhdr[i].s_name, 8);
bytes += regions[i + 3].size; bytes += regions[i + 3].size;
if (regions[i+2].data + regions[i+2].size
!= regions[i+3].data) {
fprintf(stderr, "warning: gap in section table between "
"%s and %s\n",
regions[i+2].name, regions[i+3].name);
gap_warn = 1;
}
} }
if (gap_warn)
fprintf(stderr, "gaps in the section table may result in "
"different checksums\n");
if (bytes != image->size) { if (bytes != image->size) {
fprintf(stderr, "warning: data remaining[%zd vs %zd]: gaps " fprintf(stderr, "warning: data remaining[%zd vs %zd]: gaps "
"between PE/COFF sections?\n", "between PE/COFF sections?\n",