* [PATCH] net/wanrouter/wanmain.c: check kmalloc() return value.
@ 2007-03-09 7:26 Amit Choudhary
2007-03-09 7:31 ` David Miller
0 siblings, 1 reply; 2+ messages in thread
From: Amit Choudhary @ 2007-03-09 7:26 UTC (permalink / raw)
To: Linux Kernel
Description: Check the return value of kmalloc() in function dbg_kmalloc(), in file net/wanrouter/wanmain.c.
Signed-off-by: Amit Choudhary <amit2030@gmail.com>
diff --git a/net/wanrouter/wanmain.c b/net/wanrouter/wanmain.c
index 316211d..263450c 100644
--- a/net/wanrouter/wanmain.c
+++ b/net/wanrouter/wanmain.c
@@ -67,6 +67,8 @@ static void * dbg_kmalloc(unsigned int s
int i = 0;
void * v = kmalloc(size+sizeof(unsigned int)+2*KMEM_SAFETYZONE*8,prio);
char * c1 = v;
+ if (!v)
+ return NULL;
c1 += sizeof(unsigned int);
*((unsigned int *)v) = size;
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] net/wanrouter/wanmain.c: check kmalloc() return value.
2007-03-09 7:26 [PATCH] net/wanrouter/wanmain.c: check kmalloc() return value Amit Choudhary
@ 2007-03-09 7:31 ` David Miller
0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2007-03-09 7:31 UTC (permalink / raw)
To: amit2030; +Cc: linux-kernel
From: Amit Choudhary <amit2030@gmail.com>
Date: Thu, 8 Mar 2007 23:26:54 -0800
> Description: Check the return value of kmalloc() in function dbg_kmalloc(), in file net/wanrouter/wanmain.c.
>
> Signed-off-by: Amit Choudhary <amit2030@gmail.com>
There is no reason for any subsystem to implement it's
own debugging allocator when we have one that works
perfectly fine already in SLAB.
So to fix this we should simply remove all of the
allocation debugging code here.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2007-03-09 7:31 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-03-09 7:26 [PATCH] net/wanrouter/wanmain.c: check kmalloc() return value Amit Choudhary
2007-03-09 7:31 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®