On Wed, May 13, 2015 at 11:15:56AM +0200, David Lamparter wrote:
> In "lib/cli: reduce strcmp in CLI hot paths", I failed to notice that
> CMD_VARIABLE as a boolean test covers a superset of the other types of
> variables.  Thus, the patch broke processing of IP/IPv6/Integer range
> parameters in the CLI.
> 
> Fix by some reordering and introducing TERMINAL_RECORD macro (which
> marks whether a given terminal type is a parameter) to be used in places
> where the check is really for all kinds of variables.
> 
> Reported-by: Timo Teräs <[email protected]>
> Signed-off-by: David Lamparter <[email protected]>
> ---
> This is currently being tested by Martin Winter;  as soon as I
> have an OK I'll merge this.

Tested-by: Martin Winter <[email protected]>

and applied.

_______________________________________________
Quagga-dev mailing list
[email protected]
https://lists.quagga.net/mailman/listinfo/quagga-dev

Reply via email to