On 2/12/2010 10:26 AM, Phil Carmody wrote:
From: Phil Carmody<[email protected]>
We free in the tail cleanup, so don't free before jumping there.
Signed-off-by: Phil Carmody<[email protected]>
---
[...]
}
+
+ /* paranoid - must be able to kfree this on remaining error paths */
+ pProcObject->g_pszLastCoff = NULL;
+
do we need this? afaik it should be NULL at this point, because of
kmalloc + memset (which btw should be replaced by kzalloc).
i.e:
MEM_AllocObject(pProcObject, struct PROC_OBJECT, PROC_SIGNATURE);
-> MEM_Calloc(sizeof(Obj), MEM_NONPAGED);
-> pMem = kmalloc(...)
if (pMem)
memset(pMem, 0, cBytes);
/* Get the default executable for this board... */
DEV_GetDevType(hDevObject, (u32 *)&devType);
pProcObject->uProcessor = devType;
[...]
Regards,
Omar
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
