On Wed, 3 Apr 2019 15:45:01 +0100 Bruce Richardson <[email protected]> wrote:
>
> static void
> -arp_op_name(uint16_t arp_op, char *buf)
> +arp_op_name(uint16_t arp_op, char *buf, size_t buf_len)
> {
> switch (arp_op) {
> case ARP_OP_REQUEST:
> - snprintf(buf, sizeof("ARP Request"), "%s", "ARP Request");
> + snprintf(buf, buf_len, "%s", "ARP Request");
> return;
This should be strlcpy not snprintf

