On Wed, Mar 21, 2012 at 08:49:45AM +0000, James Bottomley wrote: > On Wed, 2012-03-21 at 14:08 +0530, santosh nayak wrote: > > From: Santosh Nayak <santoshprasadnayak@xxxxxxxxx> > > > > We need to add error handler if kzalloc fails to allocate and initialize. > > Otherwise it may lead to Null derefernce error in > > 'pm8001_chip_get_nvmd_req()'. > > It's a GFP_KERNEL allocation. They sleep until memory is available > (__GFP_WAIT). It's GFP_ATOMIC that can return NULL. > That's not right at all. Only __GFP_NOFAIL allocations don't need to be checked. regards, dan carpenter
Attachment:
signature.asc
Description: Digital signature