diff options
| author | Aaron Durbin <adurbin@chromium.org> | 2014-03-13 08:52:17 -0500 |
|---|---|---|
| committer | Aaron Durbin <adurbin@google.com> | 2014-03-13 23:02:46 +0100 |
| commit | 19a11d6fe9593e585aff8a5787d25e38faa3d88c (patch) | |
| tree | 31d0f9584997ed0e0ba3b32dc211f5f5fb4651ee | |
| parent | ed2bcaa7310840073ac20f2087bd786d5874bd81 (diff) | |
| download | coreboot-19a11d6fe9593e585aff8a5787d25e38faa3d88c.tar.xz | |
cbfstool: remove incorrect section size check
I was overzealous in checking the section size with respect
to the file size. That check makes no sense as the section only
deals with link sizes -- not on-disk sizes. Remove the check as
it doesn't make any sense.
Change-Id: I348e7847ae3a50badc22693439614f813462445a
Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: http://review.coreboot.org/5384
Tested-by: build bot (Jenkins)
Reviewed-by: Marc Jones <marc.jones@se-eng.com>
| -rw-r--r-- | util/cbfstool/elfheaders.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/util/cbfstool/elfheaders.c b/util/cbfstool/elfheaders.c index 505af78aa9..03d8e7d68c 100644 --- a/util/cbfstool/elfheaders.c +++ b/util/cbfstool/elfheaders.c @@ -306,11 +306,6 @@ shdr_read(const struct buffer *in, Elf64_Ehdr *ehdr, struct xdr *xdr, int bit64) for (i = 0; i < ehdr->e_shnum; i++) { DEBUG("Parsing section %d\n", i); elf_shdr(&b, &shdr[i], ehdr->e_shentsize, xdr, bit64); - - /* Ensure the contents are valid within the elf file. */ - if (check_size(in, shdr[i].sh_offset, shdr[i].sh_size, - "section contents")) - return NULL; } return shdr; |