Re: [PATCH 6/8] netfilter: fix string extension for case insensitive pattern matching

[Patrick McHardy <kaber@trash.net>]

Joonwoo Park wrote:
> icase of xt_string_info indicates case [in]sensitive matching.
> netfilter can find cmd.exe, Cmd.exe, cMd.exe and etc easily.
> 
> Signed-off-by: Joonwoo Park <joonwpark81@gmail.com>
> ---
>  include/linux/netfilter/xt_string.h |    1 +
>  net/netfilter/xt_string.c           |    2 +-
>  2 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/include/linux/netfilter/xt_string.h b/include/linux/netfilter/xt_string.h
> index bb21dd1..dfd347f 100644
> --- a/include/linux/netfilter/xt_string.h
> +++ b/include/linux/netfilter/xt_string.h
> @@ -12,6 +12,7 @@ struct xt_string_info
>  	char 	  pattern[XT_STRING_MAX_PATTERN_SIZE];
>  	u_int8_t  patlen;
>  	u_int8_t  invert;
> +	u_int8_t  icase;
>  
>  	/* Used internally by the kernel */
>  	struct ts_config __attribute__((aligned(8))) *config;


A few words for the changelog why this won't break compatibility
by changing the data structure would be nice, especially
considering CRIS that doesn't perform any padding itself (you
might be lucky though because of aligned attribute, but please
verify that yourself).