RE: [PATCH] PPC + ARC: Fix calloc() call

[Claudiu Zissulescu <Claudiu.Zissulescu@synopsys.com>]

Hi,

Ok. Please commit it.

Thank you,
Claudiu

-----Original Message-----
From: Jan-Benedict Glaw <jbglaw@lug-owl.de> 
Sent: Tuesday, November 21, 2023 3:40 PM
To: binutils@sourceware.org
Cc: Claudiu Zissulescu <claziss@synopsys.com>; Geoff Keating <geoffk@geoffk.org>; Alan Modra <amodra@gmail.com>; Peter Bergner <bergner@vnet.ibm.com>
Subject: [PATCH] PPC + ARC: Fix calloc() call

H!


Recently, -Walloc-size warnings started to kick in. Fix these two
calloc() calls to match the intended usage pattern.

opcodes/ChangeLog:

	* arc-dis.c (init_arc_disasm_info): Fix calloc() call.
	* ppc-dis.c (powerpc_init_dialect): Ditto.

diff --git a/opcodes/arc-dis.c b/opcodes/arc-dis.c index 59b668ff64e..7498e75cee2 100644
--- a/opcodes/arc-dis.c
+++ b/opcodes/arc-dis.c
@@ -147,7 +147,7 @@ static bool
 init_arc_disasm_info (struct disassemble_info *info)  {
   struct arc_disassemble_info *arc_infop
-    = calloc (sizeof (*arc_infop), 1);
+    = calloc (1, sizeof (*arc_infop));
 
   if (arc_infop == NULL)
     return false;
diff --git a/opcodes/ppc-dis.c b/opcodes/ppc-dis.c index d97137d8b71..4d5652ebfeb 100644
--- a/opcodes/ppc-dis.c
+++ b/opcodes/ppc-dis.c
@@ -348,7 +348,7 @@ powerpc_init_dialect (struct disassemble_info *info)  {
   ppc_cpu_t dialect = 0;
   ppc_cpu_t sticky = 0;
-  struct dis_private *priv = calloc (sizeof (*priv), 1);
+  struct dis_private *priv = calloc (1, sizeof (*priv));
 
   if (priv == NULL)
     return;



Ok to commit?

MfG, JBG

--