More signed overflow fixes
[deliverable/binutils-gdb.git] / opcodes / mcore-dis.c
index c6183933d66dc3f6545225567156881d2cd3ce6b..5b3acb80e2804c18417af78d71c593afd534e0c7 100644 (file)
@@ -1,27 +1,31 @@
 /* Disassemble Motorola M*Core instructions.
-   Copyright 1993, 1999, 2000, 2001, 2002 Free Software Foundation, Inc.
+   Copyright (C) 1993-2019 Free Software Foundation, Inc.
 
-This program is free software; you can redistribute it and/or modify
-it under the terms of the GNU General Public License as published by
-the Free Software Foundation; either version 2 of the License, or
-(at your option) any later version.
+   This file is part of the GNU opcodes library.
 
-This program is distributed in the hope that it will be useful,
-but WITHOUT ANY WARRANTY; without even the implied warranty of
-MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-GNU General Public License for more details.
+   This library is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 3, or (at your option)
+   any later version.
 
-You should have received a copy of the GNU General Public License
-along with this program; if not, write to the Free Software
-Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
+   It is distributed in the hope that it will be useful, but WITHOUT
+   ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
+   or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public
+   License for more details.
+
+   You should have received a copy of the GNU General Public License
+   along with this program; if not, write to the Free Software
+   Foundation, Inc., 51 Franklin Street - Fifth Floor, Boston,
+   MA 02110-1301, USA.  */
 
 #include "sysdep.h"
 #include <stdio.h>
+#include "libiberty.h"
 #define STATIC_TABLE
 #define DEFINE_TABLE
 
 #include "mcore-opc.h"
