diff options
| author | Yuriy M. Kaminskiy <yumkam@gmail.com> | 2015-08-04 16:51:53 +0100 |
|---|---|---|
| committer | Nick Clifton <nickc@redhat.com> | 2015-08-04 16:51:53 +0100 |
| commit | 7e27a9d5f22f9f7ead11738b1546d0b5c737266b (patch) | |
| tree | 46bffdfecd821e8f439790cab4da495658777807 /bfd/srec.c | |
| parent | 4e13f8fb05eb7ffd163d96e519cc011d8d21e3d7 (diff) | |
Fix stack buffer overflows when parsing corrupt ihex files.
PR binutils/18750
* ihex.c (ihex_scan): Fixes incorrect escape sequence in error message
and stack overflow when char is signed and \200-\376 was in place of hex
digit; also fixes \377 was handled as EOF instead of "incorrect character".
(ihex_read_section): Changed for consistency.
(ihex_bad_byte): Prevent (now impossible to trigger) stack
overflow and incorrect escape sequence handling.
* srec.c (srec_bad_byte): Likewise.
* readelf.c (process_mips_specific): Fix incorrect escape
sequence handling.
Diffstat (limited to 'bfd/srec.c')
| -rw-r--r-- | bfd/srec.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/bfd/srec.c b/bfd/srec.c index 24573cff61..96b6a2f084 100644 --- a/bfd/srec.c +++ b/bfd/srec.c @@ -249,7 +249,7 @@ srec_bad_byte (bfd *abfd, char buf[40]; if (! ISPRINT (c)) - sprintf (buf, "\\%03o", (unsigned int) c); + sprintf (buf, "\\%03o", (unsigned int) c & 0xff); else { buf[0] = c; |