Re: [kbuild] [next:master 3648/3880] fs/eventfd.c:130 eventfd_poll() warn: impossible condition '(count == (~0)) = (0-u32max == u64max)'

2015-01-20 Thread Dan Carpenter
I sent a patch for this. regards, dan carpenter On Tue, Jan 20, 2015 at 09:13:11AM +0800, kbuild test robot wrote: TO: Chris Mason chris.ma...@fusionio.com Chris Mason c...@fb.com CC: Andrew Morton a...@linux-foundation.org CC: Linux Memory Management List linux...@kvack.org tree: git

Re: [kbuild] [next:master 1637/3880] drivers/net/wireless/ti/wlcore/main.c:1848 wl1271_op_suspend() warn: inconsistent returns 'mutex:wl-mutex'.

2015-01-19 Thread Dan Carpenter
This is a bug, but I sent a patch for it last week. regards, dan carpenter On Tue, Jan 20, 2015 at 02:52:38AM +0800, kbuild test robot wrote: TO: Ram Amrani ramr...@ti.com CC: Kalle Valo kv...@adurom.com Kalle Valo kv...@adurom.com CC: Arik Nemtsov a...@wizery.com CC: Eliad Peller el

[kbuild] [gpio:devel 22/26] drivers/gpio/gpiolib.c:1660:33: sparse: duplicate const

2015-01-15 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-gpio.git devel head: 19d54e21dfe3ac12c1fa687e0769ea1caff32e00 commit: e370430e0f3171abfbacc9162591d93631ffd9e8 [22/26] gpio:gpiolib: use static const char const * for a suffixes array reproduce: # apt-get install sparse git

[kbuild] [infiniband:for-next 11/36] drivers/infiniband/hw/ocrdma/ocrdma_verbs.c:683 ocrdma_alloc_pd() error: double free of 'pd'

2015-02-18 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/roland/infiniband.git for-next head: 2d79d42b11e84e5bf577a6ef0917ad799dfe610e commit: 72278f474221abd5c09af5d3a4dff39f9d10948f [11/36] RDMA/ocrdma: Move PD resource management to driver. New smatch warnings:

[kbuild] [infiniband:for-next 2/4] drivers/infiniband/hw/cxgb4/iw_cxgb4.h:232 c4iw_wait_for_reply() warn: KERN_* level not at start of string

2015-02-18 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/roland/infiniband.git for-next head: 2d79d42b11e84e5bf577a6ef0917ad799dfe610e commit: 718aefb1fbca79444ca8959843074761f54f68ed [2/4] RDMA/cxgb4: Don't hang threads forever waiting on WR replies New smatch warnings:

[kbuild] [block:mq-deadline 1/2] block/blk-mq-sysfs.c:250 blk_mq_hw_sysfs_dead_store() warn: impossible condition '(store (~0)) = (0-u64max u64max)'

2015-02-18 Thread Dan Carpenter
tree: git://git.kernel.dk/linux-block.git mq-deadline head: d74047eb4575f822801ec2de38c3f0ac8ede55b6 commit: ade03b11721ffbad2b062de1fb7a4b200aad76ef [1/2] blk-mq: first cut deadline scheduling block/blk-mq-sysfs.c:250 blk_mq_hw_sysfs_dead_store() warn: impossible condition '(store (~0)) =

Re: [kbuild] [ipmi:master-ipmi-rebase 31/32] drivers/char/ipmi/ipmi_serial.c:1977 ipmi_serial_setup_one() error: we previously assumed 'info-codec_name' could be null (see line 1938)

2015-02-20 Thread Dan Carpenter
The hash doesn't exist anymore. regards, dan carpenter On Fri, Feb 20, 2015 at 10:54:11AM +0800, kbuild test robot wrote: TO: Corey Minyard cminy...@mvista.com tree: git://git.code.sf.net/p/openipmi/linux-ipmi master-ipmi-rebase head: 8efd976ca62239dccea4f7d2be412205f86c2d1a commit

[kbuild] [spi:topic/fsl-dspi 2/2] drivers/spi/spi-fsl-dspi.c:364 dspi_setup() warn: possible memory leak of 'chip'

2015-01-28 Thread Dan Carpenter
[ Probably move the check for -bits_per_word in front of the allocation -dan] tree: git://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi topic/fsl-dspi head: 9298bc727385cd625cdda5e877bd3cba8acff668 commit: 9298bc727385cd625cdda5e877bd3cba8acff668 [2/2] spi: spi-fsl-dspi: Remove

[kbuild] [pci:pci/virtualization 10/37] drivers/pci/iov.c:425 sriov_init() warn: possible memory leak of 'iov'

2015-01-31 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git pci/virtualization head: 42a22ecf97273f83a0f34ff4510984524aac3231 commit: 7be5aed243b8b901d5772eb89fe410206d3a9b21 [10/37] PCI: Store individual VF BAR size in struct pci_sriov drivers/pci/iov.c:425 sriov_init() warn:

[kbuild] [ath6kl:ath-next-test 104/113] drivers/net/wireless/ath/ath10k/wmi.c:969 ath10k_wmi_tx_beacon_nowait() warn: variable dereferenced before check 'bcn' (see line 967)

2015-01-24 Thread Dan Carpenter
tree: git://github.com/kvalo/ath ath-next-test head: 59030a70b61c41268383d1406bb49fc559e0da4e commit: 60845cf1bc104bbec7509eb4b6c9f09a1559bfdc [104/113] ath10k: fix beacon deadlock New smatch warnings: drivers/net/wireless/ath/ath10k/wmi.c:969 ath10k_wmi_tx_beacon_nowait() warn: variable

[kbuild] [iio:testing 20/20] drivers/iio/light/cm3232.c:354 cm3232_probe() warn: this array is probably non-NULL. 'id-name'

2015-01-25 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git testing head: aa6799c6df77b76230bd150a668be920eba1ebcd commit: aa6799c6df77b76230bd150a668be920eba1ebcd [20/20] iio: Added Capella cm3232 ambient light sensor driver. drivers/iio/light/cm3232.c:354 cm3232_probe() warn: this

[kbuild] [staging:staging-testing 369/370] drivers/staging/fbtft/fbtft_device.c:1403 fbtft_device_init() warn: variable dereferenced before check 'pdata' (see line 1350)

2015-01-26 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing head: 484c60e206b03fe19129dc6ec4582142293cd876 commit: 06d30f0208576f3cf2a0ee2072c1408e392038b4 [369/370] staging: fbtft: add fbtft_device driver drivers/staging/fbtft/fbtft_device.c:1403

[kbuild] [jtkirshe-net-next:i40e-queue 18/25] drivers/net/ethernet/intel/i40evf/i40e_txrx.c:1036 i40e_clean_rx_irq_ps() error: we previously assumed 'skb' could be null (see line 1033)

