diff options
| author | Stefano Brivio <sbrivio@redhat.com> | 2022-05-19 13:07:33 +0200 |
|---|---|---|
| committer | Stefano Brivio <sbrivio@redhat.com> | 2022-05-19 16:27:20 +0200 |
| commit | c318ffcb4c932752cd1f48bf5d1b0268f58895bd (patch) | |
| tree | 30f28814818a2ef35a424379749732b5f32bef32 /udp.c | |
| parent | 3d4c2a44a69bba90f3ca70ba998b92ce892bbcd8 (diff) | |
| download | passt-c318ffcb4c932752cd1f48bf5d1b0268f58895bd.tar passt-c318ffcb4c932752cd1f48bf5d1b0268f58895bd.tar.gz passt-c318ffcb4c932752cd1f48bf5d1b0268f58895bd.tar.bz2 passt-c318ffcb4c932752cd1f48bf5d1b0268f58895bd.tar.lz passt-c318ffcb4c932752cd1f48bf5d1b0268f58895bd.tar.xz passt-c318ffcb4c932752cd1f48bf5d1b0268f58895bd.tar.zst passt-c318ffcb4c932752cd1f48bf5d1b0268f58895bd.zip | |
udp: Ignore bogus -Wstringop-overread for write() from gcc 12.1
With current OpenSUSE Tumbleweed on aarch64 (gcc-12-1.3.aarch64) and
on x86_64 (gcc-12-1.4.x86_64), but curiously not on armv7hl
(gcc-12-1.3.armv7hl), gcc warns about using the _pointer_ to the
802.3 header to write the whole frame to the tap descriptor:
reading between 62 and 4294967357 bytes from a region of size 14
which is bogus:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103483
Probably declaring udp_sock_fill_data_v{4,6}() as noinline would
"fix" this, but that's on the data path, so I'd rather not. Use
a gcc pragma instead.
Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
Diffstat (limited to 'udp.c')
| -rw-r--r-- | udp.c | 4 |
1 files changed, 4 insertions, 0 deletions
@@ -714,8 +714,10 @@ static void udp_sock_fill_data_v4(const struct ctx *c, int n, b->uh.len = htons(udp4_l2_mh_sock[n].msg_len + sizeof(b->uh)); if (c->mode == MODE_PASTA) { + PRAGMA_STRINGOP_OVERREAD_IGNORE if (write(c->fd_tap, &b->eh, sizeof(b->eh) + ip_len) < 0) debug("tap write: %s", strerror(errno)); + PRAGMA_STRINGOP_OVERREAD_IGNORE_POP pcap((char *)&b->eh, sizeof(b->eh) + ip_len); return; @@ -813,8 +815,10 @@ static void udp_sock_fill_data_v6(const struct ctx *c, int n, b->ip6h.hop_limit = 255; if (c->mode == MODE_PASTA) { + PRAGMA_STRINGOP_OVERREAD_IGNORE if (write(c->fd_tap, &b->eh, sizeof(b->eh) + ip_len) < 0) debug("tap write: %s", strerror(errno)); + PRAGMA_STRINGOP_OVERREAD_IGNORE_POP pcap((char *)&b->eh, sizeof(b->eh) + ip_len); return; |