On 02/08/2016 11:21 PM, Stephen Hemminger wrote: > On Mon, 8 Feb 2016 12:55:07 +0100 > Nikolay Aleksandrov <ra...@blackwall.org> wrote: > >> >> +static void br_dump_bridge_id(const struct ifla_bridge_id *id, char *buf, >> + size_t len) >> +{ >> + const unsigned char *x = (const unsigned char *)id; >> + >> + snprintf(buf, len, "%.2x%.2x.%.2x%.2x%.2x%.2x%.2x%.2x", x[0], x[1], >> + x[2], x[3], x[4], x[5], x[6], x[7]); >> +} > > Please don't reinvent ether_ntoa_r > This was taken from bridge-utils, it actually includes 2 more bytes, it's not only an ethernet address.
- [PATCH iproute2 00/21] bridge: complete netlink suppor... Nikolay Aleksandrov
- [PATCH iproute2 04/21] iplink: bridge: add suppor... Nikolay Aleksandrov
- [PATCH iproute2 01/21] iplink: bridge: export bri... Nikolay Aleksandrov
- Re: [PATCH iproute2 01/21] iplink: bridge: ex... Stephen Hemminger
- Re: [PATCH iproute2 01/21] iplink: bridge... Nikolay Aleksandrov
- Re: [PATCH iproute2 01/21] iplink: br... Nikolay Aleksandrov
- [PATCH iproute2 21/21] iplink: bridge: add suppor... Nikolay Aleksandrov
- [PATCH iproute2 06/21] iplink: bridge: add suppor... Nikolay Aleksandrov
- [PATCH iproute2 09/21] iplink: bridge: add suppor... Nikolay Aleksandrov
- [PATCH iproute2 18/21] iplink: bridge: add suppor... Nikolay Aleksandrov
- [PATCH iproute2 17/21] iplink: bridge: add suppor... Nikolay Aleksandrov
- [PATCH iproute2 19/21] iplink: bridge: add suppor... Nikolay Aleksandrov
- [PATCH iproute2 20/21] iplink: bridge: add suppor... Nikolay Aleksandrov
- [PATCH iproute2 10/21] iplink: bridge: add suppor... Nikolay Aleksandrov
- [PATCH iproute2 14/21] iplink: bridge: add suppor... Nikolay Aleksandrov