Re: [PATCH 02/10] scsi: fix integer as NULL pointer warnings

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

From: Harvey Harrison

Subject: Re: [PATCH 02/10] scsi: fix integer as NULL pointer warnings

Date: Wednesday, April 30, 2008 - 3:34 pm

On Wed, 2008-04-30 at 17:29 -0500, James Bottomley wrote:
quoted text> On Wed, 2008-04-30 at 15:03 -0700, Harvey Harrison wrote:
> > drivers/scsi/FlashPoint.c:906:9: warning: Using plain integer as NULL pointer
> > drivers/scsi/FlashPoint.c:907:53: warning: Using plain integer as NULL pointer
> > drivers/scsi/FlashPoint.c:922:1: warning: Using plain integer as NULL pointer

quoted text> The flashpoint initialiser changes look wrong:  {0} is a conventional
> notation for zero initialise this structure.  Sparse should probably be
> taught to recognise that.

Actually, looking closer, should this be initialized at all?  Can we
just remove the initialization here and let them end up in .bss?

Harvey

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

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:

[PATCH 02/10] scsi: fix integer as NULL pointer warnings, Harvey Harrison, (Wed Apr 30, 3:03 pm)

Re: [PATCH 02/10] scsi: fix integer as NULL pointer warnings, James Bottomley, (Wed Apr 30, 3:29 pm)

Re: [PATCH 02/10] scsi: fix integer as NULL pointer warnings, Harvey Harrison, (Wed Apr 30, 3:34 pm)

Re: [PATCH 02/10] scsi: fix integer as NULL pointer warnings, James Bottomley, (Wed Apr 30, 3:54 pm)

Re: [PATCH 02/10] scsi: fix integer as NULL pointer warnings, Alan Cox, (Wed Apr 30, 4:29 pm)

RE: [PATCH 02/10] scsi: fix integer as NULL pointer warnings, Mark Harvey, (Thu May 1, 2:50 pm)


linux-kernel:
David Howells	[PATCH] KEYS: Use the variable 'key' in keyctl_describe_key()
Greg Kroah-Hartman	[PATCH 17/36] sysdev: detect multiple driver registrations
Greg Kroah-Hartman	[PATCH 09/36] driver core: register_memory/unregister_memory clean ups and bugfix
Pierre Ossman	Re: sdio: enhance IO_RW_EXTENDED support
Sam Ravnborg	Re: [PATCH] kbuild: fix make V=1
git:
Mark Junker	git on MacOSX and files with decomposed utf-8 file names
Johannes Schindelin	Re: error: cannot lock ref 'refs/remotes/origin/*'
Pat Thoyts	[PATCH] git-gui: use themed tk widgets with Tk 8.5
Johannes Schindelin	Re: [PATCH 2/2] git-svn: support fetch with autocrlf on
Jonathan Nieder	Re: [PATCH v2] git-send-email.perl: fix In-Reply-To for second and subsequent patc...
linux-netdev:
David Miller	Re: [PATCH 32/53] netns xfrm: finding policy in netns
Jean-Louis Dupond	Re: tg3 driver not advertising 1000mbit
Jan Engelhardt	[PATCH 1/3] net: tcp: make hybla selectable as default congestion module
Daniel Schaffrath	Re: tcp bw in 2.6
Matt Mackall	Re: [regression] nf_iterate(), BUG: unable to handle kernel NULL pointer dereference
git-commits-head:
Linux Kernel Mailing List	ipv6: fix an oops when force unload ipv6 module
Linux Kernel Mailing List	tracing: protect reader of cmdline output
Linux Kernel Mailing List	KVM: VMX: Clear CR4.VMXE in hardware_disable
Linux Kernel Mailing List	imxfb: Fix margin settings
Linux Kernel Mailing List	USB: set correct configuration in probe of ti_usb_3410_5052
openbsd-misc:
Stas Miasnikou	Re: Another question: device naming convention
Darrin Chandler	Re: That whole "Linux stealing our code" thing
Community First Financial	Teacher A+ Loan
Jan Stary	Re: audio recording levels
Andrej Elizarov	Re: Web Browsers