summaryrefslogtreecommitdiff
path: root/util/cbfstool
diff options
context:
space:
mode:
authorAaron Durbin <adurbin@chromium.org>2014-03-13 08:52:17 -0500
committerAaron Durbin <adurbin@google.com>2014-03-13 23:02:46 +0100
commit19a11d6fe9593e585aff8a5787d25e38faa3d88c (patch)
tree31d0f9584997ed0e0ba3b32dc211f5f5fb4651ee /util/cbfstool
parented2bcaa7310840073ac20f2087bd786d5874bd81 (diff)
downloadcoreboot-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>
Diffstat (limited to 'util/cbfstool')
-rw-r--r--util/cbfstool/elfheaders.c5
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;