[kbuild] [next:master 5753/6487] drivers/spi/spidev.c:583 spidev_release() warn: variable dereferenced before check 'spidev-spi' (see line 579)

TO: Mark Brown broo...@kernel.org tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master head: bbdef57970d5e1887de755474ff1562baa17ef11 commit: 9169051617df7fca597274e9e43324332cb8f0ee [5753/6487] spi: spidev: Don't mangle max_speed_hz in underlying spi device ::

Re: [kbuild] [next:master 5753/6487] drivers/spi/spidev.c:583 spidev_release() warn: variable dereferenced before check 'spidev-spi' (see line 579)

I reported this one earlier. regards, dan carpenter On Thu, Nov 13, 2014 at 04:24:12PM +0800, kbuild test robot wrote: TO: Mark Brown broo...@kernel.org tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master head: bbdef57970d5e1887de755474ff1562baa17ef11

Re: [kbuild] fs/overlayfs/super.c:367 ovl_lookup() error: passing non negative 2147383647 to ERR_PTR

I see you have upgraded to the latest and greatest version of Smatch. :) This one is a false positive, though, sorry for that. I'm not sure how it gets triggered and I can't reproduce it on my end. 2147383647 is a magic number which means approximately the highest valid pointer on 32 bit

Re: [kbuild] [PATCH] iio: adc: fix platform_no_drv_owner.cocci warnings (fwd)

Hi Julia, On Thu, Nov 13, 2014 at 11:26:23AM +0100, Julia Lawall wrote: Is there any way that we can improve this process? At the moment, I just take the message I get and forward it. How about always send the patches to public LKML, with you in the CC list? Thanks, Fengguang --

Re: [kbuild] fs/overlayfs/super.c:367 ovl_lookup() error: passing non negative 2147383647 to ERR_PTR

On Thu, Nov 13, 2014 at 08:22:21PM +0800, Fengguang Wu wrote: No problem. From now on the smatch I run here should be auto upgraded to your latest git HEAD. :) Oh wow... That's a lot of presure and I've traditionally sucked at QC. Right now, I feel like Smatch is at a stable place. I guess

Re: [kbuild] [PATCH] iio: adc: fix platform_no_drv_owner.cocci warnings (fwd)

On Thu, Nov 13, 2014 at 08:23:36PM +0800, Fengguang Wu wrote: Hi Julia, On Thu, Nov 13, 2014 at 11:26:23AM +0100, Julia Lawall wrote: Is there any way that we can improve this process? At the moment, I just take the message I get and forward it. I normally forward bug warnings if I

Re: [kbuild] [PATCH] iio: adc: fix platform_no_drv_owner.cocci warnings (fwd)

