[apparmor] [patch 22/26] change syntax to specify a signals target from

[Tyler Hicks]

On 2014-03-28 07:20:07, John Johansen wrote:
> fix: the what is treated and a condlistid

huh? :)

> 
> The match
>   {VARIABLE_NAME}/{WS}*={WS}*\(
> 
> is too broad causing mount and dbus rules to fail for sets of values eg.
> 
>   mount options=(ro bind)
> 
> Instead of doing a broad match, for now lets lock it down to just
> peer=(...) being the only cond that can cause entry into CONDLISTID
> 
> Signed-off-by: John Johansen <john.johansen at canonical.com>
> ---
>  parser/parser_lex.l |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> --- 2.9-test.orig/parser/parser_lex.l
> +++ 2.9-test/parser/parser_lex.l
> @@ -295,7 +295,7 @@
>  }
>  
>  <INITIAL,MOUNT_MODE,DBUS_MODE,SIGNAL_MODE>{
> -	{VARIABLE_NAME}/{WS}*={WS}*\(	{
> +	peer/{WS}*={WS}*\(	{
>  		/* we match to the = in the lexer so that we can switch scanner
>  		 * state.  By the time the parser see the = it may be too late
>  		 * as bison may have requested the next token from the scanner
> This patch needs the following additional patch on it
> 

What happened here at the end of the patch? The diff headers tell me I
can safely remove/ignore the "This patch needs the following additional
patch on it" line, but I'm wondering if some of the patch was
accidentally dropped?

Tyler

> 
> 
> -- 
> AppArmor mailing list
> AppArmor at lists.ubuntu.com
> Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <https://lists.ubuntu.com/archives/apparmor/attachments/20140328/cc78b611/attachment.pgp>