linux-kernel - Re: [mtd:spi-nor/next 16/18] drivers/mtd/spi-nor/debugfs.c:84 spi_nor_params_show() warn: '%ph' cannot be followed by 'n'

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <f1399a5413d8f151e536c7005eca6763@walle.cc>
Date:   Wed, 11 May 2022 13:35:16 +0200
From:   Michael Walle <michael@...le.cc>
To:     Dan Carpenter <dan.carpenter@...cle.com>
Cc:     kbuild@...ts.01.org, lkp@...el.com, kbuild-all@...ts.01.org,
        linux-kernel@...r.kernel.org, Pratyush Yadav <p.yadav@...com>
Subject: Re: [mtd:spi-nor/next 16/18] drivers/mtd/spi-nor/debugfs.c:84
 spi_nor_params_show() warn: '%ph' cannot be followed by 'n'

Am 2022-05-11 12:17, schrieb Dan Carpenter:
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git
> spi-nor/next
> head:   cdbc44dbb2c73d8bafa1a8ae0d780608be5dbd40
> commit: 0257be79fc4a16a3252ce80aa13b3640f728c425 [16/18] mtd: spi-nor:
> expose internal parameters via debugfs
> config: x86_64-randconfig-m001-20220509
> (https://download.01.org/0day-ci/archive/20220511/202205110320.WNi7ZQaB-lkp@intel.com/config)
> compiler: gcc-11 (Debian 11.2.0-20) 11.2.0
> 
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@...el.com>
> Reported-by: Dan Carpenter <dan.carpenter@...cle.com>
> 
> smatch warnings:
> drivers/mtd/spi-nor/debugfs.c:84 spi_nor_params_show() warn: '%ph'
> cannot be followed by 'n'
> 
> vim +84 drivers/mtd/spi-nor/debugfs.c
> 
> 0257be79fc4a16 Michael Walle 2022-04-29   73  static int
> spi_nor_params_show(struct seq_file *s, void *data)
> 0257be79fc4a16 Michael Walle 2022-04-29   74  {
> 0257be79fc4a16 Michael Walle 2022-04-29   75  	struct spi_nor *nor = 
> s->private;
> 0257be79fc4a16 Michael Walle 2022-04-29   76  	struct
> spi_nor_flash_parameter *params = nor->params;
> 0257be79fc4a16 Michael Walle 2022-04-29   77  	struct
> spi_nor_erase_map *erase_map = &params->erase_map;
> 0257be79fc4a16 Michael Walle 2022-04-29   78  	struct
> spi_nor_erase_region *region;
> 0257be79fc4a16 Michael Walle 2022-04-29   79  	const struct flash_info
> *info = nor->info;
> 0257be79fc4a16 Michael Walle 2022-04-29   80  	char buf[16], *str;
> 0257be79fc4a16 Michael Walle 2022-04-29   81  	int i;
> 0257be79fc4a16 Michael Walle 2022-04-29   82
> 0257be79fc4a16 Michael Walle 2022-04-29   83  	seq_printf(s,
> "name\t\t%s\n", info->name);
> 0257be79fc4a16 Michael Walle 2022-04-29  @84  	seq_printf(s,
> "id\t\t%*phn\n", info->id_len, info->id);
> 
> Maybe N was intended instead of n?  (I have no idea).

Thanks for reporting. Yeah I had %*phN previously but then
I wanted to switch to "xx xx xx" style and i missed checking
printk-formats.rst. %*phn worked as intended ;) I'll send
a fixup patch.

-michael