2015-01-26 Thread Dan Carpenter
tree: jtkirshe-net-next/i40e-queue head: 90811b2dd89f7a28239a79d4b259c425f43fe152 commit: 785fb2d0619c37ffe7c2b5ae5835be0caa97701f [18/25] i40e/i40evf: Refactor the receive routines drivers/net/ethernet/intel/i40evf/i40e_txrx.c:1036 i40e_clean_rx_irq_ps() error: we previously assumed 'skb'

[kbuild] [pinchartl-fbdev:dma/iommu/debug 9/17] drivers/dma/dmatest.c:273 dmatest_run() warn: we never enter this loop

2015-01-26 Thread Dan Carpenter
[ Probably this is intentional to handle some upcoming future code. -dan ] tree: git://linuxtv.org/pinchartl/fbdev.git dma/iommu/debug head: be0c1a1fcc5215116b86c9de3a7a5d1a22b3fe14 commit: f33a7d5ca85521588032fb047019d65969105aea [9/17] [DEBUG] dmaengine: dmatest: IOMMU TLB test

[kbuild] [mmotm:master 200/417] mm/slab_common.c:166 update_memcg_params() warn: variable dereferenced before check 'old' (see line 162)

2015-01-26 Thread Dan Carpenter
tree: git://git.cmpxchg.org/linux-mmotm.git master head: c64429bcc60a702f19f5cfdb5c39277863278a8c commit: 5d06629c100b942a51f02b4d886c116ba3afb32a [200/417] slab: embed memcg_cache_params to kmem_cache mm/slab_common.c:166 update_memcg_params() warn: variable dereferenced before check 'old'

[kbuild] [staging:staging-testing 342/370] drivers/staging/fbtft/fbtft-core.c:1101 fbtft_init_display_dt() error: snprintf() is printing too much 128 vs 16

2015-01-26 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing head: 484c60e206b03fe19129dc6ec4582142293cd876 commit: c296d5f9957c03994a699d6739c27d4581a9f6c7 [342/370] staging: fbtft: core support drivers/staging/fbtft/fbtft-core.c:1101 fbtft_init_display_dt()

Re: [kbuild] [dhowells-fs:file-pin 106/116] fs/cachefiles/namei.c:704 cachefiles_walk_to_object() warn: inconsistent returns 'mutex:dentry_inodedir-i_mutex'.

2015-01-23 Thread Dan Carpenter
This one is ok I think. Anyway, it's not introduced in this patch which just renames things. regards, dan carpenter On Fri, Jan 23, 2015 at 03:03:47PM +0800, kbuild test robot wrote: TO: David Howells dhowe...@redhat.com tree: git://git.kernel.org/pub/scm/linux/kernel/git/dhowells/linux

[kbuild] drivers/net/ethernet/chelsio/cxgb4/clip_tbl.c:252 clip_tbl_show() warn: %pI4 can only be followed by one of [hnbl], not c

2015-02-13 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: c7d7b98671552abade78834c522b7308bda73c0d commit: b5a02f503caa0837b64907468359b075990afcce cxgb4 : Update ipv6 address handling api drivers/net/ethernet/chelsio/cxgb4/clip_tbl.c:252 clip_tbl_show() warn: %pI4

Re: [kbuild] [next:master 7695/9788] drivers/staging/i2o/i2o_config.c:255 i2o_cfg_swdl() warn: check for integer overflow 'swlen'

2015-02-15 Thread Dan Carpenter
This code is scary from a security perspective. It's old code, we just moved it to staging so we could delete it in a later kernel release. So I'm not going to bother with this warning. regards, dan carpenter On Mon, Feb 16, 2015 at 06:04:22AM +0800, kbuild test robot wrote: TO: Alan Cox

[kbuild] [next:master 4863/5749] drivers/hid/usbhid/hid-pidff.c:577 pidff_upload_effect() error: we previously assumed 'effect' could be null (see line 572)

2015-03-20 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master head: ce040cc379e3e1e094db5559f393d794fbfd01e4 commit: e8f46e4f93f52b2bcc681cf8bc3ff84a22a2d356 [4863/5749] HID: pidff: support more than one concurrent effect drivers/hid/usbhid/hid-pidff.c:577

[kbuild] [next:master 956/5749] drivers/net/bonding/bond_3ad.c:1153 ad_churn_machine() warn: inconsistent indenting

2015-03-20 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master head: ce040cc379e3e1e094db5559f393d794fbfd01e4 commit: 14c9551a32eba086c9f20c9d6a8e378481f15333 [956/5749] bonding: Implement port churn-machine (AD standard 43.4.17). New smatch warnings:

Re: [kbuild] [next:master 2689/5994] net/ipv4/fib_trie.c:327 leaf_new() warn: variable dereferenced before check 'kv' (see line 325)

2015-03-20 Thread Dan Carpenter
I reported this one earlier, it looks real but it turns out is it a false positive. kv-kv is an array so the assignment l = kv-kv is not a dereference. regards, dan carpenter On Fri, Mar 20, 2015 at 11:27:15PM +0800, kbuild test robot wrote: TO: Alexander Duyck alexander.h.du...@redhat.com

[kbuild] [drm-intel:for-linux-next 136/140] drivers/gpu/drm/i915/i915_gem_gtt.c:2559 intel_rotate_fb_obj_pages() error: passing non negative 12 to ERR_PTR

2015-03-24 Thread Dan Carpenter
[ Heh... Hiding the wrong value in a variable make the bug hard to spot. Tricky. :). When you fix this, please remove the intialization entirely. - dan ] tree: git://anongit.freedesktop.org/drm-intel for-linux-next head: 096fad9ebfadb4922332f98895ea948e8d7fe6a7 commit:

[kbuild] [lunn:v4.0-rc5-groeck-dsa-next-cleanup 9/12] drivers/net/dsa/mv88e6xxx.c:722 mv88e6xxx_get_eee() warn: inconsistent returns 'mutex:ps-phy_mutex'.

2015-03-30 Thread Dan Carpenter
tree: git://github.com/lunn/linux.git v4.0-rc5-groeck-dsa-next-cleanup head: e92f5528f18e3388133389109f072416ad0a9495 commit: cb7ba0eae2f0c2bb7de94dc4dad2caf4f496a0a0 [9/12] net: dsa: mv88e6xxx: Add missing mutex's in EEE operations. drivers/net/dsa/mv88e6xxx.c:722 mv88e6xxx_get_eee() warn:

[kbuild] [scsi:misc 7/8] drivers/scsi/ufs/ufshcd.c:4386 ufshcd_query_ioctl() warn: maybe return -EFAULT instead of the bytes remaining?

2015-03-30 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git misc head: d0b68ffe7bf1689bfab3c31407256b31157e0b91 commit: 4aca8e8975dbdf879c4ec4ff143c65fecf0370f8 [7/8] ufs: add ioctl interface for query request New smatch warnings: drivers/scsi/ufs/ufshcd.c:4386 ufshcd_query_ioctl()

