OK, I can use it)

Thanks,

On Wed, Dec 3, 2014 at 4:40 PM, Roopa Prabhu [off-list ref] wrote:

On 12/2/14, 5:13 PM, vadim4j@gmail.com wrote:

quoted

On Tue, Dec 02, 2014 at 04:55:44PM -0800, Roopa Prabhu wrote:

quoted

quoted

        int master;
+       char *link_kind;

The name can be just "kind", given all the others dont use the link
prefix

quoted

  } filter;

OK

quoted

quoted

+       if (filter.link_kind && tb[IFLA_LINKINFO]) {
+               char *link_kind = parse_link_kind(tb[IFLA_LINKINFO]);
+               if (strcmp(link_kind, filter.link_kind)) {
+                       return -1;
+               }

you can skip the braces

quoted

+       } else if (filter.link_kind)

you have if (filter.link_kind) twice, you can use a single if without the
else.

quoted

+               return -1;
+

I need to skip interfaces which has not IFLA_LINKINFO attribute,
w/o "else if(...)" I get bridges with normal ether devices:
          bash# ip link show type bridge

     1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN
mode DEFAULT group default
         link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
     2: enp0s25: <NO-CARRIER,BROADCAST,MULTICAST,UP> mtu 1500 qdisc
pfifo_fast state DOWN mode DEFAULT group default qlen 1000
         link/ether XX:XX:XX:XX:XX:XX brd ff:ff:ff:ff:ff:ff
     5: br0: <BROADCAST,MULTICAST> mtu 1500 qdisc noop state DOWN mode
DEFAULT group default
         link/ether 6e:02:f9:17:7f:10 brd ff:ff:ff:ff:ff:ff

but expected result should be only bridges.

I was just saying, it could be:

if (filter.link_kind) {
    if (tb[IFLA_LINKINFO]) {
        char *kind = parse_link_kind(tb[IFLA_LINKINFO]);
        if (strcmp(kind, filter.kind))
            return -1;
    } else {
      return -1;
    }
}