Hi Fred!

On Sun, Apr 16, 2017 at 05:33:01PM +0200, Frederic Lecaille wrote:
> The patch attached to this mail fixes this major bug.

Applied, thanks!

> Note that all such added lines 'if (kw->skip == -1)' may be removed,
> but I am not sure it is a good thing.

I think we'll be able to simplify some parts of the parser after we
complete the pending ssl version keyword changes.

I'm even thinking that "source" + "usesrc" is the only couple causing
trouble, and most people use "source 0.0.0.0 usesrc foo", so we could
imagine relaxing this in the future to allow "usesrc" alone and let
"source" take exactly one argument.

Cheers,
Willy

Reply via email to