This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [PATCH RFA] utils.c: Fix xcalloc (0, 0) behavior
- To: Kevin Buettner <kevinb at cygnus dot com>
- Subject: Re: [PATCH RFA] utils.c: Fix xcalloc (0, 0) behavior
- From: David Taylor <taylor at cygnus dot com>
- Date: Mon, 05 Mar 2001 10:52:16 -0500
- cc: gdb-patches at sources dot redhat dot com
Date: Sat, 3 Mar 2001 00:58:08 -0700
From: Kevin Buettner <kevinb@cygnus.com>
According to section 16.1 in Harbison & Steele, it is permissible for
calloc(0,0) to return either NULL or an implementation defined unique
pointer. I've come across an implementation of calloc() which chooses
to return NULL.
Unfortunately, our implementation of xcalloc() and xmrealloc() choose
to treat the NULL return value as an error condition and call nomem()
as a result.
The patch below corrects this oversight.
Okay to apply?
* utils.c (xmrealloc): Don't call nomem() when return value
is NULL _and_ request size is 0.
(xcalloc): Likewise.
Yes. Approved.
Index: utils.c
===================================================================
RCS file: /cvs/src/src/gdb/utils.c,v
retrieving revision 1.31
diff -u -p -r1.31 utils.c
--- utils.c 2001/02/25 04:45:11 1.31
+++ utils.c 2001/03/03 07:47:54
@@ -1050,7 +1050,7 @@ xmrealloc (PTR md, PTR ptr, long size)
{
val = mmalloc (md, size);
}
- if (val == NULL)
+ if (val == NULL && size != 0)
{
nomem (size);
}
@@ -1072,7 +1072,7 @@ PTR
xcalloc (size_t number, size_t size)
{
void *mem = mcalloc (NULL, number, size);
- if (mem == NULL)
+ if (mem == NULL && number != 0 && size != 0)
nomem (number * size);
return mem;
}