aboutgitcodebugslistschat
path: root/pcap.c
diff options
context:
space:
mode:
authorDavid Gibson <david@gibson.dropbear.id.au>2024-09-18 20:44:05 +1000
committerStefano Brivio <sbrivio@redhat.com>2024-09-18 17:14:59 +0200
commitbfc294b90dc46d132a56dc0a2ae118f2bea5a266 (patch)
tree2d3a7011fa15b92783a289c8f6ecbe1946289207 /pcap.c
parentbb41901c719f9ba422b538f773025dad5c398823 (diff)
downloadpasst-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.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/pcap.c b/pcap.c
index 46cc4b0..e6b5ced 100644
--- a/pcap.c
+++ b/pcap.c
@@ -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);
}