On Mon, Nov 13, 2017 at 09:09:40AM +0100, eric.sesterhenn@xxxxxxxxxxx wrote:
> From: Eric Sesterhenn <eric.sesterhenn@xxxxxxxxxxx>
>
> It is bad practive to return in a macro, this patch
> moves the check into a function.
Applied with minor changes, see below.
[...]
> diff --git a/net/netfilter/nf_conntrack_h323_asn1.c b/net/netfilter/nf_conntrack_h323_asn1.c
> index 89b2e46925c4..f358222b1e5e 100644
> --- a/net/netfilter/nf_conntrack_h323_asn1.c
> +++ b/net/netfilter/nf_conntrack_h323_asn1.c
> @@ -103,7 +103,6 @@ typedef struct {
> #define INC_BIT(bs) if((++(bs)->bit)>7){(bs)->cur++;(bs)->bit=0;}
> #define INC_BITS(bs,b) if(((bs)->bit+=(b))>7){(bs)->cur+=(bs)->bit>>3;(bs)->bit&=7;}
> #define BYTE_ALIGN(bs) if((bs)->bit){(bs)->cur++;(bs)->bit=0;}
> -#define CHECK_BOUND(bs,n) if((bs)->cur+(n)>(bs)->end)return(H323_ERROR_BOUND)
> static unsigned int get_len(bitstr_t *bs);
> static unsigned int get_bit(bitstr_t *bs);
> static unsigned int get_bits(bitstr_t *bs, unsigned int b);
> @@ -166,6 +165,14 @@ static unsigned int get_len(bitstr_t *bs)
> }
>
> /****************************************************************************/
> +static int nf_h323_error_boundary(bitstr_t *bs, size_t bytes)
Make sure you make you patches on top of nf-next.git:
https://git.kernel.org/pub/scm/linux/kernel/git/pablo/nf-next.git/
Look, bitstr_t is gone there already, we got a patch to remove
typedefs.
Anyway, I have mangled this here and it's now applied.
It would be great if your follow up patch subject is prefixes like
this:
netfilter: nf_ct_h323: blah
So we know what subsystem this is targeting to, just for the next
time.
Thanks for following up on this!
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
