commit 2b7a972e3f36bfcdc6fbe2b59d7ffdcde49c9405
author Matt Turner <mattst88@gmail.com> Mon Sep 03 19:44:00 2012 -0700
committer Matt Turner <mattst88@gmail.com> Wed Sep 05 22:28:50 2012 -0700
tree 4b9504b4799e9d29363690fb9083ac4bbcf78d51
parent 812931f602ff913a51a608a9b1b6826b7f2bfae0

Don't cast the return value of malloc/realloc

This patch has been generated by the following Coccinelle semantic
patch:

// Don't cast the return value of malloc/realloc.
//
// Casting the return value of malloc/realloc only stands to hide
// errors.

@@
type T;
expression E1, E2;
@@
- (T)
(
_mesa_align_calloc(E1, E2)
|
_mesa_align_malloc(E1, E2)
|
calloc(E1, E2)
|
malloc(E1)
|
realloc(E1, E2)
)

src/glx/glxcmds.c

diff --git a/src/glx/glxcmds.c b/src/glx/glxcmds.c
index 0b90b42..ddfe12c 100644
--- a/src/glx/glxcmds.c
+++ b/src/glx/glxcmds.c
@@ -2454,7 +2454,7 @@
 __glXstrdup(const char *str)
 {
    char *copy;
-   copy = (char *) malloc(strlen(str) + 1);
+   copy = malloc(strlen(str) + 1);
    if (!copy)
       return NULL;
    strcpy(copy, str);