[kbuild] [hch-scsi:target-rcu-hch 16/19] drivers/target/target_core_device.c:1249 core_dev_add_initiator_node_lun_acl() error: we previously assumed 'deve' could be null (see line 1244)

2015-03-30 Thread Dan Carpenter
tree: git://git.infradead.org/users/hch/scsi.git target-rcu-hch head: b1ab96e3c174ea7f9a17554fb93b86c9a49a7ccf commit: d76cb9ffabc9ba62e5a1cce723fe3528a6975783 [16/19] target: convert device list to dynamic allocation and RCU lookup drivers/target/target_core_device.c:1249

Re: [kbuild] [spi:topic/pxa2xx 3/3] drivers/spi/spi-pxa2xx.c:795 pxa2xx_ssp_get_clk_div() warn: missing break? reassigning 'clk_div'

2015-03-30 Thread Dan Carpenter
I sent a patch for this. regards, dan carpenter On Thu, Mar 26, 2015 at 01:26:43AM +0800, kbuild test robot wrote: TO: Andy Shevchenko andriy.shevche...@linux.intel.com CC: Mark Brown broo...@kernel.org tree: git://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi topic/pxa2xx head

[kbuild] [ath6kl:pending 162/174] drivers/net/wireless/ath/ath10k/mac.c:2163 ath10k_peer_assoc_h_ht() warn: buffer overflow 'vht_mcs_mask' 8 = 9

2015-03-30 Thread Dan Carpenter
tree: git://github.com/kvalo/ath pending head: ce4865fdc78680864853e97d9d523f8a3c955906 commit: 385fd8140be0f7d1ce559fb089b2aa9cc27287b7 [162/174] ath10k: implement more versatile set_bitrate_mask drivers/net/wireless/ath/ath10k/mac.c:2163 ath10k_peer_assoc_h_ht() warn: buffer overflow

[kbuild] [rogerq:for-v4.2/usb-drd-working 10/25] drivers/usb/common/usb-otg.c:382 usb_otg_unregister() warn: inconsistent returns 'mutex:otg_list_mutex'.

2015-03-31 Thread Dan Carpenter
tree: git://github.com/rogerq/linux for-v4.2/usb-drd-working head: 127f6b4e45df51a4e299e38f1895343471079a67 commit: 660b95cec3ce2f513489ad595702a9c3087f5ee1 [10/25] usb: otg: add OTG core drivers/usb/common/usb-otg.c:382 usb_otg_unregister() warn: inconsistent returns 'mutex:otg_list_mutex'.

Re: [kbuild] [bp:tip-x86-ras 1/2] arch/x86/kernel/cpu/mcheck/mce-severity.c:214 mce_severity_amd() warn: curly braces intended?

2015-03-31 Thread Dan Carpenter
MCI_STATUS_OVER) 218 return MCE_PANIC_SEVERITY; 219 } regards, dan carpenter ___ kbuild mailing list kbuild@lists.01.org https://lists.01.org/mailman/listinfo/kbuild

Re: [kbuild] [bp:tip-x86-ras 1/2] arch/x86/kernel/cpu/mcheck/mce-severity.c:210:4-31: code aligned with following code on line 214

2015-03-31 Thread Dan Carpenter
On Tue, Mar 31, 2015 at 07:03:15PM +0300, Dan Carpenter wrote: On Tue, Mar 31, 2015 at 06:16:46PM +0300, Dan Carpenter wrote: I added a check for this in Smatch and I sent that warning already. It's a bit duplicative, sorry. I have done a little more research into the differences

Re: [kbuild] [bp:tip-x86-ras 1/2] arch/x86/kernel/cpu/mcheck/mce-severity.c:210:4-31: code aligned with following code on line 214

2015-03-31 Thread Dan Carpenter
is probably the same. There is only one other one like this I think. The others issues are all different. if statement vs while loop. #ifdef in the middle resets the alignment else statements seem not handled? regards, dan carpenter ___ kbuild mailing

Re: [kbuild] [bp:tip-x86-ras 1/2] arch/x86/kernel/cpu/mcheck/mce-severity.c:210:4-31: code aligned with following code on line 214

2015-03-31 Thread Dan Carpenter
I added a check for this in Smatch and I sent that warning already. It's a bit duplicative, sorry. regards, dan carpenter On Tue, Mar 31, 2015 at 11:07:21PM +0800, kbuild test robot wrote: TO: Aravind Gopalakrishnan aravind.gopalakrish...@amd.com CC: Borislav Petkov b...@suse.de tree

[kbuild] [bp:tip-x86-ras 1/2] arch/x86/kernel/cpu/mcheck/mce-severity.c:214 mce_severity_amd() warn: curly braces intended?

2015-03-31 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/bp/bp tip-x86-ras head: 43eaa2a1ad70d72876cdbb2eb5450a2665e4770f commit: bf80bbd7dcf525e41e0673fbaa8cd21d2344b460 [1/2] x86/mce: Add an AMD severities-grading function arch/x86/kernel/cpu/mcheck/mce-severity.c:214 mce_severity_amd() warn:

[kbuild] [amirv:for-upstream 113/139] drivers/infiniband/ulp/ipoib/ipoib_ib.c:449 ipoib_tx_poll() warn: inconsistent indenting

2015-03-23 Thread Dan Carpenter
Hi Fengguang, do you think you could make these inconsistent indenting warnings go out automatically? They are very low false positive. Ideally, they would be caught by checkpatch.pl but you need quite good C parsing to do it correctly so checkpatch.pl can't do it. tree:

[kbuild] [gpio:devel 7/10] drivers/gpio/devres.c:124:33: sparse: duplicate const

2015-03-05 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-gpio.git devel head: 331758eef83620eef3f21289f0f44aba094d0503 commit: 1feb57a245a4910b03202a814ffc51a900bd4aca [7/10] gpio: add parameter to allow the use named gpios reproduce: # apt-get install sparse git checkout

[kbuild] [next:master 10486/10701] drivers/md/dm-cache-policy-mq.c:168 is_sentinel() warn: potential pointer math issue ('q-sentinels' is a 64 bit pointer)

2015-02-22 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master head: 53e418f1e9f4d409cee84e573e510877e343ce38 commit: 3c513cb6090b18b7f2d7db918208de7054751fa7 [10486/10701] dm cache policy mq: track entries hit this 'tick' via sentinel objects

[kbuild] [linuxtv-media:master 348/454] drivers/media/dvb-core/dvb_frontend.c:861 dvb_frontend_thread() warn: inconsistent returns 'sem:fepriv-sem'.

2015-03-24 Thread Dan Carpenter
tree: git://linuxtv.org/media_tree.git master head: 8a56b6b5fd6ff92b7e27d870b803b11b751660c2 commit: 135f9be9194cf7778eb73594aa55791b229cf27c [348/454] [media] dvb_frontend: start media pipeline while thread is running New smatch warnings: drivers/media/dvb-core/dvb_frontend.c:861

