From 6077de0645ce12a9c4e99f8839a846b42a535b0a Mon Sep 17 00:00:00 2001 From: Nick Clifton Date: Wed, 20 Jun 2018 16:30:05 +0100 Subject: Fix potential illegal memroy access when using a build-id note with a negative size. PR 23316 * opncls.c (get_build_id): Check for a negative or excessive data size in the build-id note. --- bfd/opncls.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'bfd/opncls.c') diff --git a/bfd/opncls.c b/bfd/opncls.c index 16b568c8ab2..e27504545cf 100644 --- a/bfd/opncls.c +++ b/bfd/opncls.c @@ -1877,10 +1877,11 @@ get_build_id (bfd *abfd) inote.descdata = inote.namedata + BFD_ALIGN (inote.namesz, 4); /* FIXME: Should we check for extra notes in this section ? */ - if (inote.descsz == 0 + if (inote.descsz <= 0 || inote.type != NT_GNU_BUILD_ID || inote.namesz != 4 /* sizeof "GNU" */ || strncmp (inote.namedata, "GNU", 4) != 0 + || inote.descsz > 0x7ffffffe || size < (12 + BFD_ALIGN (inote.namesz, 4) + inote.descsz)) { free (contents); -- cgit v1.2.3-65-gdbad