diff options
author | David Gibson <david@gibson.dropbear.id.au> | 2024-09-18 20:44:05 +1000 |
---|---|---|
committer | Stefano Brivio <sbrivio@redhat.com> | 2024-09-18 17:14:59 +0200 |
commit | bfc294b90dc46d132a56dc0a2ae118f2bea5a266 (patch) | |
tree | 2d3a7011fa15b92783a289c8f6ecbe1946289207 /pcap.c | |
parent | bb41901c719f9ba422b538f773025dad5c398823 (diff) | |
download | passt-bfc294b90dc46d132a56dc0a2ae118f2bea5a266.tar passt-bfc294b90dc46d132a56dc0a2ae118f2bea5a266.tar.gz passt-bfc294b90dc46d132a56dc0a2ae118f2bea5a266.tar.bz2 passt-bfc294b90dc46d132a56dc0a2ae118f2bea5a266.tar.lz passt-bfc294b90dc46d132a56dc0a2ae118f2bea5a266.tar.xz passt-bfc294b90dc46d132a56dc0a2ae118f2bea5a266.tar.zst passt-bfc294b90dc46d132a56dc0a2ae118f2bea5a266.zip |
util: Add helper to write() all of a buffer
write(2) might not write all the data it is given. Add a write_all_buf()
helper to keep calling it until all the given data is written, or we get an
error.
Currently we use write_remainder() to do this operation in pcap_frame().
That's a little awkward since it requires constructing an iovec, and future
changes we want to make to write_remainder() will be easier in terms of
this single buffer helper.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
Diffstat (limited to 'pcap.c')
-rw-r--r-- | pcap.c | 3 |
1 files changed, 1 insertions, 2 deletions
@@ -86,9 +86,8 @@ static void pcap_frame(const struct iovec *iov, size_t iovcnt, .caplen = l2len, .len = l2len }; - struct iovec hiov = { &h, sizeof(h) }; - if (write_remainder(pcap_fd, &hiov, 1, 0) < 0 || + if (write_all_buf(pcap_fd, &h, sizeof(h)) < 0 || write_remainder(pcap_fd, iov, iovcnt, offset) < 0) debug_perror("Cannot log packet, length %zu", l2len); } |