[kbuild] [drm-intel:topic/atomic-core 5/8] drivers/gpu/drm/drm_atomic_helper.c:1939 drm_atomic_helper_connector_dpms() error: 'crtc_state' dereferencing possible ERR_PTR()

2015-01-23 Thread Dan Carpenter
tree: git://anongit.freedesktop.org/drm-intel topic/atomic-core head: 1cf729872a4cdad2b251e0058b623318f3c5997f commit: 35b219a1ca6ac3a89ee06dae5b4e73900eeab94c [5/8] drm/atomic-helper: add connector-dpms() implementation drivers/gpu/drm/drm_atomic_helper.c:1939

[kbuild] [dhowells-fs:file-pin 34/116] fs/configfs/dir.c:1636 configfs_dir_lseek() warn: inconsistent returns 'mutex:fs_inodedentry-i_mutex'.

2015-01-23 Thread Dan Carpenter
[ An error path needs to be updated as well. -dan ] tree: git://git.kernel.org/pub/scm/linux/kernel/git/dhowells/linux-fs file-pin head: 576999e31feae9640540ef9e7dd1d37d0af78794 commit: 188ea723f402164b7aea09d9bd430639d5a4b218 [34/116] VFS: (Scripted) Convert -d_inode to fs_inode() in

[kbuild] [jes:rtl8723au-mac80211 1071/1079] drivers/net/wireless/rtl8xxxu.c:3670 rtl8xxxu_tx() warn: variable dereferenced before check 'control' (see line 3669)

2015-03-23 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/jes/linux.git rtl8723au-mac80211 head: 63eab597e116bb17f59229fee1a81439aa43ffc4 commit: d2183e7307a5f8931db91387ac43ffa07a1299fa [1071/1079] rtl8xxxu: re-enable ampdu or we get no speed . drivers/net/wireless/rtl8xxxu.c:3670

[kbuild] [target:target-rcu-wip 25/38] drivers/target/target_core_pr.c:2022 core_scsi3_emulate_pro_register() warn: variable dereferenced before check 'se_sess' (see line 2011)

2015-03-23 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/nab/target-pending.git target-rcu-wip head: a8635dd67a968941e2b90f9b81875228af526ba4 commit: e75921427f7cca55fae9e13998bd330a8073c1f3 [25/38] target: Convert se_dev_entry to RCU + percpu-refcount for PR drivers/target/target_core_pr.c:2022

[kbuild] [cgroup:review-cgroup-writeback-switch-20150322 22/75] mm/page-writeback.c:2108 account_page_dirtied() error: we previously assumed 'inode' could be null (see line 2104)

2015-03-23 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/tj/cgroup.git review-cgroup-writeback-switch-20150322 head: 16fc85e89229e50170c78deb83a56a7c9c7782e4 commit: edf58a36b8fac8a463d81e4d2c97d34176ac16bb [22/75] writeback: make backing_dev_info host cgroup-specific bdi_writebacks

[kbuild] [radeon-alex:drm-next-4.1 2/40] drivers/gpu/drm/radeon/radeon_connectors.c:741 radeon_connector_set_property() error: we previously assumed 'connector-encoder' could be null (see line 729)

2015-03-23 Thread Dan Carpenter
tree: git://people.freedesktop.org/~agd5f/linux.git drm-next-4.1 head: 9843ead08f18270948498f37eb95d6189eed31af commit: 643b1f5629a30f979e2da678ecf0b1855e074a8c [2/40] drm/radeon: implement output csc property for DCE5+ drivers/gpu/drm/radeon/radeon_connectors.c:741

[kbuild] [rrnayak:tsens 9/11] drivers/thermal/qcom_tsens_b.c:284 tsens_calib_8974_sensors() warn: mask and shift to zero

2015-03-23 Thread Dan Carpenter
tree: git://github.com/rrnayak/linux tsens head: b78182117bf4511a14200e8c4a9e0052b58ec4c5 commit: 456e16feb785ec3ec71c10490ae0cce2549d5aec [9/11] thermal: Add msm tsens thermal sensor driver for B family drivers/thermal/qcom_tsens_b.c:284 tsens_calib_8974_sensors() warn: mask and shift to

[kbuild] [drm-intel:for-linux-next 120/130] drivers/gpu/drm/i915/i915_gem_context.c:720 do_switch() error: we previously assumed 'to-ppgtt' could be null (see line 711)

2015-03-23 Thread Dan Carpenter
tree: git://anongit.freedesktop.org/drm-intel for-linux-next head: 0b37a9a9ebdf47dc9d5290a1c69bd944dde5cc15 commit: 563222a745012cc2bb20c5d5cbff1c1ec4832c05 [120/130] drm/i915: Track page table reload need drivers/gpu/drm/i915/i915_gem_context.c:720 do_switch() error: we previously assumed

Re: [kbuild] [cgroup:review-cgroup-writeback-switch-20150322 69/75] fs/fs-writeback.c:908 writeback_single_inode() error: double lock 'spin_lock:inode-i_lock'

2015-03-23 Thread Dan Carpenter
Tree doesn't exist any more. No idea. regards, dan carpenter On Mon, Mar 23, 2015 at 11:29:36AM +0800, kbuild test robot wrote: TO: Tejun Heo t...@kernel.org tree: git://git.kernel.org/pub/scm/linux/kernel/git/tj/cgroup.git review-cgroup-writeback-switch-20150322 head

[kbuild] [amirv:for-upstream 10/124] drivers/infiniband/hw/ocrdma/ocrdma_ah.c:132 ocrdma_create_ah() error: we previously assumed 'sgid_attr.ndev' could be null (see line 125)

2015-03-23 Thread Dan Carpenter
tree: git://flatbed.openfabrics.org/~amirv/linux.git for-upstream head: ec4df3c1aabdca45b165923c4687cb97ac6d00b7 commit: d32cb6dcd6ccf9901d3ffdff377a3abc78af9863 [10/124] IB/core: Modify ib_verbs and cma in order to use roce_gid_cache [net-next:roce_per_gid_net-next]

[kbuild] [drm-intel:drm-intel-next-queued 133/206] drivers/gpu/drm/i915/i915_gem.c:4226 i915_gem_object_ggtt_unpin_view() error: 'vma' dereferencing possible ERR_PTR()

