On Wed, 3 Apr 2019 15:45:01 +0100 Bruce Richardson <bruce.richard...@intel.com> 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