remove redundant NULL check on free
free() already checks the pointer to be non NULL. No need to do it again. Signed-off-by: Lucas Stach <dev@lynxeye.de> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
This commit is contained in:
parent
34ab8ddd04
commit
aa6694b542
|
@ -371,8 +371,7 @@ skip:
|
|||
ret = 0;
|
||||
out:
|
||||
close(envfd);
|
||||
if (buf_free)
|
||||
free(buf_free);
|
||||
free(buf_free);
|
||||
return ret;
|
||||
}
|
||||
|
||||
|
|
|
@ -294,8 +294,7 @@ static int at25_probe(struct device_d *dev)
|
|||
return 0;
|
||||
|
||||
fail:
|
||||
if (at25)
|
||||
free(at25);
|
||||
free(at25);
|
||||
|
||||
return err;
|
||||
}
|
||||
|
|
|
@ -504,8 +504,7 @@ int usb_host_detect(struct usb_host *host, int force)
|
|||
|
||||
list_del(&dev->list);
|
||||
unregister_device(&dev->dev);
|
||||
if (dev->hub)
|
||||
free(dev->hub);
|
||||
free(dev->hub);
|
||||
dma_free(dev->setup_packet);
|
||||
dma_free(dev->descriptor);
|
||||
free(dev);
|
||||
|
|
|
@ -108,8 +108,7 @@ int dev_set_param(struct device_d *dev, const char *name, const char *val)
|
|||
int dev_param_set_generic(struct device_d *dev, struct param_d *p,
|
||||
const char *val)
|
||||
{
|
||||
if (p->value)
|
||||
free(p->value);
|
||||
free(p->value);
|
||||
if (!val) {
|
||||
p->value = NULL;
|
||||
return 0;
|
||||
|
|
Loading…
Reference in New Issue