Re: [staging:staging-next 86/151] drivers/staging/iio/iio_hwmon.c:96:21-27: ERROR: application of sizeof to pointer

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 09/19/2012 06:31 PM, Fengguang Wu wrote:
> Hi Lars-Peter,
> 
> FYI, there are new coccinelle warnings show up in
> 
> tree:   git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-next
> head:   5d06e3df280bd230e2eadc16372e62818c63e894
> commit: a0e545e0e75006a7de0e9bc5397f6b44c61990b2 [86/151] staging:iio:hwmon bridge: Use iio_read_channel_processed

Hi,

Thanks for info, patch looks good, but the warning is not introduced by the
commit you mention, though.

- Lars

> 
> All coccinelle warnings:
> 
> + drivers/staging/iio/iio_hwmon.c:96:21-27: ERROR: application of sizeof to pointer
> 
> vim +96 drivers/staging/iio/iio_hwmon.c
>     86		st->channels = iio_channel_get_all(dev_name(&pdev->dev));
>     87		if (IS_ERR(st->channels)) {
>     88			ret = PTR_ERR(st->channels);
>     89			goto error_free_state;
>     90		}
>     91	
>     92		/* count how many attributes we have */
>     93		while (st->channels[st->num_channels].indio_dev)
>     94			st->num_channels++;
>     95	
>   > 96		st->attrs = kzalloc(sizeof(st->attrs) * (st->num_channels + 1),
>     97				    GFP_KERNEL);
>     98		if (st->attrs == NULL) {
>     99			ret = -ENOMEM;
>    100			goto error_release_channels;
>    101		}
>    102		for (i = 0; i < st->num_channels; i++) {
>    103			a = kzalloc(sizeof(*a), GFP_KERNEL);
>    104			if (a == NULL) {
>    105				ret = -ENOMEM;
>    106				goto error_free_attrs;
> 
> Please consider folding the attached diff :-)
> 
> ---
> 0-DAY kernel build testing backend         Open Source Technology Centre
> Fengguang Wu, Yuanhan Liu                              Intel Corporation

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel


[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux