Re: [block:for-5.11/drivers 49/60] drivers/block/rnbd/rnbd-clt.c:1397:42: warning: size argument in 'strlcpy' call appears to be size of the source; expected the size of the destination

[Haris Iqbal <haris.iqbal@cloud.ionos.com>]

[-- Attachment #1: Type: text/plain, Size: 4474 bytes --]

Working on it.

On Tue, Dec 8, 2020 at 10:42 AM kernel test robot <lkp@intel.com> wrote:
>
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git for-5.11/drivers
> head:   eebf34a85c8c724676eba502d15202854f199b05
> commit: 64e8a6ece1a5b1fa21316918053d068baeac84af [49/60] block/rnbd-clt: Dynamically alloc buffer for pathname & blk_symlink_name
> config: arm-randconfig-r002-20201208 (attached as .config)
> compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project a2f922140f5380571fb74179f2bf622b3b925697)
> 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
>         # install arm cross compiling tool for clang build
>         # apt-get install binutils-arm-linux-gnueabi
>         # https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git/commit/?id=64e8a6ece1a5b1fa21316918053d068baeac84af
>         git remote add block https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git
>         git fetch --no-tags block for-5.11/drivers
>         git checkout 64e8a6ece1a5b1fa21316918053d068baeac84af
>         # save the attached .config to linux build tree
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm
>
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
>
> All warnings (new ones prefixed by >>):
>
> >> drivers/block/rnbd/rnbd-clt.c:1397:42: warning: size argument in 'strlcpy' call appears to be size of the source; expected the size of the destination [-Wstrlcpy-strlcat-size]
>            strlcpy(dev->pathname, pathname, strlen(pathname) + 1);
>                                             ~~~~~~~^~~~~~~~~~~~~
>    1 warning generated.
>
> vim +/strlcpy +1397 drivers/block/rnbd/rnbd-clt.c
>
>   1363
>   1364  static struct rnbd_clt_dev *init_dev(struct rnbd_clt_session *sess,
>   1365                                        enum rnbd_access_mode access_mode,
>   1366                                        const char *pathname)
>   1367  {
>   1368          struct rnbd_clt_dev *dev;
>   1369          int ret;
>   1370
>   1371          dev = kzalloc_node(sizeof(*dev), GFP_KERNEL, NUMA_NO_NODE);
>   1372          if (!dev)
>   1373                  return ERR_PTR(-ENOMEM);
>   1374
>   1375          dev->hw_queues = kcalloc(nr_cpu_ids, sizeof(*dev->hw_queues),
>   1376                                   GFP_KERNEL);
>   1377          if (!dev->hw_queues) {
>   1378                  ret = -ENOMEM;
>   1379                  goto out_alloc;
>   1380          }
>   1381
>   1382          mutex_lock(&ida_lock);
>   1383          ret = ida_simple_get(&index_ida, 0, 1 << (MINORBITS - RNBD_PART_BITS),
>   1384                               GFP_KERNEL);
>   1385          mutex_unlock(&ida_lock);
>   1386          if (ret < 0) {
>   1387                  pr_err("Failed to initialize device '%s' from session %s, allocating idr failed, err: %d\n",
>   1388                         pathname, sess->sessname, ret);
>   1389                  goto out_queues;
>   1390          }
>   1391
>   1392          dev->pathname = kzalloc(strlen(pathname) + 1, GFP_KERNEL);
>   1393          if (!dev->pathname) {
>   1394                  ret = -ENOMEM;
>   1395                  goto out_queues;
>   1396          }
> > 1397          strlcpy(dev->pathname, pathname, strlen(pathname) + 1);
>   1398
>   1399          dev->clt_device_id      = ret;
>   1400          dev->sess               = sess;
>   1401          dev->access_mode        = access_mode;
>   1402          mutex_init(&dev->lock);
>   1403          refcount_set(&dev->refcount, 1);
>   1404          dev->dev_state = DEV_STATE_INIT;
>   1405
>   1406          /*
>   1407           * Here we called from sysfs entry, thus clt-sysfs is
>   1408           * responsible that session will not disappear.
>   1409           */
>   1410          WARN_ON(!rnbd_clt_get_sess(sess));
>   1411
>   1412          return dev;
>   1413
>   1414  out_queues:
>   1415          kfree(dev->hw_queues);
>   1416  out_alloc:
>   1417          kfree(dev);
>   1418          return ERR_PTR(ret);
>   1419  }
>   1420
>
> ---
> 0-DAY CI Kernel Test Service, Intel Corporation
> https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org