-#include "dis-asm.h"
+#include "disassemble.h"
 
 /* Mask for each mcore_opclass: */
 static const unsigned short imsk[] = {
@@ -85,15 +89,14 @@ static const char *crname[] = {
 static const unsigned isiz[] = { 2, 0, 1, 0 };
 
 int
-print_insn_mcore (memaddr, info)
-     bfd_vma memaddr;
-     struct disassemble_info *info;
+print_insn_mcore (bfd_vma memaddr,
+                 struct disassemble_info *info)
 {
   unsigned char ibytes[4];
-  fprintf_ftype fprintf = info->fprintf_func;
+  fprintf_ftype print_func = info->fprintf_func;
   void *stream = info->stream;
   unsigned short inst;
-  const mcore_opcode_info *op;
+  unsigned int i;
   int status;
 
   info->bytes_per_chunk = 2;
@@ -114,60 +117,60 @@ print_insn_mcore (memaddr, info)
     abort ();
 
   /* Just a linear search of the table.  */
-  for (op = mcore_table; op->name != 0; op++)
-    if (op->inst == (inst & imsk[op->opclass]))
+  for (i = 0; i < ARRAY_SIZE (mcore_table); i++)
+    if (mcore_table[i].inst == (inst & imsk[mcore_table[i].opclass]))
       break;
 
-  if (op->name == 0)
-    fprintf (stream, ".short 0x%04x", inst);
+  if (i == ARRAY_SIZE (mcore_table))
+    (*print_func) (stream, ".short 0x%04x", inst);
   else
     {
       const char *name = grname[inst & 0x0F];
 
-      fprintf (stream, "%s", op->name);
+      (*print_func) (stream, "%s", mcore_table[i].name);
 
-      switch (op->opclass)
+      switch (mcore_table[i].opclass)
        {
        case O0:
          break;
 
        case OT:
-         fprintf (stream, "\t%d", inst & 0x3);
+         (*print_func) (stream, "\t%d", inst & 0x3);
          break;
 
        case O1:
        case JMP:
        case JSR:
-         fprintf (stream, "\t%s", name);
+         (*print_func) (stream, "\t%s", name);
          break;
 
        case OC:
-         fprintf (stream, "\t%s, %s", name, crname[(inst >> 4) & 0x1F]);
+         (*print_func) (stream, "\t%s, %s", name, crname[(inst >> 4) & 0x1F]);
          break;
 
        case O1R1:
-         fprintf (stream, "\t%s, r1", name);
+         (*print_func) (stream, "\t%s, r1", name);
          break;
 
        case MULSH:
        case O2:
-         fprintf (stream, "\t%s, %s", name, grname[(inst >> 4) & 0xF]);
+         (*print_func) (stream, "\t%s, %s", name, grname[(inst >> 4) & 0xF]);
          break;
 
        case X1:
-         fprintf (stream, "\tr1, %s", name);
+         (*print_func) (stream, "\tr1, %s", name);
          break;
 
        case OI:
-         fprintf (stream, "\t%s, %d", name, ((inst >> 4) & 0x1F) + 1);
+         (*print_func) (stream, "\t%s, %d", name, ((inst >> 4) & 0x1F) + 1);
          break;
 
        case RM:
-         fprintf (stream, "\t%s-r15, (r0)", name);
+         (*print_func) (stream, "\t%s-r15, (r0)", name);
          break;
 
        case RQ:
-         fprintf (stream, "\tr4-r7, (%s)", name);
+         (*print_func) (stream, "\tr4-r7, (%s)", name);
          break;
 
        case OB:
@@ -179,35 +182,31 @@ print_insn_mcore (memaddr, info)
        case OMa:
        case OMb:
        case OMc:
-         fprintf (stream, "\t%s, %d", name, (inst >> 4) & 0x1F);
+         (*print_func) (stream, "\t%s, %d", name, (inst >> 4) & 0x1F);
          break;
 
        case I7:
-         fprintf (stream, "\t%s, %d", name, (inst >> 4) & 0x7F);
+         (*print_func) (stream, "\t%s, %d", name, (inst >> 4) & 0x7F);
          break;
 
        case LS:
-         fprintf (stream, "\t%s, (%s, %d)", grname[(inst >> 8) & 0xF],
-                  name, ((inst >> 4) & 0xF) << isiz[(inst >> 13) & 3]);
+         (*print_func) (stream, "\t%s, (%s, %d)", grname[(inst >> 8) & 0xF],
+                        name, ((inst >> 4) & 0xF) << isiz[(inst >> 13) & 3]);
          break;
 
        case BR:
          {
-           long val = inst & 0x3FF;
-
-           if (inst & 0x400)
-             val |= 0xFFFFFC00;
+           uint32_t val = ((inst & 0x3FF) ^ 0x400) - 0x400;
 
-           fprintf (stream, "\t0x%x", memaddr + 2 + (val << 1));
+           val = memaddr + 2 + (val << 1);
+           (*print_func) (stream, "\t0x%x", val);
 
-           if (strcmp (op->name, "bsr") == 0)
+           if (strcmp (mcore_table[i].name, "bsr") == 0)
              {
                /* For bsr, we'll try to get a symbol for the target.  */
-               val = memaddr + 2 + (val << 1);
-
                if (info->print_address_func && val != 0)
                  {
-                   fprintf (stream, "\t// ");
+                   (*print_func) (stream, "\t// ");
                    info->print_address_func (val, info);
                  }
              }
@@ -216,19 +215,22 @@ print_insn_mcore (memaddr, info)
 
        case BL:
          {
-           long val;
-           val = (inst & 0x000F);
-           fprintf (stream, "\t%s, 0x%x",
-                    grname[(inst >> 4) & 0xF], memaddr - (val << 1));
+           uint32_t val = inst & 0x000F;
+           (*print_func) (stream, "\t%s, 0x%x",
+                          grname[(inst >> 4) & 0xF],
+                          (uint32_t) (memaddr - (val << 1)));
          }
          break;
 
        case LR:
          {
-           unsigned long val;
+           uint32_t val;
 
-           val = (memaddr + 2 + ((inst & 0xFF) << 2)) & 0xFFFFFFFC;
+           val = (memaddr + 2 + ((inst & 0xFF) << 2)) & ~3;
 
+           /* We are not reading an instruction, so allow
+              reads to extend beyond the next symbol.  */
+           info->stop_vma = 0;
            status = info->read_memory_func (val, ibytes, 4, info);
            if (status != 0)
              {
@@ -237,27 +239,31 @@ print_insn_mcore (memaddr, info)
              }
 
            if (info->endian == BFD_ENDIAN_LITTLE)
-             val = (ibytes[3] << 24) | (ibytes[2] << 16)
-               | (ibytes[1] << 8) | (ibytes[0]);
+             val = (((unsigned) ibytes[3] << 24) | (ibytes[2] << 16)
+                    | (ibytes[1] << 8) | (ibytes[0]));
            else
-             val = (ibytes[0] << 24) | (ibytes[1] << 16)
-               | (ibytes[2] << 8) | (ibytes[3]);
+             val = (((unsigned) ibytes[0] << 24) | (ibytes[1] << 16)
+                    | (ibytes[2] << 8) | (ibytes[3]));
 
            /* Removed [] around literal value to match ABI syntax 12/95.  */
-           fprintf (stream, "\t%s, 0x%X", grname[(inst >> 8) & 0xF], val);
+           (*print_func) (stream, "\t%s, 0x%X", grname[(inst >> 8) & 0xF], val);
 
            if (val == 0)
-             fprintf (stream, "\t// from address pool at 0x%x",
-                      (memaddr + 2 + ((inst & 0xFF) << 2)) & 0xFFFFFFFC);
+             (*print_func) (stream, "\t// from address pool at 0x%x",
+                            (uint32_t) (memaddr + 2
+                                        + ((inst & 0xFF) << 2)) & ~3);
          }
          break;
 
        case LJ:
          {
-           unsigned long val;
+           uint32_t val;
 
-           val = (memaddr + 2 + ((inst & 0xFF) << 2)) & 0xFFFFFFFC;
+           val = (memaddr + 2 + ((inst & 0xFF) << 2)) & ~3;
 
+           /* We are not reading an instruction, so allow
+              reads to extend beyond the next symbol.  */
+           info->stop_vma = 0;
            status = info->read_memory_func (val, ibytes, 4, info);
            if (status != 0)
              {
@@ -266,24 +272,25 @@ print_insn_mcore (memaddr, info)
              }
 
            if (info->endian == BFD_ENDIAN_LITTLE)
-             val = (ibytes[3] << 24) | (ibytes[2] << 16)
-               | (ibytes[1] << 8) | (ibytes[0]);
+             val = (((unsigned) ibytes[3] << 24) | (ibytes[2] << 16)
+                    | (ibytes[1] << 8) | (ibytes[0]));
            else
-             val = (ibytes[0] << 24) | (ibytes[1] << 16)
-               | (ibytes[2] << 8) | (ibytes[3]);
+             val = (((unsigned) ibytes[0] << 24) | (ibytes[1] << 16)
+                    | (ibytes[2] << 8) | (ibytes[3]));
 
            /* Removed [] around literal value to match ABI syntax 12/95.  */
-           fprintf (stream, "\t0x%X", val);
+           (*print_func) (stream, "\t0x%X", val);
            /* For jmpi/jsri, we'll try to get a symbol for the target.  */
            if (info->print_address_func && val != 0)
              {
-               fprintf (stream, "\t// ");
+               (*print_func) (stream, "\t// ");
                info->print_address_func (val, info);
              }
            else
              {
-               fprintf (stream, "\t// from address pool at 0x%x",
-                        (memaddr + 2 + ((inst & 0xFF) << 2)) & 0xFFFFFFFC);
+               (*print_func) (stream, "\t// from address pool at 0x%x",
+                              (uint32_t) (memaddr + 2
+                                          + ((inst & 0xFF) << 2)) & ~3);
              }
          }
          break;
@@ -295,13 +302,13 @@ print_insn_mcore (memaddr, info)
              "ee", "ee,ie", "ee,fe", "ee,fe,ie"
            };
 
-           fprintf (stream, "\t%s", fields[inst & 0x7]);
+           (*print_func) (stream, "\t%s", fields[inst & 0x7]);
          }
          break;
 
        default:
          /* If the disassembler lags the instruction set.  */
-         fprintf (stream, "\tundecoded operands, inst is 0x%04x", inst);
+         (*print_func) (stream, "\tundecoded operands, inst is 0x%04x", inst);
          break;
        }
     }
This page took 0.028488 seconds and 4 git commands to generate.