From bbe6d95f460afad7a02a4828fee94a8bf0a2ba63 Mon Sep 17 00:00:00 2001 From: Alan Modra Date: Mon, 19 Mar 2001 12:20:10 +0000 Subject: [PATCH] Don't try to read past end of info buffer, and correct test results. --- gas/testsuite/ChangeLog | 4 ++++ gas/testsuite/gas/vax/quad.exp | 2 +- opcodes/ChangeLog | 5 +++++ opcodes/vax-dis.c | 15 +++++++++++++-- 4 files changed, 23 insertions(+), 3 deletions(-) diff --git a/gas/testsuite/ChangeLog b/gas/testsuite/ChangeLog index bdde2a37878..5420f9d9ddf 100644 --- a/gas/testsuite/ChangeLog +++ b/gas/testsuite/ChangeLog @@ -1,3 +1,7 @@ +2001-03-19 John David Anglin + + * gas/vax/quad.exp: Correct expected result. + 2001-03-19 Andreas Schwab * gas/m68k/pcrel.s: Force alignment to 8 byte boundary to get diff --git a/gas/testsuite/gas/vax/quad.exp b/gas/testsuite/gas/vax/quad.exp index 34770c5b6cf..46fc9ca52d4 100644 --- a/gas/testsuite/gas/vax/quad.exp +++ b/gas/testsuite/gas/vax/quad.exp @@ -8,7 +8,7 @@ proc do_quad {} { expect { -re "^ +2\[ \t\]+0000+ 7D8F7856\[ \t\]+movq\[^\n\]*\n" { set x1 1 } -re "^ +2\[ \t\]+3412DDCC\[^\n\]*\n" { set x2 1 } - -re "^ +2\[ \t\]+BBAA5001\[ \t\]*\r\n" { set x3 1 } + -re "^ +2\[ \t\]+BBAA50\[ \t\]*\r\n" { set x3 1 } -re "\[^\n\]*\n" { } timeout { perror "timeout\n"; break } eof { break } diff --git a/opcodes/ChangeLog b/opcodes/ChangeLog index 38f7fdfd80a..32185d83946 100644 --- a/opcodes/ChangeLog +++ b/opcodes/ChangeLog @@ -1,3 +1,8 @@ +2001-03-19 John David Anglin + + * vax-dis.c (print_insn_vax): Only fetch two bytes if the info buffer + has more than one byte left to read. + 2001-03-16 Martin Schwidefsky * s390-opc.c: Add new opcodes. Smooth out formatting. diff --git a/opcodes/vax-dis.c b/opcodes/vax-dis.c index d6cb9001f51..e33f87e5e68 100644 --- a/opcodes/vax-dis.c +++ b/opcodes/vax-dis.c @@ -1,5 +1,5 @@ /* Print VAX instructions. - Copyright 1995, 1998, 2000 Free Software Foundation, Inc. + Copyright 1995, 1998, 2000, 2001 Free Software Foundation, Inc. Contributed by Pauline Middelink This program is free software; you can redistribute it and/or modify @@ -126,7 +126,18 @@ print_insn_vax (memaddr, info) return -1; } - FETCH_DATA (info, buffer + 2); + /* Check if the info buffer has more than one byte left since + the last opcode might be a single byte with no argument data. */ + if (info->buffer_length - (memaddr - info->buffer_vma) > 1) + { + FETCH_DATA (info, buffer + 2); + } + else + { + FETCH_DATA (info, buffer + 1); + buffer[1] = 0; + } + for (votp = &votstrs[0]; votp->name[0]; votp++) { register vax_opcodeT opcode = votp->detail.code; -- 2.30.2