2015-04-02 Thread Dan Carpenter
[ i915_gem_obj_to_ggtt_view() only returns an error pointer when view is NULL. None of the callers ever pass a NULL as view so it never really returns an ERR_PTR. But the other callers check. It would be easy enough to silence this warning in Smatch and then turn it on where there is a

Re: [kbuild] drivers/video/fbdev/core/fbmem.c:1674 do_register_framebuffer() error: buffer overflow 'registered_fb' 32 = 32

2015-04-02 Thread Dan Carpenter
False positive. But the old warnings are probably real bugs. regards, dan carpenter On Thu, Apr 02, 2015 at 09:42:48AM +0800, kbuild test robot wrote: TO: Tomi Valkeinen tomi.valkei...@ti.com tree: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head

[kbuild] [slave-dma:for-linus 25/31] drivers/dma/sh/usb-dmac.c:294 usb_dmac_desc_free() error: dereferencing freed memory 'desc'

2015-04-03 Thread Dan Carpenter
[ This should be list_for_each_entry_safe() - dan ] tree: git://git.kernel.org/pub/scm/linux/kernel/git/vkoul/slave-dma.git for-linus head: 453dcdb5ba048011d823cd31710f78ce2fd57da3 commit: 0c1c8ff32fa29e425b4938934c21afdb81104431 [25/31] dmaengine: usb-dmac: Add Renesas USB DMA Controller

[kbuild] [mlin:tcm_aoe 1/1] drivers/target/tcm_aoe/aoe_conf.c:109 tcm_aoe_add_tpg() warn: impossible condition '(index (~0)) = (0-u32max u32max)'

2015-04-03 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/mlin/linux.git tcm_aoe head: ad41db98d967fc9aef6d9ce47755aa61e5b21010 commit: ad41db98d967fc9aef6d9ce47755aa61e5b21010 [1/1] tcm_aoe: Initial skeleton commit drivers/target/tcm_aoe/aoe_conf.c:109 tcm_aoe_add_tpg() warn: impossible

[kbuild] [vfs:for-linus 92/109] fs/cachefiles/namei.c:704 cachefiles_walk_to_object() warn: inconsistent returns 'mutex:d_inodedir-i_mutex'.

2015-04-25 Thread Dan Carpenter
[ It is complaining because we don't unlock on the goto delete_error; path. - dan ] tree: git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs.git for-linus head: 3cab989afd8d8d1bc3d99fef0e7ed87c31e7b647 commit: 5153bc817cdbed826a18938393cc1f81ebbbd898 [92/109] VFS: Cachefiles should

[kbuild] [jkirsher-next-queue:dev-queue 17/35] drivers/net/ethernet/intel/i40e/i40e_ethtool.c:1557 i40e_active_vfs() warn: bit shifter 'I40E_VF_STAT_ACTIVE' used for logical ''

2015-04-25 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/jkirsher/next-queue.git dev-queue head: b0dc432053ae847f7f26ac327347c11bc9ab4c22 commit: 36a6f34625001d9e43c596e0dc0534ae69ec034b [17/35] i40e: Disable offline diagnostics if VFs are enabled

Re: [kbuild] [vfs:for-linus 92/109] fs/cachefiles/namei.c:704 cachefiles_walk_to_object() warn: inconsistent returns 'mutex:d_inodedir-i_mutex'.

2015-04-25 Thread Dan Carpenter
On Sat, Apr 25, 2015 at 10:16:32AM +0100, David Howells wrote: Dan Carpenter dan.carpen...@oracle.com wrote: [ It is complaining because we don't unlock on the goto delete_error; cachefiles_bury_object() should unlock the mutex prior to getting to delete_error. See the comment ~10 lines

[kbuild] [leds:for-next 6/10] drivers/leds/leds-max77693.c:193 max77693_add_mode() warn: maybe use instead of

2015-04-29 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/cooloney/linux-leds.git for-next head: 78a435c5a48401c6a99a8928e4f0eb1f865a0401 commit: 5abfad78ce114c9539f079d0151c69d74095d03c [6/10] leds: Add support for max77693 mfd flash cell drivers/leds/leds-max77693.c:193 max77693_add_mode()

[kbuild] [amirv:for-upstream 18/25] drivers/net/ethernet/mellanox/mlx4/main.c:796 mlx4_slave_special_qp_cap() warn: possible memory leak of 'func_cap'

2015-04-29 Thread Dan Carpenter
tree: git://flatbed.openfabrics.org/~amirv/linux.git for-upstream head: df1f74f1e2b01b3a01dedbc1603d4d20eca8ec4b commit: 3701245d8cc63dfbe0b307684ef999d40bbdb590 [18/25] net/mlx4_core: Dynamicaly allocate structs at mlx4_slave_cap [net-next:reduce_stack_size]

[kbuild] [thierryreding:staging/work 128/189] drivers/video/backlight/pwm_bl.c:279 pwm_backlight_probe() warn: impossible condition '(IS_ERR(pb-pwm) == -517) = (0-1 == (-517))'

2015-04-30 Thread Dan Carpenter
tree: https://github.com/thierryreding/linux staging/work head: 9a217612ee957a8a1fc06cb74e3a62b38836d053 commit: e7942c680de039923ad4d9665b08944c6386fb4b [128/189] video: backlight: pwm: Support deferred probe New smatch warnings: drivers/video/backlight/pwm_bl.c:279 pwm_backlight_probe()

[kbuild] [radeon-alex:amdgpu 35/45] drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c:356 amdgpu_info_ioctl() warn: check that 'gds_info._pad' doesn't leak information

2015-04-30 Thread Dan Carpenter
[ People will file a CVE for this if it hits mainline because this is the world we live in today with the hyperventilating about security. - dan] tree: git://people.freedesktop.org/~agd5f/linux.git amdgpu head: 9998c5af00d3d864ce272c0a41164a7f515d2ab5 commit:

Re: [kbuild] [jkirsher-next-queue:dev-queue 17/35] drivers/net/ethernet/intel/i40e/i40e_ethtool.c:1557 i40e_active_vfs() warn: bit shifter 'I40E_VF_STAT_ACTIVE' used for logical ''

2015-04-27 Thread Dan Carpenter
On Mon, Apr 27, 2015 at 02:24:52PM +, Rose, Gregory V wrote: -Original Message- From: Dan Carpenter [mailto:dan.carpen...@oracle.com] Sent: Monday, April 27, 2015 7:19 AM To: Rose, Gregory V Cc: kbu...@01.org; Kirsher, Jeffrey T Subject: Re: [jkirsher-next-queue:dev

[kbuild] [radeon-alex:amdgpu 35/42] drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c:116 amdgpu_driver_load_kms() warn: if statement not indented

2015-04-27 Thread Dan Carpenter
tree: git://people.freedesktop.org/~agd5f/linux.git amdgpu head: e29109dae234c4a38cbc0ca4e35d854fd44f0090 commit: e6c1131916995fbc253b3c050692cc278bf1c011 [35/42] drm/amdgpu: add core driver (v3) drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c:116 amdgpu_driver_load_kms() warn: if statement not

[kbuild] [dledford:for-4.1 3/13] drivers/infiniband/hw/cxgb4/mem.c:931 c4iw_alloc_fastreg_pbl() error: '%pad' expects argument of type 'dma_addr_t*', argument 6 has type 'void*'

2015-04-28 Thread Dan Carpenter
tree: git://github.com/dledford/linux.git for-4.1 head: 4e9467eb2bbb5ab099448f4485bb96b93466a209 commit: 20dca80fcb0caea966bd21d0ba2c2ab0cd54ca86 [3/13] iw_cxgb4: 32b platform fixes drivers/infiniband/hw/cxgb4/mem.c:931 c4iw_alloc_fastreg_pbl() error: '%pad' expects argument of type

Re: [kbuild] drivers/gpu/drm/i915/intel_display.c:11346 intel_crtc_set_config() error: we previously assumed 'pipe_config' could be null (see line 11330)

2015-05-04 Thread Dan Carpenter
I sent this one earlier. regards, dan carpenter On Mon, May 04, 2015 at 03:41:42PM +0800, kbuild test robot wrote: TO: Jesse Barnes jbar...@virtuousgeek.org CC: Daniel Vetter daniel.vet...@ffwll.ch CC: Ander Conselvan de Oliveira ander.conselvan.de.olive...@intel.com tree: git

Re: [kbuild] [staging:staging-testing 106/157] drivers/staging/android/ion/ion_chunk_heap.c:176 ion_chunk_heap_create() error: '%pad' expects argument of type 'dma_addr_t*', argument 3 has type 'ulong

2015-05-04 Thread Dan Carpenter
Smatch was correct, but when I reviewed the original patch there were other issues as well so I sent a different email to thread. regards, dan carpenter On Mon, May 04, 2015 at 05:14:50AM +0800, kbuild test robot wrote: TO: Mitchel Humpherys mitch...@codeaurora.org CC: Greg Kroah-Hartman gre

[kbuild] [net-next:master 8/17] drivers/net/ethernet/apm/xgene/xgene_enet_ring2.c:43 xgene_enet_ring_init() error: buffer overflow 'ring_cfg' 5 = 5

2015-04-30 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git master head: 327941f8d322685633f74fb43dc3aec86529f5f4 commit: ed9b7da019ea01dcead9f1d231bf03ab033f5f9c [8/17] drivers: net: xgene: Add ring manager v2 functions drivers/net/ethernet/apm/xgene/xgene_enet_ring2.c:43

Re: [kbuild] [pinctrl:devel 2/3] drivers/pinctrl/pinctrl-lpc18xx.c:913 lpc18xx_pconf_set() error: buffer overflow 'lpc18xx_pins' 182 = 182

2015-05-07 Thread Dan Carpenter
more developed (like on my other test system) then it would generate a warning. regards, dan carpenter ___ kbuild mailing list kbuild@lists.01.org https://lists.01.org/mailman/listinfo/kbuild

Re: [kbuild] [aa:userfault 33/44] fs/userfaultfd.c:551 userfaultfd_read() warn: check that 'msg' doesn't leak information (struct has a hole after 'flags')

2015-05-07 Thread Dan Carpenter
False positive. regards, dan carpenter On Thu, May 07, 2015 at 11:48:00AM +0800, kbuild test robot wrote: TO: Andrea Arcangeli aarca...@redhat.com tree: git://git.kernel.org/pub/scm/linux/kernel/git/andrea/aa.git userfault head: a164f776b1896ea4acb4c666f1327d7fb2e62159 commit

Re: [kbuild] [drm-intel:for-linux-next 397/433] drivers/gpu/drm/i915/i915_debugfs.c:4035 i915_displayport_test_active_show() warn: variable dereferenced before check 'dev' (see line 4032)

2015-05-07 Thread Dan Carpenter
These are all false positives. I have a patch to make these go away. regards, dan carpenter On Thu, May 07, 2015 at 10:09:21PM +0800, kbuild test robot wrote: TO: Todd Previte tprev...@gmail.com CC: Daniel Vetter daniel.vet...@ffwll.ch CC: Paulo Zanoni paulo.r.zan...@intel.com tree

Re: [kbuild] [tip:tmp.fpu 194/208] arch/x86/kernel/fpu/signal.c:89 save_xstate_epilog() warn: maybe return -EFAULT instead of the bytes remaining?

2015-05-05 Thread Dan Carpenter
This was a deliberate for speed. We only care about zero on success and non-zero, not the actual error. regards, dan carpenter On Wed, May 06, 2015 at 01:09:56AM +0800, kbuild test robot wrote: TO: Ingo Molnar mi...@kernel.org tree: git://git.kernel.org/pub/scm/linux/kernel/git/tip

[kbuild] [pinctrl:devel 2/3] drivers/pinctrl/pinctrl-lpc18xx.c:913 lpc18xx_pconf_set() error: buffer overflow 'lpc18xx_pins' 182 = 182

2015-05-05 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-pinctrl.git devel head: 6208b882c710d69fd706e39fd271ce3906620d8c commit: 3aba09daf3d4a79fe92db304b30868b7f868844c [2/3] pinctrl: add lpc18xx pinctrl driver drivers/pinctrl/pinctrl-lpc18xx.c:913 lpc18xx_pconf_set() error:

[kbuild] [net-next:master 150/158] drivers/net/ethernet/emulex/benet/be_main.c:4784:24: sparse: odd constant _Bool cast (ffffffffffffffff becomes 1)

2015-05-09 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git master head: b063bc5ea77b1c1c0e7798f641f53504d0f64bf8 commit: a6e6ff6eee12f3edc7630e81993bf880a4f0d340 [150/158] be2net: simplify UFI compatibility checking reproduce: # apt-get install sparse git checkout

[kbuild] [scsi:misc 14/106] drivers/scsi/storvsc_drv.c:1658 storvsc_queuecommand() warn: curly braces intended?

2015-04-13 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git misc head: 0351b8f81392c6d036e5c8f73ceff68726e9 commit: be0cf6ca301c61458dc4aa1a37acf4f58d2ed3d6 [14/106] scsi: storvsc: Set the tablesize based on the information given by the host drivers/scsi/storvsc_drv.c:1658

[kbuild] [iio:testing 5/9] drivers/iio/temperature/mlx90614.c:189:71: sparse: duplicate const

2015-04-13 Thread Dan Carpenter
[ You want const struct foo * const bar, I think - dan ] tree: git://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git testing head: abda2b4f21249a079b83ed3356937fa6d9faa789 commit: fad65a8fe5b85b5039b316258c2790e773cc3502 [5/9] iio: mlx90614: Add emissivity setting reproduce: #

[kbuild] [scsi:misc 23/106] drivers/scsi/aacraid/aachba.c:476 get_container_name_callback() error: buffer overflow 'sp' 16 = 16

2015-04-13 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git misc head: 0351b8f81392c6d036e5c8f73ceff68726e9 commit: b836439faf04b16c80ffd9a0699a2354ebde13e1 [23/106] aacraid: 4KB sector support New smatch warnings: drivers/scsi/aacraid/aachba.c:476 get_container_name_callback()

[kbuild] [kvm:queue 24/31] arch/x86/kvm/lapic.c:696 kvm_irq_delivery_to_apic_fast() error: we previously assumed 'src' could be null (see line 691)

2015-04-13 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/virt/kvm/kvm.git queue head: ca3f0874723fad81d0c701b63ae3a17a408d5f25 commit: 9ea369b032d87b88f1a47187b51ad4321dea5766 [24/31] KVM: x86: fix mixed APIC mode broadcast arch/x86/kvm/lapic.c:696 kvm_irq_delivery_to_apic_fast() error: we previously assumed

Re: [kbuild] [kvm:queue 24/31] arch/x86/kvm/lapic.c:696 kvm_irq_delivery_to_apic_fast() error: we previously assumed 'src' could be null (see line 691)

2015-04-13 Thread Dan Carpenter
On Mon, Apr 13, 2015 at 02:24:26PM +0200, Radim Krčmář wrote: 2015-04-13 14:52+0300, Dan Carpenter: tree: git://git.kernel.org/pub/scm/virt/kvm/kvm.git queue head: ca3f0874723fad81d0c701b63ae3a17a408d5f25 commit: 9ea369b032d87b88f1a47187b51ad4321dea5766 [24/31] KVM: x86: fix mixed

Re: [kbuild] [next:master 5048/10353] fs/quota/dquot.c:1239 flush_warnings() error: buffer overflow 'warn' 2 = 2

2015-04-09 Thread Dan Carpenter
Cross function database is out of sync. I'm not totally sure why this is. regards, dan carpenter On Thu, Apr 09, 2015 at 06:08:06PM +0800, kbuild test robot wrote: TO: Li Xi pkuelel...@gmail.com CC: Jan Kara j...@suse.cz CC: Dmitry Monakhov dmonak...@openvz.org tree: git

[kbuild] [char-misc:char-misc-testing 112/125] drivers/hv/channel_mgmt.c:217 vmbus_free_channels() error: dereferencing freed memory 'channel'

2015-04-07 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git char-misc-testing head: 1ac4e6fee41d6534b6e54dcbed381590e242bdcb commit: aadc3780f31865edc84c587ab718a33a8eeeb09d [112/125] hv: remove the per-channel workqueue drivers/hv/channel_mgmt.c:217 vmbus_free_channels()

Re: [kbuild] [usb-serial:usb-next 9/12] drivers/usb/serial/f81232.c:615 f81232_get_serial_info() warn: check that 'ser' doesn't leak information (struct has a hole after 'iomem_reg_shift')

2015-04-07 Thread Dan Carpenter
False positive. I haven't looked at the pre-compiled but I'm guessing that this is because it was calling __builtin_memset(). I've added that now so hopefully it won't happen again. regards, dan carpenter On Sat, Mar 28, 2015 at 02:00:33AM +0800, kbuild test robot wrote: TO: Peter Hung hpe

Re: [kbuild] [next:master 6249/8741] fs/xfs/xfs_inode.c:357 xfs_lock_inodes() error: buffer overflow 'ips' 4 = 4

2015-04-07 Thread Dan Carpenter
False positive. The cross database was slightly out of sync. regards, dan carpenter On Sun, Apr 05, 2015 at 11:16:54PM +0800, kbuild test robot wrote: TO: Dave Chinner da...@fromorbit.com tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master head

Re: [kbuild] [linuxtv-media:master 331/653] drivers/media/dvb-core/dvbdev.c:270 dvb_register_device() error: __builtin_memcpy() 'template' too small (96 vs 104)

2015-04-07 Thread Dan Carpenter
False positive. I tried to fix this before but my patch caused random segfaults and I gave up. I should look again. Sorry. regards, dan carpenter On Mon, Apr 06, 2015 at 12:17:38AM +0800, kbuild test robot wrote: TO: Mauro Carvalho Chehab m.che...@samsung.com CC: linux-me...@vger.kernel.org

Re: [kbuild] [next:master 2183/8741] drivers/platform/x86/thinkpad_acpi.c:3683 adaptive_keyboard_hotkey_notify_hotkey() error: buffer overflow 'hotkey_keycode_map' 32 = 52

2015-04-07 Thread Dan Carpenter
I sent a patch for this already. regards, dan carpenter On Mon, Apr 06, 2015 at 12:41:13AM +0800, kbuild test robot wrote: TO: Bastien Nocera had...@hadess.net CC: Darren Hart dvh...@linux.intel.com tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master head

Re: [kbuild] [next:master 7424/7993] drivers/target/target_core_sbc.c:696 sbc_check_prot() error: we previously assumed 'is_write' could be null (see line 683)

2015-04-07 Thread Dan Carpenter
Database out of sync I think. I should probably silence this one though because is_write is a bool so obviously this warning makes no sense. regards, dan carpenter On Mon, Apr 06, 2015 at 01:03:08AM +0800, kbuild test robot wrote: TO: Nicholas Bellinger n...@linux-iscsi.org tree: git

[kbuild] [vfs:for-next 65/66] fs/fuse/file.c:2837 fuse_direct_IO() warn: if statement not indented

2015-04-06 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs.git for-next head: 7bfd391fd08e2c10efc4119833949faf6d50d7d7 commit: 48d8dc19de85b549007d636e4db6a8884c11fc96 [65/66] direct_IO: use iov_iter_rw() instead of rw everywhere fs/fuse/file.c:2837 fuse_direct_IO() warn: if statement not

[kbuild] [ntb:ntb-next-djiang 2/2] drivers/ntb/ntb_transport.c:784 ntb_transport_link_work() warn: should '1 ((val (i * 8)) 255)' be a 64 bit type?

2015-04-06 Thread Dan Carpenter
tree: git://github.com/jonmason/ntb ntb-next-djiang head: 3ab45b3c3841a78b3ea3398c85518d2c56520743 commit: 3ab45b3c3841a78b3ea3398c85518d2c56520743 [2/2] ntb: workaround for high traffic hardware hang drivers/ntb/ntb_transport.c:784 ntb_transport_link_work() warn: should '1 ((val (i * 8))

Re: [kbuild] [next:master 8143/8741] fs/ocfs2/export.c:121 ocfs2_get_dentry() warn: unchecked 'PTR_ERR'

2015-04-04 Thread Dan Carpenter
It's a false positive-ish. I sent a patch to fix it already. regards, dan carpenter On Sat, Apr 04, 2015 at 02:43:55PM +0800, kbuild test robot wrote: TO: Andrew Morton a...@linux-foundation.org CC: Linux Memory Management List linux...@kvack.org tree: git://git.kernel.org/pub/scm/linux

[kbuild] [mac80211-next:fast-xmit 3/16] net/mac80211/rc80211_minstrel_debugfs.c:74 minstrel_stats_open() warn: format string contains non-ascii character 'xc3'

2015-04-04 Thread Dan Carpenter
[ Can dmesg be non-ascii? I think the answer is no, but I'm not positive. - dan ] tree: git://git.kernel.org/pub/scm/linux/kernel/git/jberg/mac80211-next.git fast-xmit head: 878359b161f692d918c2be5de7427ca8dc635a24 commit: e161f7f6c4143ef0d945e98b2d1032cf82939fb6 [3/16] mac80211: enhance

Re: [kbuild] [next:master 9220/10050] drivers/input/mouse/alps.c:961 alps_get_mt_count() error: buffer overflow 'mt' 2 = 3

2015-04-08 Thread Dan Carpenter
Database slightly out of sync. regards, dan carpenter On Wed, Apr 08, 2015 at 07:44:41PM +0800, kbuild test robot wrote: TO: Masaki Ota masaki@jp.alps.com CC: Dmitry Torokhov dmitry.torok...@gmail.com tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master

[kbuild] [next:master 9613/10050] mm/cma_debug.c:45 cma_used_get() warn: should 'used cma-order_per_bit' be a 64 bit type?

2015-04-08 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master head: a897436e0e233e84b664bb7f33c4e0d4d3e3bdad commit: 8b0c0ea86849b55091281d146d62bbd9cda87556 [9613/10050] mm-cma-add-functions-to-get-region-pages-counters-fix-2 mm/cma_debug.c:45 cma_used_get() warn: should

Re: [kbuild] [next:master 9619/10050] mm/memory.c:2211 wp_pfn_shared() error: double unlock 'spin_lock:ptl'

2015-04-08 Thread Dan Carpenter
False positive. Mm locking is so complicated. And smatch doesn't do cross function analysis here. regards, dan carpenter On Wed, Apr 08, 2015 at 08:10:45PM +0800, kbuild test robot wrote: TO: Boaz Harrosh b...@plexistor.com CC: Yigal Korman yi...@plexistor.com CC: Andrew Morton a...@linux

[kbuild] [next:master 9268/10050] drivers/vfio/pci/vfio_pci.c:949 vfio_pci_probe() warn: passing freed memory 'vdev'

2015-04-08 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master head: a897436e0e233e84b664bb7f33c4e0d4d3e3bdad commit: ecaa1f6a01544604de5f9531379a303eee886162 [9268/10050] vfio-pci: Add VGA arbiter client drivers/vfio/pci/vfio_pci.c:949 vfio_pci_probe() warn: passing freed

[kbuild] [nvdimm:libnd-for-next 5/21] drivers/acpi/nfit.c:53:40: sparse: duplicate const

2015-05-28 Thread Dan Carpenter
tree: git://git.kernel.org/pub/scm/linux/kernel/git/djbw/nvdimm.git libnd-for-next head: dbcc765a7830454abb78e5352147324605455116 commit: 973e2dccb9f9cefb222ca4d1996a805cbabefde7 [5/21] libnd: control (ioctl) messages for libnd bus and dimm devices reproduce: # apt-get install sparse git

Re: [kbuild] [linux-next:master 5659/5676] kernel/trace/trace_syscalls.c:106 syscall_nr_to_meta() error: buffer overflow 'syscalls_metadata' 374 = 375

2015-07-31 Thread Dan Carpenter
We're getting a lot of false positives today because the database is out of sync with the kernel we are testing. regards, dan carpenter On Fri, Jul 31, 2015 at 09:36:46PM +0800, kbuild test robot wrote: TO: Stephen Rothwell s...@canb.auug.org.au CC: kbuild-...@01.org tree: git

Re: [kbuild] [linux-next:master 5659/5676] kernel/trace/trace_syscalls.c:106 syscall_nr_to_meta() error: buffer overflow 'syscalls_metadata' 374 = 375

2015-08-01 Thread Dan Carpenter
On Sat, Aug 01, 2015 at 08:22:41PM +0800, Fengguang Wu wrote: Hi Dan, On Fri, Jul 31, 2015 at 05:03:20PM +0300, Dan Carpenter wrote: We're getting a lot of false positives today because the database is out of sync with the kernel we are testing. Sorry we just fixed a bug that prevented

Re: [kbuild] net/tipc/link.c:1768 tipc_link_failover_rcv() error: buffer overflow 'link-owner-links' 2 = s32max

2015-07-31 Thread Dan Carpenter
This is a false positive. msg_bearer_id() returns a number between 0-1 so it can't overflow. I remember fixing this bug on my computer but I may not have pushed the fix. I'll check. regards, dan carpenter On Fri, Jul 31, 2015 at 01:03:18PM +0800, kbuild test robot wrote: TO: Jon Paul

Re: [kbuild] [linux-next:master 4077/5676] drivers/net/ethernet/cadence/macb.c:2876 macb_probe() warn: variable dereferenced before check 'native_io' (see line 2860)

2015-07-31 Thread Dan Carpenter
False positive. Should be fixed in the latest... regards, dan carpenter On Fri, Jul 31, 2015 at 03:51:05PM +0800, kbuild test robot wrote: TO: Andy Shevchenko andriy.shevche...@linux.intel.com CC: kbuild-...@01.org tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git

Re: [kbuild] [linux-next:master 1529/5676] fs/gfs2/glock.c:1798 gfs2_glock_iter_next() error: 'gi-gl' dereferencing possible ERR_PTR()

2015-07-31 Thread Dan Carpenter
I reported this earlier. Bob Peterson said he would write a fix. regards, dan carpenter On Fri, Jul 31, 2015 at 04:51:38PM +0800, kbuild test robot wrote: TO: Bob Peterson rpete...@redhat.com CC: kbuild-...@01.org tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git

[kbuild] [sunxi:sunxi/wip/a20-i2s 10/17] drivers/dma/sun4i-dma.c:603 sun4i_dma_free_contract() error: dereferencing freed memory 'promise'

2015-08-06 Thread Dan Carpenter
tree: git://github.com/mripard/linux sunxi/wip/a20-i2s head: 88f326fc32920b7dd90923e017462962a80b8300 commit: 9154610c3c1600990cdae8bb2b7684b24c487660 [10/17] dma: sun4i: Add support for the DMA engine on sun[457]i SoCs New smatch warnings: drivers/dma/sun4i-dma.c:603

Re: [kbuild] [net-next:master 153/164] drivers/net/usb/lan78xx.c:2282 tx_complete() warn: variable dereferenced before check 'skb' (see line 2249)

2015-08-06 Thread Dan Carpenter
Sorry, I should have seen this earlier. Anyway, I email this report already. regards, dan carpenter On Sat, Aug 01, 2015 at 11:31:53AM +0800, kbuild test robot wrote: TO: woojung@microchip.com woojung@microchip.com CC: kbuild-...@01.org CC: Woojung Huh woojung@microchip.com CC

[kbuild] [cschaufler:smack-for-4.3 4/5] security/smack/smack_lsm.c:2779 smack_socket_connect() warn: variable dereferenced before check 'sock-sk' (see line 2776)

2015-08-06 Thread Dan Carpenter
tree: git://github.com/cschaufler/smack-next smack-for-4.3 head: 1eddfe8edb8f85a0d7f793090abcbdf8741455de commit: 21abb1ec414c75abe32c3854848ff30e2b4a6113 [4/5] Smack: IPv6 host labeling New smatch warnings: security/smack/smack_lsm.c:2779 smack_socket_connect() warn: variable dereferenced

<    1   2   3   4   5   6   7   8   9   10   >