Re: [PATCH iproute2 net-next] tc: m_mirred: Fix parsing of 'index' optional argument

2016-10-26 Thread Stephen Hemminger
On Fri, 21 Oct 2016 23:01:15 +0300 Shmulik Ladkani wrote: > Code in parse_mirred() suggests "index" argument can be placed either > after the "egress" clause, or as the first argument (after "action mirred"). > > However, parse_egress() fails to correctly parse

[PATCH iproute2 net-next] tc: m_mirred: Fix parsing of 'index' optional argument

2016-10-21 Thread Shmulik Ladkani
Code in parse_mirred() suggests "index" argument can be placed either after the "egress" clause, or as the first argument (after "action mirred"). However, parse_egress() fails to correctly parse "index" if it's the first argument. For example: # tc filter add ... action mirred index 5