From: kernel test robot <lkp@intel.com>
To: "Marek Behún" <kabel@kernel.org>
Cc: oe-kbuild-all@lists.linux.dev, linux-kernel@vger.kernel.org,
"Gregory CLEMENT" <gregory.clement@bootlin.com>,
"Pali Rohár" <pali@kernel.org>, "Andrew Lunn" <andrew@lunn.ch>
Subject: drivers/firmware/turris-mox-rwtm.c:249:90: warning: '%08x' directive writing 8 bytes into a region of size between 7 and 9
Date: Mon, 6 Jan 2025 15:00:10 +0800 [thread overview]
Message-ID: <202501061439.SOt2vO20-lkp@intel.com> (raw)
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 9d89551994a430b50c4fffcb1e617a057fa76e20
commit: e34e60253d9272311831daed8a2d967cf80ca3dc firmware: turris-mox-rwtm: fix reply status decoding function
date: 3 years, 7 months ago
config: arm-randconfig-r002-20220811 (https://download.01.org/0day-ci/archive/20250106/202501061439.SOt2vO20-lkp@intel.com/config)
compiler: arm-linux-gnueabi-gcc (GCC) 12.4.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250106/202501061439.SOt2vO20-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202501061439.SOt2vO20-lkp@intel.com/
All warnings (new ones prefixed by >>):
drivers/firmware/turris-mox-rwtm.c: In function 'mox_get_board_info':
>> drivers/firmware/turris-mox-rwtm.c:249:90: warning: '%08x' directive writing 8 bytes into a region of size between 7 and 9 [-Wformat-overflow=]
249 | "%06x%08x%08x%08x%08x%08x%08x%08x%08x%08x%08x%08x%08x%08x%08x%08x%08x",
| ^~~~
drivers/firmware/turris-mox-rwtm.c:248:17: note: 'sprintf' output between 135 and 137 bytes into a destination of size 135
248 | sprintf(rwtm->pubkey,
| ^~~~~~~~~~~~~~~~~~~~~
249 | "%06x%08x%08x%08x%08x%08x%08x%08x%08x%08x%08x%08x%08x%08x%08x%08x%08x",
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
250 | ret, s[0], s[1], s[2], s[3], s[4], s[5], s[6], s[7],
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
251 | s[8], s[9], s[10], s[11], s[12], s[13], s[14], s[15]);
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
vim +249 drivers/firmware/turris-mox-rwtm.c
389711b3749399 Marek Behún 2019-08-22 190
389711b3749399 Marek Behún 2019-08-22 191 static int mox_get_board_info(struct mox_rwtm *rwtm)
389711b3749399 Marek Behún 2019-08-22 192 {
389711b3749399 Marek Behún 2019-08-22 193 struct armada_37xx_rwtm_tx_msg msg;
389711b3749399 Marek Behún 2019-08-22 194 struct armada_37xx_rwtm_rx_msg *reply = &rwtm->reply;
389711b3749399 Marek Behún 2019-08-22 195 int ret;
389711b3749399 Marek Behún 2019-08-22 196
389711b3749399 Marek Behún 2019-08-22 197 msg.command = MBOX_CMD_BOARD_INFO;
389711b3749399 Marek Behún 2019-08-22 198 ret = mbox_send_message(rwtm->mbox, &msg);
389711b3749399 Marek Behún 2019-08-22 199 if (ret < 0)
389711b3749399 Marek Behún 2019-08-22 200 return ret;
389711b3749399 Marek Behún 2019-08-22 201
389711b3749399 Marek Behún 2019-08-22 202 ret = wait_for_completion_timeout(&rwtm->cmd_done, HZ / 2);
389711b3749399 Marek Behún 2019-08-22 203 if (ret < 0)
389711b3749399 Marek Behún 2019-08-22 204 return ret;
389711b3749399 Marek Behún 2019-08-22 205
389711b3749399 Marek Behún 2019-08-22 206 ret = mox_get_status(MBOX_CMD_BOARD_INFO, reply->retval);
389711b3749399 Marek Behún 2019-08-22 207 if (ret < 0 && ret != -ENODATA) {
389711b3749399 Marek Behún 2019-08-22 208 return ret;
389711b3749399 Marek Behún 2019-08-22 209 } else if (ret == -ENODATA) {
389711b3749399 Marek Behún 2019-08-22 210 dev_warn(rwtm->dev,
389711b3749399 Marek Behún 2019-08-22 211 "Board does not have manufacturing information burned!\n");
389711b3749399 Marek Behún 2019-08-22 212 } else {
389711b3749399 Marek Behún 2019-08-22 213 rwtm->serial_number = reply->status[1];
389711b3749399 Marek Behún 2019-08-22 214 rwtm->serial_number <<= 32;
389711b3749399 Marek Behún 2019-08-22 215 rwtm->serial_number |= reply->status[0];
389711b3749399 Marek Behún 2019-08-22 216 rwtm->board_version = reply->status[2];
389711b3749399 Marek Behún 2019-08-22 217 rwtm->ram_size = reply->status[3];
389711b3749399 Marek Behún 2019-08-22 218 reply_to_mac_addr(rwtm->mac_address1, reply->status[4],
389711b3749399 Marek Behún 2019-08-22 219 reply->status[5]);
389711b3749399 Marek Behún 2019-08-22 220 reply_to_mac_addr(rwtm->mac_address2, reply->status[6],
389711b3749399 Marek Behún 2019-08-22 221 reply->status[7]);
389711b3749399 Marek Behún 2019-08-22 222 rwtm->has_board_info = 1;
389711b3749399 Marek Behún 2019-08-22 223
389711b3749399 Marek Behún 2019-08-22 224 pr_info("Turris Mox serial number %016llX\n",
389711b3749399 Marek Behún 2019-08-22 225 rwtm->serial_number);
389711b3749399 Marek Behún 2019-08-22 226 pr_info(" board version %i\n", rwtm->board_version);
389711b3749399 Marek Behún 2019-08-22 227 pr_info(" burned RAM size %i MiB\n", rwtm->ram_size);
389711b3749399 Marek Behún 2019-08-22 228 }
389711b3749399 Marek Behún 2019-08-22 229
389711b3749399 Marek Behún 2019-08-22 230 msg.command = MBOX_CMD_ECDSA_PUB_KEY;
389711b3749399 Marek Behún 2019-08-22 231 ret = mbox_send_message(rwtm->mbox, &msg);
389711b3749399 Marek Behún 2019-08-22 232 if (ret < 0)
389711b3749399 Marek Behún 2019-08-22 233 return ret;
389711b3749399 Marek Behún 2019-08-22 234
389711b3749399 Marek Behún 2019-08-22 235 ret = wait_for_completion_timeout(&rwtm->cmd_done, HZ / 2);
389711b3749399 Marek Behún 2019-08-22 236 if (ret < 0)
389711b3749399 Marek Behún 2019-08-22 237 return ret;
389711b3749399 Marek Behún 2019-08-22 238
389711b3749399 Marek Behún 2019-08-22 239 ret = mox_get_status(MBOX_CMD_ECDSA_PUB_KEY, reply->retval);
389711b3749399 Marek Behún 2019-08-22 240 if (ret < 0 && ret != -ENODATA) {
389711b3749399 Marek Behún 2019-08-22 241 return ret;
389711b3749399 Marek Behún 2019-08-22 242 } else if (ret == -ENODATA) {
389711b3749399 Marek Behún 2019-08-22 243 dev_warn(rwtm->dev, "Board has no public key burned!\n");
389711b3749399 Marek Behún 2019-08-22 244 } else {
389711b3749399 Marek Behún 2019-08-22 245 u32 *s = reply->status;
389711b3749399 Marek Behún 2019-08-22 246
389711b3749399 Marek Behún 2019-08-22 247 rwtm->has_pubkey = 1;
389711b3749399 Marek Behún 2019-08-22 248 sprintf(rwtm->pubkey,
389711b3749399 Marek Behún 2019-08-22 @249 "%06x%08x%08x%08x%08x%08x%08x%08x%08x%08x%08x%08x%08x%08x%08x%08x%08x",
389711b3749399 Marek Behún 2019-08-22 250 ret, s[0], s[1], s[2], s[3], s[4], s[5], s[6], s[7],
389711b3749399 Marek Behún 2019-08-22 251 s[8], s[9], s[10], s[11], s[12], s[13], s[14], s[15]);
389711b3749399 Marek Behún 2019-08-22 252 }
389711b3749399 Marek Behún 2019-08-22 253
389711b3749399 Marek Behún 2019-08-22 254 return 0;
389711b3749399 Marek Behún 2019-08-22 255 }
389711b3749399 Marek Behún 2019-08-22 256
:::::: The code at line 249 was first introduced by commit
:::::: 389711b374939968d2d0adede01b9e1f3bb4f531 firmware: Add Turris Mox rWTM firmware driver
:::::: TO: Marek Behún <marek.behun@nic.cz>
:::::: CC: Arnd Bergmann <arnd@arndb.de>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next reply other threads:[~2025-01-06 7:01 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-01-06 7:00 kernel test robot [this message]
-- strict thread matches above, loose matches on Subject: below --
2025-02-26 22:17 drivers/firmware/turris-mox-rwtm.c:249:90: warning: '%08x' directive writing 8 bytes into a region of size between 7 and 9 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=202501061439.SOt2vO20-lkp@intel.com \
--to=lkp@intel.com \
--cc=andrew@lunn.ch \
--cc=gregory.clement@bootlin.com \
--cc=kabel@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=pali@kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox