On Tue, 2018-03-13 at 21:19 +0100, Alexander Zubkov wrote: > Debian maintainer found that basic command: > # ip route flush all > No longer worked as expected which breaks user scripts and > expectations. It no longer flushed all IPv4 routes. > > Recently behaviour of "default" prefix parameter was corrected. But > at > the same time behaviour of "all"/"any" was altered too, because they > were the same branch of the code. As those parameters mean different, > they need to be treated differently in code too. This patch reflects > the difference. > > Reported-by: Luca Boccassi <[email protected]> > Signed-off-by: Alexander Zubkov <[email protected]> > --- > lib/utils.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/lib/utils.c b/lib/utils.c > index 9fa5220..b289d9c 100644 > --- a/lib/utils.c > +++ b/lib/utils.c > @@ -658,7 +658,8 @@ int get_prefix_1(inet_prefix *dst, char *arg, > int > family) > dst->family = family; > dst->bytelen = 0; > dst->bitlen = 0; > - dst->flags |= PREFIXLEN_SPECIFIED; > + if (strcmp(arg, "default") == 0) > + dst->flags |= PREFIXLEN_SPECIFIED; > return 0; > } > > -- > 1.9.1
Tested-by: Luca Boccassi <[email protected]> Yep solves the problem, ls all/any now work as before. Thanks! -- Kind regards, Luca Boccassi
signature.asc
Description: This is a digitally signed message part
