Message ID | 20200407092852.GI68494@mwanda |
---|---|
State | Accepted |
Headers | show |
Series | rtc: rc5t619: Fix an ERR_PTR vs NULL check | expand |
On 07/04/2020 12:28:52+0300, Dan Carpenter wrote: > The devm_kzalloc() function returns NULL on error, it doesn't return > error pointers so this check doesn't work. > > Fixes: 540d1e15393d ("rtc: rc5t619: Add Ricoh RC5T619 RTC driver") > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> > --- > drivers/rtc/rtc-rc5t619.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > Applied, thanks.
diff --git a/drivers/rtc/rtc-rc5t619.c b/drivers/rtc/rtc-rc5t619.c index 24e386ecbc7e..dd1a20977478 100644 --- a/drivers/rtc/rtc-rc5t619.c +++ b/drivers/rtc/rtc-rc5t619.c @@ -356,10 +356,8 @@ static int rc5t619_rtc_probe(struct platform_device *pdev) int err; rtc = devm_kzalloc(dev, sizeof(*rtc), GFP_KERNEL); - if (IS_ERR(rtc)) { - err = PTR_ERR(rtc); + if (!rtc) return -ENOMEM; - } rtc->rn5t618 = rn5t618;
The devm_kzalloc() function returns NULL on error, it doesn't return error pointers so this check doesn't work. Fixes: 540d1e15393d ("rtc: rc5t619: Add Ricoh RC5T619 RTC driver") Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> --- drivers/rtc/rtc-rc5t619.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)