On Thu, Nov 13, 2014 at 01:45:39PM +0100, Julia Lawall wrote: But overall the problem is for the lower confidence ones how to get the patch to the developer in the form that they want. Is the idea that there are some lines that I should be removing (To, CC, and the file and line generated by

Re: [kbuild] [PATCH] iio: adc: fix platform_no_drv_owner.cocci warnings (fwd)

On Thu, Nov 13, 2014 at 02:12:33PM +0100, Julia Lawall wrote: On Thu, 13 Nov 2014, Dan Carpenter wrote: On Thu, Nov 13, 2014 at 01:45:39PM +0100, Julia Lawall wrote: But overall the problem is for the lower confidence ones how to get the patch to the developer in the form that they

Re: [kbuild] [PATCH] iio: adc: fix platform_no_drv_owner.cocci warnings (fwd)

On Thu, Nov 13, 2014 at 09:20:16PM +0800, Fengguang Wu wrote: I'm a robot, after all. ;) Heh. One time I was credited to as: Reported-by: The Intel Backend And I had to tell them that I might be a backend, but I work for Oracle. regards, dan carpenter

Re: [kbuild] [PATCH] iio: adc: fix platform_no_drv_owner.cocci warnings (fwd)

On Thu, Nov 13, 2014 at 04:29:15PM +0300, Dan Carpenter wrote: On Thu, Nov 13, 2014 at 09:20:16PM +0800, Fengguang Wu wrote: I'm a robot, after all. ;) Heh. One time I was credited to as: Reported-by: The Intel Backend And I had to tell them that I might be a backend, but I work for

[kbuild] [next:master 125/6487] drivers/net/wireless/ath/ath10k/wmi.c:3170 ath10k_wmi_start_scan_verify() warn: this array is probably non-NULL. 'arg-ie'

TO: Michal Kazior michal.kaz...@tieto.com CC: Kalle Valo kv...@adurom.com tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master head: bbdef57970d5e1887de755474ff1562baa17ef11 commit: a6aa5da3029670816b3eb9358bc1a63f2e2564fb [125/6487] ath10k: re-work scan start

[kbuild] [next:master 2788/6487] drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c:1073 brcmf_ops_sdio_probe() error: we previously assumed 'sdiodev-pdata' could be null (see line 1064)

TO: Hante Meuleman meule...@broadcom.com CC: John W. Linville linvi...@tuxdriver.com CC: Arend van Spriel ar...@broadcom.com tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master head: bbdef57970d5e1887de755474ff1562baa17ef11 commit:

[kbuild] [next:master 2918/6487] drivers/net/wireless/mac80211_hwsim.c:2745 hwsim_del_radio_nl() warn: can 'hwname' even be NULL?

TO: Jukka Rissanen jukka.rissa...@linux.intel.com CC: Johannes Berg johannes.b...@intel.com tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master head: bbdef57970d5e1887de755474ff1562baa17ef11 commit: 579a05f29611dcef94b6bda21f1d059ab189a263 [2918/6487]

[kbuild] [next:master 4226/6487] drivers/net/ethernet/amd/xgbe/xgbe-dev.c:1587 xgbe_dev_read() warn: missing break? reassigning 'packet-rss_hash_type'

TO: Lendacky, Thomas thomas.lenda...@amd.com CC: Tom Lendacky thomas.lenda...@amd.com tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master head: bbdef57970d5e1887de755474ff1562baa17ef11 commit: 5b9dfe299e55604af47cdca9d03d2e9d4fe2ad53 [4226/6487] amd-xgbe: Provide

[kbuild] [next:master 125/6487] drivers/net/wireless/ath/ath10k/wmi.c:3170 ath10k_wmi_start_scan_verify() warn: this array is probably non-NULL. 'arg-ie'

tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master head: bbdef57970d5e1887de755474ff1562baa17ef11 commit: a6aa5da3029670816b3eb9358bc1a63f2e2564fb [125/6487] ath10k: re-work scan start command building drivers/net/wireless/ath/ath10k/wmi.c:3170

Re: [kbuild] [next:master 2788/6487] drivers/net/wireless/brcm80211/brcmfmac/bcmsdh.c:1073 brcmf_ops_sdio_probe() error: we previously assumed 'sdiodev-pdata' could be null (see line 1064)

I reported this one earlier. regards, dan carpenter On Thu, Nov 13, 2014 at 10:08:36PM +0800, kbuild test robot wrote: TO: Hante Meuleman meule...@broadcom.com CC: John W. Linville linvi...@tuxdriver.com CC: Arend van Spriel ar...@broadcom.com tree:

Re: [kbuild] [next:master 200/6487] drivers/net/wireless/ath/ath10k/debug.c:1195 ath10k_debug_get_et_strings() error: __builtin_memcpy() '*ath10k_gstrings_stats' too small (32 vs 1472)

This one is sort of a false positive. The code works. I feel like it should be written: memcpy(data, ath10k_gstrings_stats, sizeof(ath10k_gstrings_stats)); But that's a style issue rather than a bug. regards, dan carpenter On Thu, Nov 13, 2014 at 10:04:33PM +0800, kbuild test robot

[kbuild] [hch-scsi:queue-depth-cleanups 78/90] drivers/infiniband/ulp/srp/ib_srp.c:2129 srp_cm_rep_handler() error: we previously assumed 'ch-rx_ring' could be null (see line 2108)

TO: Bart Van Assche bvanass...@acm.org CC: Christoph Hellwig h...@lst.de tree: git://git.infradead.org/users/hch/scsi.git queue-depth-cleanups head: 00868993ece2e5af973e0f60990f7781f5e03e88 commit: 509c07bc18500c3ded1a8e6273ace5002136c9d2 [78/90] IB/srp: Separate target and channel variables

[kbuild] [PATCH] PCI: layerscape: fix platform_no_drv_owner.cocci warnings

TO: Minghuan Lian minghuan.l...@freescale.com CC: Bjorn Helgaas bhelg...@google.com drivers/pci/host/pci-layerscape.c:170:3-8: No need to set .owner here. The core will do it. Remove .owner field if calls are used which set it automatically Generated by: