mirrors/passt
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

packet: Offset plus length is not always uint32_t, but it's always size_t

Browse source 
According to gcc, PRIu32 matches the type of the argument we're
printing here on both 64 and 32-bits architectures. According to
Clang, though, that's not the case, as the result of the sum is an
unsigned long on 64-bit.

Use the z modifier, given that we're summing uint32_t to size_t, and
the result is at most promoted to size_t.

Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
Reviewed-by: David Gibson <david@gibson.dropbear.id.au>
This commit is contained in:
Stefano Brivio 2023-11-29 13:20:18 +01:00
parent 06559048e7
commit 7e175f32c1
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
packet.c
View file

@ -106,7 +106,7 @@ void *packet_get_do(const struct pool *p, size_t idx, size_t offset,
if (p->pkt[idx].offset + len + offset > p->buf_size) { if (p->pkt[idx].offset + len + offset > p->buf_size) {
if (func) { if (func) {
trace("packet offset plus length %lu from size %zu, " trace("packet offset plus length %zu from size %zu, "
"%s:%i", p->pkt[idx].offset + len + offset, "%s:%i", p->pkt[idx].offset + len + offset,
p->buf_size, func, line); p->buf_size, func, line);
} }