On Tue, Oct 1, 2019 at 7:59 AM Stephen Hemminger <step...@networkplumber.org> wrote: > > On Mon, 30 Sep 2019 21:52:22 -0700 > Roopa Prabhu <ro...@cumulusnetworks.com> wrote: > > > + > > + if (sscanf(addr, "%hhx:%hhx:%hhx:%hhx:%hhx:%hhx", > > + abuf, abuf+1, abuf+2, > > + abuf+3, abuf+4, abuf+5) != 6) { > > + fprintf(stderr, "Invalid mac address %s\n", addr); > > + return -1; > > + } > > You could use ether_aton here if that would help. > Not required, but ether_ntoa already used in iplink_bridge.
ok ack, i will take a look. I think i picked this up from bridge/fdb.c:fdb_modify