linux-kernel - Re: [PATCH] aic7xxx: fix coccicheck warnings

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAPWTe+JAmhnaxbfW2LW1=YGM4PKDyfW7WGgMHEwiPmw3yKtbTw@mail.gmail.com>
Date:	Tue, 12 Mar 2013 08:48:24 +0200
From:	Silviu Popescu <silviupopescu1990@...il.com>
To:	linux-scsi@...r.kernel.org
Cc:	hare@...e.de, JBottomley@...allels.com,
	linux-kernel@...r.kernel.org,
	Silviu-Mihai Popescu <silviupopescu1990@...il.com>
Subject: Re: [PATCH] aic7xxx: fix coccicheck warnings

On Sun, Mar 10, 2013 at 10:04 PM, Silviu-Mihai Popescu
<silviupopescu1990@...il.com> wrote:
> This replaces a call to kmalloc() followed by memset() with just one
> call to kzalloc().
>
> Signed-off-by: Silviu-Mihai Popescu <silviupopescu1990@...il.com>
> ---
>  drivers/scsi/aic7xxx/aic79xx_osm.c |    3 +--
>  drivers/scsi/aic7xxx/aic7xxx_osm.c |    3 +--
>  2 files changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/scsi/aic7xxx/aic79xx_osm.c b/drivers/scsi/aic7xxx/aic79xx_osm.c
> index 9328121..f6fdb65 100644
> --- a/drivers/scsi/aic7xxx/aic79xx_osm.c
> +++ b/drivers/scsi/aic7xxx/aic79xx_osm.c
> @@ -1324,10 +1324,9 @@ int
>  ahd_platform_alloc(struct ahd_softc *ahd, void *platform_arg)
>  {
>         ahd->platform_data =
> -           kmalloc(sizeof(struct ahd_platform_data), GFP_ATOMIC);
> +           kzalloc(sizeof(struct ahd_platform_data), GFP_ATOMIC);
>         if (ahd->platform_data == NULL)
>                 return (ENOMEM);
> -       memset(ahd->platform_data, 0, sizeof(struct ahd_platform_data));
>         ahd->platform_data->irq = AHD_LINUX_NOIRQ;
>         ahd_lockinit(ahd);
>         ahd->seltime = (aic79xx_seltime & 0x3) << 4;
> diff --git a/drivers/scsi/aic7xxx/aic7xxx_osm.c b/drivers/scsi/aic7xxx/aic7xxx_osm.c
> index 5a477cd..54bb1de 100644
> --- a/drivers/scsi/aic7xxx/aic7xxx_osm.c
> +++ b/drivers/scsi/aic7xxx/aic7xxx_osm.c
> @@ -1222,10 +1222,9 @@ ahc_platform_alloc(struct ahc_softc *ahc, void *platform_arg)
>  {
>
>         ahc->platform_data =
> -           kmalloc(sizeof(struct ahc_platform_data), GFP_ATOMIC);
> +           kzalloc(sizeof(struct ahc_platform_data), GFP_ATOMIC);
>         if (ahc->platform_data == NULL)
>                 return (ENOMEM);
> -       memset(ahc->platform_data, 0, sizeof(struct ahc_platform_data));
>         ahc->platform_data->irq = AHC_LINUX_NOIRQ;
>         ahc_lockinit(ahc);
>         ahc->seltime = (aic7xxx_seltime & 0x3) << 4;
> --
> 1.7.9.5
>

Hello,

Would anyone be so kind as to review this?

Thanks,
Silviu Popescu
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/