[NETFILTER]: Parenthesize macro parameters
Parenthesize macro parameters. Signed-off-by: Jan Engelhardt <jengelh@computergmbh.de> Signed-off-by: Patrick McHardy <kaber@trash.net> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
parent
643a2c15a4
commit
e79ec50b95
10 changed files with 19 additions and 16 deletions
|
@ -84,7 +84,7 @@ static inline int arp_packet_match(const struct arphdr *arphdr,
|
|||
__be32 src_ipaddr, tgt_ipaddr;
|
||||
int i, ret;
|
||||
|
||||
#define FWINV(bool,invflg) ((bool) ^ !!(arpinfo->invflags & invflg))
|
||||
#define FWINV(bool, invflg) ((bool) ^ !!(arpinfo->invflags & (invflg)))
|
||||
|
||||
if (FWINV((arphdr->ar_op & arpinfo->arpop_mask) != arpinfo->arpop,
|
||||
ARPT_INV_ARPOP)) {
|
||||
|
@ -180,6 +180,7 @@ static inline int arp_packet_match(const struct arphdr *arphdr,
|
|||
}
|
||||
|
||||
return 1;
|
||||
#undef FWINV
|
||||
}
|
||||
|
||||
static inline int arp_checkentry(const struct arpt_arp *arp)
|
||||
|
|
|
@ -85,7 +85,7 @@ ip_packet_match(const struct iphdr *ip,
|
|||
size_t i;
|
||||
unsigned long ret;
|
||||
|
||||
#define FWINV(bool,invflg) ((bool) ^ !!(ipinfo->invflags & invflg))
|
||||
#define FWINV(bool, invflg) ((bool) ^ !!(ipinfo->invflags & (invflg)))
|
||||
|
||||
if (FWINV((ip->saddr&ipinfo->smsk.s_addr) != ipinfo->src.s_addr,
|
||||
IPT_INV_SRCIP)
|
||||
|
@ -216,6 +216,7 @@ unconditional(const struct ipt_ip *ip)
|
|||
return 0;
|
||||
|
||||
return 1;
|
||||
#undef FWINV
|
||||
}
|
||||
|
||||
#if defined(CONFIG_NETFILTER_XT_TARGET_TRACE) || \
|
||||
|
|
|
@ -60,7 +60,7 @@ MODULE_ALIAS("ip_nat_snmp_basic");
|
|||
|
||||
#define SNMP_PORT 161
|
||||
#define SNMP_TRAP_PORT 162
|
||||
#define NOCT1(n) (*(u8 *)n)
|
||||
#define NOCT1(n) (*(u8 *)(n))
|
||||
|
||||
static int debug;
|
||||
static DEFINE_SPINLOCK(snmp_lock);
|
||||
|
|
|
@ -102,7 +102,7 @@ ip6_packet_match(const struct sk_buff *skb,
|
|||
unsigned long ret;
|
||||
const struct ipv6hdr *ipv6 = ipv6_hdr(skb);
|
||||
|
||||
#define FWINV(bool,invflg) ((bool) ^ !!(ip6info->invflags & invflg))
|
||||
#define FWINV(bool, invflg) ((bool) ^ !!(ip6info->invflags & (invflg)))
|
||||
|
||||
if (FWINV(ipv6_masked_addr_cmp(&ipv6->saddr, &ip6info->smsk,
|
||||
&ip6info->src), IP6T_INV_SRCIP)
|
||||
|
|
|
@ -100,10 +100,10 @@ typedef struct {
|
|||
} bitstr_t;
|
||||
|
||||
/* Tool Functions */
|
||||
#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)
|
||||
#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 get_len(bitstr_t * bs);
|
||||
static unsigned get_bit(bitstr_t * bs);
|
||||
static unsigned get_bits(bitstr_t * bs, unsigned b);
|
||||
|
|
|
@ -534,7 +534,7 @@ static int ctnetlink_done(struct netlink_callback *cb)
|
|||
return 0;
|
||||
}
|
||||
|
||||
#define L3PROTO(ct) ct->tuplehash[IP_CT_DIR_ORIGINAL].tuple.src.l3num
|
||||
#define L3PROTO(ct) (ct)->tuplehash[IP_CT_DIR_ORIGINAL].tuple.src.l3num
|
||||
|
||||
static int
|
||||
ctnetlink_dump_table(struct sk_buff *skb, struct netlink_callback *cb)
|
||||
|
|
|
@ -186,10 +186,10 @@ static int sctp_print_conntrack(struct seq_file *s,
|
|||
}
|
||||
|
||||
#define for_each_sctp_chunk(skb, sch, _sch, offset, dataoff, count) \
|
||||
for (offset = dataoff + sizeof(sctp_sctphdr_t), count = 0; \
|
||||
offset < skb->len && \
|
||||
(sch = skb_header_pointer(skb, offset, sizeof(_sch), &_sch)); \
|
||||
offset += (ntohs(sch->length) + 3) & ~3, count++)
|
||||
for ((offset) = (dataoff) + sizeof(sctp_sctphdr_t), (count) = 0; \
|
||||
(offset) < (skb)->len && \
|
||||
((sch) = skb_header_pointer((skb), (offset), sizeof(_sch), &(_sch))); \
|
||||
(offset) += (ntohs((sch)->length) + 3) & ~3, (count)++)
|
||||
|
||||
/* Some validity checks to make sure the chunks are fine */
|
||||
static int do_basic_checks(struct nf_conn *conntrack,
|
||||
|
|
|
@ -32,7 +32,7 @@ conntrack_mt(const struct sk_buff *skb, const struct net_device *in,
|
|||
|
||||
ct = nf_ct_get(skb, &ctinfo);
|
||||
|
||||
#define FWINV(bool,invflg) ((bool) ^ !!(sinfo->invflags & invflg))
|
||||
#define FWINV(bool, invflg) ((bool) ^ !!(sinfo->invflags & (invflg)))
|
||||
|
||||
if (ct == &nf_conntrack_untracked)
|
||||
statebit = XT_CONNTRACK_STATE_UNTRACKED;
|
||||
|
@ -108,6 +108,7 @@ conntrack_mt(const struct sk_buff *skb, const struct net_device *in,
|
|||
return false;
|
||||
}
|
||||
return true;
|
||||
#undef FWINV
|
||||
}
|
||||
|
||||
static bool
|
||||
|
|
|
@ -38,7 +38,7 @@ match_xfrm_state(const struct xfrm_state *x, const struct xt_policy_elem *e,
|
|||
unsigned short family)
|
||||
{
|
||||
#define MATCH_ADDR(x,y,z) (!e->match.x || \
|
||||
(xt_addr_cmp(&e->x, &e->y, z, family) \
|
||||
(xt_addr_cmp(&e->x, &e->y, (z), family) \
|
||||
^ e->invert.x))
|
||||
#define MATCH(x,y) (!e->match.x || ((e->x == (y)) ^ e->invert.x))
|
||||
|
||||
|
|
|
@ -37,7 +37,7 @@ string_mt(const struct sk_buff *skb, const struct net_device *in,
|
|||
!= UINT_MAX) ^ conf->invert;
|
||||
}
|
||||
|
||||
#define STRING_TEXT_PRIV(m) ((struct xt_string_info *) m)
|
||||
#define STRING_TEXT_PRIV(m) ((struct xt_string_info *)(m))
|
||||
|
||||
static bool
|
||||
string_mt_check(const char *tablename, const void *ip,
|
||||
|
|
Loading…
Reference in a new issue