linux-kernel - Re: [PATCH] SCSI: minor bug fixes and cleanups

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] [day] [month] [year] [list]

Message-Id: <200610110818.25158.eike-kernel@sf-tec.de>
Date:	Wed, 11 Oct 2006 08:18:24 +0200
From:	Rolf Eike Beer <eike-kernel@...tec.de>
To:	Jeff Garzik <jeff@...zik.org>
Cc:	linux-scsi@...r.kernel.org, Andrew Morton <akpm@...l.org>,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] SCSI: minor bug fixes and cleanups

Jeff Garzik wrote:
> BusLogic: use kzalloc(), remove cast to/from void*
>
> aic7xxx_old: fix typo in cast
>
> NCR53c406a: ifdef out static built code
>
> fd_mcs: ifdef out static built code
>
> ncr53c8xx: ifdef out static built code
>
> Signed-off-by: Jeff Garzik <jeff@...zik.org>
>
> ---
>
>  drivers/scsi/BusLogic.c       |   12 ++++++------
>  drivers/scsi/NCR53c406a.c     |    5 +++++
>  drivers/scsi/aic7xxx_old.c    |    2 +-
>  drivers/scsi/fd_mcs.c         |    2 ++
>  drivers/scsi/ncr53c8xx.c      |   19 +++++++++++--------
>
> diff --git a/drivers/scsi/BusLogic.c b/drivers/scsi/BusLogic.c
> index 7c59bba..689dc4c 100644
> --- a/drivers/scsi/BusLogic.c
> +++ b/drivers/scsi/BusLogic.c
> @@ -2186,21 +2186,21 @@ #endif
>
>  	if (BusLogic_ProbeOptions.NoProbe)
>  		return -ENODEV;
> -	BusLogic_ProbeInfoList = (struct BusLogic_ProbeInfo *)
> -	    kmalloc(BusLogic_MaxHostAdapters * sizeof(struct BusLogic_ProbeInfo),
> GFP_ATOMIC); +	BusLogic_ProbeInfoList =
> +	    kzalloc(BusLogic_MaxHostAdapters * sizeof(struct BusLogic_ProbeInfo),
> GFP_KERNEL); if (BusLogic_ProbeInfoList == NULL) {
>  		BusLogic_Error("BusLogic: Unable to allocate Probe Info List\n", NULL);
>  		return -ENOMEM;
>  	}
> -	memset(BusLogic_ProbeInfoList, 0, BusLogic_MaxHostAdapters *
> sizeof(struct BusLogic_ProbeInfo));

kcalloc

Eike

Content of type "application/pgp-signature" skipped