Message ID | 20210517065830.3477-1-heiko.thiery@gmail.com (mailing list archive) |
---|---|
State | Changes Requested |
Delegated to: | Netdev Maintainers |
Headers | show |
Series | [v2] Fix warning due to format mismatch for field width argument to fprintf() | expand |
Context | Check | Description |
---|---|---|
netdev/tree_selection | success | Not a local patch |
diff --git a/bnxt.c b/bnxt.c index b46db72..2b0ac76 100644 --- a/bnxt.c +++ b/bnxt.c @@ -63,7 +63,7 @@ int bnxt_dump_regs(struct ethtool_drvinfo *info __maybe_unused, struct ethtool_r return 0; if (regs->len < (BNXT_PXP_REG_LEN + BNXT_PCIE_STATS_LEN)) { - fprintf(stdout, "Length is too short, expected 0x%lx\n", + fprintf(stdout, "Length is too short, expected 0x%zx\n", BNXT_PXP_REG_LEN + BNXT_PCIE_STATS_LEN); return -1; }
bnxt.c:66:54: warning: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘unsigned int’ [-Wformat=] 66 | fprintf(stdout, "Length is too short, expected 0x%lx\n", | ~~^ | | | long unsigned int | %x Signed-off-by: Heiko Thiery <heiko.thiery@gmail.com> --- v2: - use '%zx' since the value is converted to size_t (thanks to Ben) bnxt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)