diff options
author | Alan Coopersmith <alan.coopersmith@oracle.com> | 2011-09-30 20:19:43 -0700 |
---|---|---|
committer | Alan Coopersmith <alan.coopersmith@oracle.com> | 2011-09-30 20:19:43 -0700 |
commit | 93812aa7950342c809a9dcbd9d50379c050b51ac (patch) | |
tree | 73fa307d6b4a52666620ee8f273c678514b422c1 | |
parent | 7392f2b780ab3346014634d2532f3b8ca99101a6 (diff) | |
download | xscope-93812aa7950342c809a9dcbd9d50379c050b51ac.tar.gz |
Clean up existing malloc calls
Remove unnecessary casts from (void *)
Ensure return value is checked.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
-rw-r--r-- | common.c | 4 | ||||
-rw-r--r-- | scope.c | 2 | ||||
-rw-r--r-- | table11.c | 2 |
3 files changed, 5 insertions, 3 deletions
@@ -239,7 +239,9 @@ SetUpConnectionSocket( debug(4,(stderr, "Warning: Failed to establish listening connections on some transports\n")); } - ListenTransFds = (int *) malloc (ListenTransCount * sizeof (int)); + ListenTransFds = malloc (ListenTransCount * sizeof (int)); + if (ListenTransFds == NULL) + panic("Can't allocate memory for ListenTransFds"); for (i = 0; i < ListenTransCount; i++) { @@ -489,7 +489,7 @@ CMDBreak ( *minorname = ':'; /* restore string for error message */ return CMDSyntax; } - bp = (BP *) malloc (sizeof (BP)); + bp = malloc (sizeof (BP)); bp->number = ++breakPointNumber; bp->request = request; bp->minorop = minorop; @@ -147,7 +147,7 @@ CreateValueRec ( int i; bucket = &buckets[HASH(key)]; - value = (ValuePtr) malloc (sizeof (ValueRec) + size * sizeof (unsigned long)); + value = malloc (sizeof (ValueRec) + (size * sizeof (unsigned long))); if (!value) return; value->values = (unsigned long *) (value + 1); |