[PATCH] remove checks for xzalloc() returning NULL
Sascha Hauer
s.hauer at pengutronix.de
Mon Sep 25 23:51:46 PDT 2017
On Mon, Sep 25, 2017 at 12:02:35PM +0200, Uwe Kleine-König wrote:
> xzalloc() either returns memory or panics, so checking for NULL is useless.
>
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig at pengutronix.de>
> ---
> common/state/backend_bucket_circular.c | 2 --
> common/state/backend_format_dtb.c | 2 --
> common/state/backend_format_raw.c | 4 ----
> drivers/clk/mxs/clk-div.c | 2 --
> drivers/clk/mxs/clk-pll.c | 2 --
> drivers/clk/mxs/clk-ref.c | 2 --
> drivers/gpio/gpio-clps711x.c | 2 --
> drivers/gpio/gpio-generic.c | 2 --
> drivers/gpio/gpio-vf610.c | 2 --
> drivers/mtd/nand/atmel_nand.c | 2 --
> drivers/net/gianfar.c | 3 ---
> drivers/of/base.c | 2 --
> drivers/pci/pci-imx6.c | 2 --
> drivers/pinctrl/pinctrl-rockchip.c | 2 --
> drivers/usb/gadget/pxa27x_udc.c | 2 --
> drivers/usb/musb/musb_host.c | 2 --
> drivers/usb/musb/phy-am335x-control.c | 4 ----
> drivers/usb/musb/phy-am335x.c | 2 --
> drivers/video/backlight-pwm.c | 2 --
> drivers/video/tc358767.c | 2 --
> fs/omap4_usbbootfs.c | 2 --
> fs/pstore/fs.c | 2 --
> 22 files changed, 49 deletions(-)
Applied, thanks
Sascha
>
> diff --git a/common/state/backend_bucket_circular.c b/common/state/backend_bucket_circular.c
> index f83bd679d788..232490325523 100644
> --- a/common/state/backend_bucket_circular.c
> +++ b/common/state/backend_bucket_circular.c
> @@ -310,8 +310,6 @@ static int state_backend_bucket_circular_write(struct state_backend_storage_buck
> * random changes
> */
> write_buf = xzalloc(written_length);
> - if (!write_buf)
> - return -ENOMEM;
>
> memcpy(write_buf, buf, len);
> meta = (struct state_backend_storage_bucket_circular_meta *)
> diff --git a/common/state/backend_format_dtb.c b/common/state/backend_format_dtb.c
> index 4c9d2eefc780..48f30db1f5f9 100644
> --- a/common/state/backend_format_dtb.c
> +++ b/common/state/backend_format_dtb.c
> @@ -136,8 +136,6 @@ int backend_format_dtb_create(struct state_backend_format **format,
> struct state_backend_format_dtb *dtb;
>
> dtb = xzalloc(sizeof(*dtb));
> - if (!dtb)
> - return -ENOMEM;
>
> dtb->dev = dev;
> dtb->format.pack = state_backend_format_dtb_pack;
> diff --git a/common/state/backend_format_raw.c b/common/state/backend_format_raw.c
> index 2ba97e08a0ae..4369f76e376f 100644
> --- a/common/state/backend_format_raw.c
> +++ b/common/state/backend_format_raw.c
> @@ -222,8 +222,6 @@ static int backend_format_raw_pack(struct state_backend_format *format,
> size_full = size_data + sizeof(*header) + backend_raw->digest_length;
>
> buf = xzalloc(size_full);
> - if (!buf)
> - return -ENOMEM;
>
> header = buf;
> data = buf + sizeof(*header);
> @@ -308,8 +306,6 @@ int backend_format_raw_create(struct state_backend_format **format,
> int ret;
>
> raw = xzalloc(sizeof(*raw));
> - if (!raw)
> - return -ENOMEM;
>
> raw->dev = dev;
> ret = backend_format_raw_init_digest(raw, node, secret_name);
> diff --git a/drivers/clk/mxs/clk-div.c b/drivers/clk/mxs/clk-div.c
> index e8dae25da5ff..797e5a274fd0 100644
> --- a/drivers/clk/mxs/clk-div.c
> +++ b/drivers/clk/mxs/clk-div.c
> @@ -86,8 +86,6 @@ struct clk *mxs_clk_div(const char *name, const char *parent_name,
> int ret;
>
> div = xzalloc(sizeof(*div));
> - if (!div)
> - return ERR_PTR(-ENOMEM);
>
> div->parent = parent_name;
> div->divider.clk.name = name;
> diff --git a/drivers/clk/mxs/clk-pll.c b/drivers/clk/mxs/clk-pll.c
> index 89fd6b5e3112..1b1c9b35434b 100644
> --- a/drivers/clk/mxs/clk-pll.c
> +++ b/drivers/clk/mxs/clk-pll.c
> @@ -96,8 +96,6 @@ struct clk *mxs_clk_pll(const char *name, const char *parent_name,
> int ret;
>
> pll = xzalloc(sizeof(*pll));
> - if (!pll)
> - return ERR_PTR(-ENOMEM);
>
> pll->parent = parent_name;
> pll->clk.name = name;
> diff --git a/drivers/clk/mxs/clk-ref.c b/drivers/clk/mxs/clk-ref.c
> index 7ff5527797df..8d0be05615c9 100644
> --- a/drivers/clk/mxs/clk-ref.c
> +++ b/drivers/clk/mxs/clk-ref.c
> @@ -144,8 +144,6 @@ struct clk *mxs_clk_ref(const char *name, const char *parent_name,
> int ret;
>
> ref = xzalloc(sizeof(*ref));
> - if (!ref)
> - return ERR_PTR(-ENOMEM);
>
> ref->parent = parent_name;
> ref->clk.name = name;
> diff --git a/drivers/gpio/gpio-clps711x.c b/drivers/gpio/gpio-clps711x.c
> index 769e6ca4f719..c1ec6ab68554 100644
> --- a/drivers/gpio/gpio-clps711x.c
> +++ b/drivers/gpio/gpio-clps711x.c
> @@ -47,8 +47,6 @@ static int clps711x_gpio_probe(struct device_d *dev)
> }
>
> bgc = xzalloc(sizeof(struct bgpio_chip));
> - if (!bgc)
> - return -ENOMEM;
>
> err = bgpio_init(bgc, dev, 1, dat, NULL, NULL, dir, dir_inv, 0);
> if (err)
> diff --git a/drivers/gpio/gpio-generic.c b/drivers/gpio/gpio-generic.c
> index c6202fc6512a..608361185aba 100644
> --- a/drivers/gpio/gpio-generic.c
> +++ b/drivers/gpio/gpio-generic.c
> @@ -373,8 +373,6 @@ static int bgpio_dev_probe(struct device_d *dev)
> dev_get_drvdata(dev, (const void **)&flags);
>
> bgc = xzalloc(sizeof(struct bgpio_chip));
> - if (!bgc)
> - return -ENOMEM;
>
> err = bgpio_init(bgc, dev, sz, dat, set, clr, dirout, dirin, flags);
> if (err)
> diff --git a/drivers/gpio/gpio-vf610.c b/drivers/gpio/gpio-vf610.c
> index 7c8d1e4c9884..2aff62be5970 100644
> --- a/drivers/gpio/gpio-vf610.c
> +++ b/drivers/gpio/gpio-vf610.c
> @@ -106,8 +106,6 @@ static int vf610_gpio_probe(struct device_d *dev)
> const __be32 *gpio_ranges;
>
> port = xzalloc(sizeof(*port));
> - if (!port)
> - return -ENOMEM;
>
> gpio_ranges = of_get_property(dev->device_node, "gpio-ranges", &size);
> if (!gpio_ranges) {
> diff --git a/drivers/mtd/nand/atmel_nand.c b/drivers/mtd/nand/atmel_nand.c
> index 299cc15c3d16..0b9071c9bce7 100644
> --- a/drivers/mtd/nand/atmel_nand.c
> +++ b/drivers/mtd/nand/atmel_nand.c
> @@ -898,8 +898,6 @@ static int __init atmel_pmecc_nand_init_params(struct device_d *dev,
> /* Set pmecc_rom_base as the begin of gf table */
> int size = sector_size == 512 ? 0x2000 : 0x4000;
> pmecc_galois_table = xzalloc(2 * size * sizeof(uint16_t));
> - if (!pmecc_galois_table)
> - return -ENOMEM;
> host->pmecc_rom_base = pmecc_galois_table;
> ret = pmecc_build_galois_table((sector_size == 512) ?
> PMECC_GF_DIMENSION_13 :
> diff --git a/drivers/net/gianfar.c b/drivers/net/gianfar.c
> index ac698332cf1b..9b2f781d9cb5 100644
> --- a/drivers/net/gianfar.c
> +++ b/drivers/net/gianfar.c
> @@ -483,9 +483,6 @@ static int gfar_probe(struct device_d *dev)
>
> priv = xzalloc(sizeof(struct gfar_private));
>
> - if (NULL == priv)
> - return -ENODEV;
> -
> edev = &priv->edev;
>
> priv->mdiobus_tbi = gfar_info->mdiobus_tbi;
> diff --git a/drivers/of/base.c b/drivers/of/base.c
> index ea330d131012..b5555d5fe63d 100644
> --- a/drivers/of/base.c
> +++ b/drivers/of/base.c
> @@ -194,8 +194,6 @@ void of_alias_scan(void)
>
> /* Allocate an alias_prop with enough space for the stem */
> ap = xzalloc(sizeof(*ap) + len + 1);
> - if (!ap)
> - continue;
> ap->alias = start;
> of_alias_add(ap, np, id, start, len);
> }
> diff --git a/drivers/pci/pci-imx6.c b/drivers/pci/pci-imx6.c
> index affc705d0aba..fc56a0d14ade 100644
> --- a/drivers/pci/pci-imx6.c
> +++ b/drivers/pci/pci-imx6.c
> @@ -580,8 +580,6 @@ static int __init imx6_pcie_probe(struct device_d *dev)
> int ret;
>
> imx6_pcie = xzalloc(sizeof(*imx6_pcie));
> - if (!imx6_pcie)
> - return -ENOMEM;
>
> pp = &imx6_pcie->pp;
> pp->dev = dev;
> diff --git a/drivers/pinctrl/pinctrl-rockchip.c b/drivers/pinctrl/pinctrl-rockchip.c
> index 1130f3d845ea..58f2c4af2686 100644
> --- a/drivers/pinctrl/pinctrl-rockchip.c
> +++ b/drivers/pinctrl/pinctrl-rockchip.c
> @@ -412,8 +412,6 @@ static int rockchip_pinctrl_probe(struct device_d *dev)
> int ret;
>
> info = xzalloc(sizeof(struct rockchip_pinctrl));
> - if (!info)
> - return -ENOMEM;
>
> ctrl = rockchip_pinctrl_get_soc_data(info, dev);
> if (!ctrl) {
> diff --git a/drivers/usb/gadget/pxa27x_udc.c b/drivers/usb/gadget/pxa27x_udc.c
> index 372c07b41866..4e6b858868c8 100644
> --- a/drivers/usb/gadget/pxa27x_udc.c
> +++ b/drivers/usb/gadget/pxa27x_udc.c
> @@ -218,8 +218,6 @@ static struct usb_request *pxa_ep_alloc_request(struct usb_ep *_ep)
> struct pxa27x_request *req;
>
> req = xzalloc(sizeof *req);
> - if (!req)
> - return NULL;
>
> INIT_LIST_HEAD(&req->queue);
> req->in_use = 0;
> diff --git a/drivers/usb/musb/musb_host.c b/drivers/usb/musb/musb_host.c
> index 32a8f0652931..68d819af2cc8 100644
> --- a/drivers/usb/musb/musb_host.c
> +++ b/drivers/usb/musb/musb_host.c
> @@ -1717,8 +1717,6 @@ int musb_host_alloc(struct musb *musb)
> {
> /* usbcore sets dev->driver_data to hcd, and sometimes uses that... */
> musb->hcd = xzalloc(sizeof(struct usb_hcd));
> - if (!musb->hcd)
> - return -EINVAL;
>
> musb->hcd->hcd_priv = musb;
>
> diff --git a/drivers/usb/musb/phy-am335x-control.c b/drivers/usb/musb/phy-am335x-control.c
> index 5fd8802b3a75..55a2ed27b7d3 100644
> --- a/drivers/usb/musb/phy-am335x-control.c
> +++ b/drivers/usb/musb/phy-am335x-control.c
> @@ -140,10 +140,6 @@ static int am335x_control_usb_probe(struct device_d *dev)
> return ret;
>
> ctrl_usb = xzalloc(sizeof(*ctrl_usb));
> - if (!ctrl_usb) {
> - dev_err(dev, "unable to alloc memory for control usb\n");
> - return -ENOMEM;
> - }
>
> ctrl_usb->dev = dev;
>
> diff --git a/drivers/usb/musb/phy-am335x.c b/drivers/usb/musb/phy-am335x.c
> index 204e51054dce..ec8c0f538b62 100644
> --- a/drivers/usb/musb/phy-am335x.c
> +++ b/drivers/usb/musb/phy-am335x.c
> @@ -34,8 +34,6 @@ static int am335x_phy_probe(struct device_d *dev)
> int ret;
>
> am_usbphy = xzalloc(sizeof(*am_usbphy));
> - if (!am_usbphy)
> - return -ENOMEM;
>
> iores = dev_request_mem_resource(dev, 0);
> if (IS_ERR(iores)) {
> diff --git a/drivers/video/backlight-pwm.c b/drivers/video/backlight-pwm.c
> index 2915dba17405..ec35bd1bc039 100644
> --- a/drivers/video/backlight-pwm.c
> +++ b/drivers/video/backlight-pwm.c
> @@ -147,8 +147,6 @@ static int pwm_backlight_parse_dt(struct device_d *dev,
> size_t size = sizeof(*pwm_backlight->levels) * length;
>
> pwm_backlight->levels = xzalloc(size);
> - if (!pwm_backlight->levels)
> - return -ENOMEM;
>
> ret = of_property_read_u32_array(node, "brightness-levels",
> pwm_backlight->levels,
> diff --git a/drivers/video/tc358767.c b/drivers/video/tc358767.c
> index 2d366ae6627a..f9ca06c62bcb 100644
> --- a/drivers/video/tc358767.c
> +++ b/drivers/video/tc358767.c
> @@ -1352,8 +1352,6 @@ static int tc_probe(struct device_d *dev)
> int ret;
>
> tc = xzalloc(sizeof(struct tc_data));
> - if (!tc)
> - return -ENOMEM;
>
> tc->client = client;
> tc->dev = dev;
> diff --git a/fs/omap4_usbbootfs.c b/fs/omap4_usbbootfs.c
> index 6085bca841e9..b35f411cbbc1 100644
> --- a/fs/omap4_usbbootfs.c
> +++ b/fs/omap4_usbbootfs.c
> @@ -73,8 +73,6 @@ static struct file_priv *omap4_usbbootfs_do_open(
> return ERR_PTR(-ENOSYS);
>
> priv = xzalloc(sizeof(*priv));
> - if (!priv)
> - return ERR_PTR(-ENOMEM);
>
> data = OMAP4_USBBOOT_FS_MAGIC ; omap4_usbboot_write(&data, 4);
> data = OMAP4_USBBOOT_FS_CMD_OPEN; omap4_usbboot_write(&data, 4);
> diff --git a/fs/pstore/fs.c b/fs/pstore/fs.c
> index 0e05d48ea041..a879a680641c 100644
> --- a/fs/pstore/fs.c
> +++ b/fs/pstore/fs.c
> @@ -64,8 +64,6 @@ int pstore_mkfile(enum pstore_type_id type, char *psname, u64 id, int count,
> }
>
> private = xzalloc(sizeof(*private) + size);
> - if (!private)
> - return -ENOMEM;
> private->type = type;
> private->id = id;
> private->count = count;
> --
> 2.11.0
>
>
> _______________________________________________
> barebox mailing list
> barebox at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
More information about the barebox
mailing list