From: kernel test robot <lkp@intel.com>
To: kbuild@lists.01.org
Subject: drivers/gpu/drm/gma500/cdv_intel_dp.c:863:2: warning: Null pointer passed as 2nd argument to string copy function [clang-analyzer-unix.cstring.NullArg]
Date: Fri, 31 Dec 2021 03:07:00 +0800 [thread overview]
Message-ID: <202112310308.afWcmBaj-lkp@intel.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 16188 bytes --]
CC: llvm(a)lists.linux.dev
CC: kbuild-all(a)lists.01.org
CC: linux-kernel(a)vger.kernel.org
TO: Thomas Zimmermann <tzimmermann@suse.de>
CC: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
CC: Patrik Jakobsson <patrik.r.jakobsson@gmail.com>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: eec4df26e24e978e49ccf9bcf49ca0f2ccdaeffe
commit: 26499e0518a77de29e7db2c53fb0d0e9e15be8fb drm/gma500: Drop DRM_GMA3600 config option
date: 11 months ago
:::::: branch date: 25 hours ago
:::::: commit date: 11 months ago
config: i386-randconfig-c001-20211226 (https://download.01.org/0day-ci/archive/20211231/202112310308.afWcmBaj-lkp(a)intel.com/config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project a9e8b1ee7fd44b53c555a7823ae8fd1a8209c520)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=26499e0518a77de29e7db2c53fb0d0e9e15be8fb
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout 26499e0518a77de29e7db2c53fb0d0e9e15be8fb
# save the config file to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=i386 clang-analyzer
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
clang-analyzer warnings: (new ones prefixed by >>)
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/usb/serial/mos7840.c:1207:2: warning: Value stored to 'lData' is never read [clang-analyzer-deadcode.DeadStores]
lData = LCR_BITS_8;
^
drivers/usb/serial/mos7840.c:1207:2: note: Value stored to 'lData' is never read
drivers/usb/serial/mos7840.c:1208:2: warning: Value stored to 'lStop' is never read [clang-analyzer-deadcode.DeadStores]
lStop = LCR_STOP_1;
^
drivers/usb/serial/mos7840.c:1208:2: note: Value stored to 'lStop' is never read
drivers/usb/serial/mos7840.c:1311:2: warning: Value stored to 'status' is never read [clang-analyzer-deadcode.DeadStores]
status = mos7840_send_cmd_write_baud_rate(mos7840_port, baud);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/usb/serial/mos7840.c:1311:2: note: Value stored to 'status' is never read
status = mos7840_send_cmd_write_baud_rate(mos7840_port, baud);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Suppressed 2 warnings (2 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
5 warnings generated.
net/core/filter.c:1736:2: warning: Null pointer passed as 1st argument to memory set function [clang-analyzer-unix.cstring.NullArg]
memset(to, 0, len);
^
net/core/filter.c:9948:57: note: Passing value via 3rd parameter 'to'
return ____bpf_skb_load_bytes(reuse_kern->skb, offset, to, len);
^~
net/core/filter.c:9948:9: note: Calling '____bpf_skb_load_bytes'
return ____bpf_skb_load_bytes(reuse_kern->skb, offset, to, len);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
net/core/filter.c:1725:15: note: Assuming 'offset' is <= 65535
if (unlikely(offset > 0xffff))
^
include/linux/compiler.h:78:42: note: expanded from macro 'unlikely'
# define unlikely(x) __builtin_expect(!!(x), 0)
^
net/core/filter.c:1725:2: note: Taking false branch
if (unlikely(offset > 0xffff))
^
net/core/filter.c:1729:15: note: Assuming 'ptr' is null
if (unlikely(!ptr))
^
include/linux/compiler.h:78:42: note: expanded from macro 'unlikely'
# define unlikely(x) __builtin_expect(!!(x), 0)
^
net/core/filter.c:1729:2: note: Taking true branch
if (unlikely(!ptr))
^
net/core/filter.c:1730:3: note: Control jumps to line 1736
goto err_clear;
^
net/core/filter.c:1736:2: note: Null pointer passed as 1st argument to memory set function
memset(to, 0, len);
^ ~~
net/core/filter.c:1770:2: warning: Null pointer passed as 1st argument to memory set function [clang-analyzer-unix.cstring.NullArg]
memset(to, 0, len);
^ ~~
net/core/filter.c:1756:15: note: Assuming 'offset' is <= 65535
if (unlikely(offset > 0xffff))
^
include/linux/compiler.h:78:42: note: expanded from macro 'unlikely'
# define unlikely(x) __builtin_expect(!!(x), 0)
^
net/core/filter.c:1756:2: note: Taking false branch
if (unlikely(offset > 0xffff))
^
net/core/filter.c:1759:15: note: Assuming field 'skb' is non-null
if (unlikely(!ctx->skb))
^
include/linux/compiler.h:78:42: note: expanded from macro 'unlikely'
# define unlikely(x) __builtin_expect(!!(x), 0)
^
net/core/filter.c:1759:2: note: Taking false branch
if (unlikely(!ctx->skb))
^
net/core/filter.c:1763:15: note: Assuming 'ptr' is null
if (unlikely(!ptr))
^
include/linux/compiler.h:78:42: note: expanded from macro 'unlikely'
# define unlikely(x) __builtin_expect(!!(x), 0)
^
net/core/filter.c:1763:2: note: Taking true branch
if (unlikely(!ptr))
^
net/core/filter.c:1764:3: note: Control jumps to line 1770
goto err_clear;
^
net/core/filter.c:1770:2: note: Null pointer passed as 1st argument to memory set function
memset(to, 0, len);
^ ~~
Suppressed 3 warnings (3 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
3 warnings generated.
Suppressed 3 warnings (3 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
3 warnings generated.
drivers/gpu/drm/gma500/cdv_intel_display.c:761:2: warning: Value stored to 'dpll' is never read [clang-analyzer-deadcode.DeadStores]
dpll |= DPLL_VCO_ENABLE;
^
drivers/gpu/drm/gma500/cdv_intel_display.c:761:2: note: Value stored to 'dpll' is never read
Suppressed 2 warnings (2 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
7 warnings generated.
>> drivers/gpu/drm/gma500/cdv_intel_dp.c:863:2: warning: Null pointer passed as 2nd argument to string copy function [clang-analyzer-unix.cstring.NullArg]
strncpy (intel_dp->adapter.name, name, sizeof(intel_dp->adapter.name) - 1);
^
drivers/gpu/drm/gma500/cdv_intel_dp.c:1960:2: note: 'name' initialized to a null pointer value
const char *name = NULL;
^~~~~~~~~~~~~~~~
drivers/gpu/drm/gma500/cdv_intel_dp.c:1963:16: note: Calling 'kzalloc'
gma_encoder = kzalloc(sizeof(struct gma_encoder), GFP_KERNEL);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/slab.h:682:9: note: Calling 'kmalloc'
return kmalloc(size, flags | __GFP_ZERO);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/slab.h:540:2: note: Taking false branch
if (__builtin_constant_p(size)) {
^
include/linux/slab.h:557:2: note: Returning pointer, which participates in a condition later
return __kmalloc(size, flags);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/slab.h:682:9: note: Returning from 'kmalloc'
return kmalloc(size, flags | __GFP_ZERO);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/slab.h:682:2: note: Returning pointer, which participates in a condition later
return kmalloc(size, flags | __GFP_ZERO);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/gma500/cdv_intel_dp.c:1963:16: note: Returning from 'kzalloc'
gma_encoder = kzalloc(sizeof(struct gma_encoder), GFP_KERNEL);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/gma500/cdv_intel_dp.c:1964:6: note: Assuming 'gma_encoder' is non-null
if (!gma_encoder)
^~~~~~~~~~~~
drivers/gpu/drm/gma500/cdv_intel_dp.c:1964:2: note: Taking false branch
if (!gma_encoder)
^
drivers/gpu/drm/gma500/cdv_intel_dp.c:1966:25: note: Calling 'kzalloc'
gma_connector = kzalloc(sizeof(struct gma_connector), GFP_KERNEL);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/slab.h:682:9: note: Calling 'kmalloc'
return kmalloc(size, flags | __GFP_ZERO);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/slab.h:540:2: note: Taking false branch
if (__builtin_constant_p(size)) {
^
include/linux/slab.h:557:2: note: Returning pointer, which participates in a condition later
return __kmalloc(size, flags);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/slab.h:682:9: note: Returning from 'kmalloc'
return kmalloc(size, flags | __GFP_ZERO);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/slab.h:682:2: note: Returning pointer, which participates in a condition later
return kmalloc(size, flags | __GFP_ZERO);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/gma500/cdv_intel_dp.c:1966:25: note: Returning from 'kzalloc'
gma_connector = kzalloc(sizeof(struct gma_connector), GFP_KERNEL);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/gma500/cdv_intel_dp.c:1967:13: note: Assuming 'gma_connector' is non-null
if (!gma_connector)
^~~~~~~~~~~~~~
drivers/gpu/drm/gma500/cdv_intel_dp.c:1967:9: note: Taking false branch
if (!gma_connector)
^
drivers/gpu/drm/gma500/cdv_intel_dp.c:1969:13: note: Calling 'kzalloc'
intel_dp = kzalloc(sizeof(struct cdv_intel_dp), GFP_KERNEL);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/slab.h:682:9: note: Calling 'kmalloc'
return kmalloc(size, flags | __GFP_ZERO);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/slab.h:540:2: note: Taking false branch
if (__builtin_constant_p(size)) {
^
include/linux/slab.h:557:2: note: Returning pointer, which participates in a condition later
return __kmalloc(size, flags);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/slab.h:682:9: note: Returning from 'kmalloc'
return kmalloc(size, flags | __GFP_ZERO);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/slab.h:682:2: note: Returning pointer, which participates in a condition later
return kmalloc(size, flags | __GFP_ZERO);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/gma500/cdv_intel_dp.c:1969:13: note: Returning from 'kzalloc'
intel_dp = kzalloc(sizeof(struct cdv_intel_dp), GFP_KERNEL);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/gma500/cdv_intel_dp.c:1970:6: note: Assuming 'intel_dp' is non-null
if (!intel_dp)
^~~~~~~~~
drivers/gpu/drm/gma500/cdv_intel_dp.c:1970:2: note: Taking false branch
if (!intel_dp)
^
drivers/gpu/drm/gma500/cdv_intel_dp.c:1973:7: note: Assuming 'output_reg' is not equal to DP_C
if ((output_reg == DP_C) && cdv_intel_dpc_is_edp(dev))
^~~~~~~~~~~~~~~~~~
drivers/gpu/drm/gma500/cdv_intel_dp.c:1973:27: note: Left side of '&&' is false
if ((output_reg == DP_C) && cdv_intel_dpc_is_edp(dev))
^
drivers/gpu/drm/gma500/cdv_intel_dp.c:1984:6: note: 'type' is equal to DRM_MODE_CONNECTOR_DisplayPort
if (type == DRM_MODE_CONNECTOR_DisplayPort)
^~~~
drivers/gpu/drm/gma500/cdv_intel_dp.c:1984:2: note: Taking true branch
if (type == DRM_MODE_CONNECTOR_DisplayPort)
^
drivers/gpu/drm/gma500/cdv_intel_dp.c:2004:2: note: 'Default' branch taken. Execution continues on line 2015
switch (output_reg) {
vim +863 drivers/gpu/drm/gma500/cdv_intel_dp.c
8695b612943561 Alan Cox 2012-08-08 846
8695b612943561 Alan Cox 2012-08-08 847 static int
367e44080e20f7 Patrik Jakobsson 2013-07-22 848 cdv_intel_dp_i2c_init(struct gma_connector *connector,
367e44080e20f7 Patrik Jakobsson 2013-07-22 849 struct gma_encoder *encoder, const char *name)
8695b612943561 Alan Cox 2012-08-08 850 {
37e7b184da18a9 Alan Cox 2012-08-08 851 struct cdv_intel_dp *intel_dp = encoder->dev_priv;
d112a8163f8375 Zhao Yakui 2012-08-08 852 int ret;
d112a8163f8375 Zhao Yakui 2012-08-08 853
8695b612943561 Alan Cox 2012-08-08 854 DRM_DEBUG_KMS("i2c_init %s\n", name);
d112a8163f8375 Zhao Yakui 2012-08-08 855
8695b612943561 Alan Cox 2012-08-08 856 intel_dp->algo.running = false;
8695b612943561 Alan Cox 2012-08-08 857 intel_dp->algo.address = 0;
37e7b184da18a9 Alan Cox 2012-08-08 858 intel_dp->algo.aux_ch = cdv_intel_dp_i2c_aux_ch;
8695b612943561 Alan Cox 2012-08-08 859
8695b612943561 Alan Cox 2012-08-08 860 memset(&intel_dp->adapter, '\0', sizeof (intel_dp->adapter));
8695b612943561 Alan Cox 2012-08-08 861 intel_dp->adapter.owner = THIS_MODULE;
8695b612943561 Alan Cox 2012-08-08 862 intel_dp->adapter.class = I2C_CLASS_DDC;
8695b612943561 Alan Cox 2012-08-08 @863 strncpy (intel_dp->adapter.name, name, sizeof(intel_dp->adapter.name) - 1);
8695b612943561 Alan Cox 2012-08-08 864 intel_dp->adapter.name[sizeof(intel_dp->adapter.name) - 1] = '\0';
8695b612943561 Alan Cox 2012-08-08 865 intel_dp->adapter.algo_data = &intel_dp->algo;
5bdebb183c9702 Dave Airlie 2013-10-11 866 intel_dp->adapter.dev.parent = connector->base.kdev;
8695b612943561 Alan Cox 2012-08-08 867
d112a8163f8375 Zhao Yakui 2012-08-08 868 if (is_edp(encoder))
d112a8163f8375 Zhao Yakui 2012-08-08 869 cdv_intel_edp_panel_vdd_on(encoder);
d112a8163f8375 Zhao Yakui 2012-08-08 870 ret = i2c_dp_aux_add_bus(&intel_dp->adapter);
d112a8163f8375 Zhao Yakui 2012-08-08 871 if (is_edp(encoder))
d112a8163f8375 Zhao Yakui 2012-08-08 872 cdv_intel_edp_panel_vdd_off(encoder);
d112a8163f8375 Zhao Yakui 2012-08-08 873
d112a8163f8375 Zhao Yakui 2012-08-08 874 return ret;
d112a8163f8375 Zhao Yakui 2012-08-08 875 }
d112a8163f8375 Zhao Yakui 2012-08-08 876
:::::: The code at line 863 was first introduced by commit
:::::: 8695b612943561478fd22f28f45e5692e5d078db gma500: Add the support of display port on CDV
:::::: TO: Alan Cox <alan@linux.intel.com>
:::::: CC: Dave Airlie <airlied@redhat.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
next reply other threads:[~2021-12-30 19:07 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-30 19:07 kernel test robot [this message]
-- strict thread matches above, loose matches on Subject: below --
2021-12-27 6:54 drivers/gpu/drm/gma500/cdv_intel_dp.c:863:2: warning: Null pointer passed as 2nd argument to string copy function [clang-analyzer-unix.cstring.NullArg] kernel test robot
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202112310308.afWcmBaj-lkp@intel.com \
--to=lkp@intel.com \
--cc=kbuild@lists.01.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.