diff options
| author | Eric Dumazet <edumazet@google.com> | 2012-05-18 18:57:34 +0000 | 
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2012-05-19 01:08:16 -0400 | 
| commit | a50feda546ac03415707a9bbcac8d6b20714db21 (patch) | |
| tree | 3c1f5d64399e713c97545ca39984d4d2efe6ca5d /net/ipv6/exthdrs.c | |
| parent | 32e9072b92a1c556a303d8d0e0d64feb667e601d (diff) | |
| download | olio-linux-3.10-a50feda546ac03415707a9bbcac8d6b20714db21.tar.xz olio-linux-3.10-a50feda546ac03415707a9bbcac8d6b20714db21.zip  | |
ipv6: bool/const conversions phase2
Mostly bool conversions, some inline removals and const additions.
Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv6/exthdrs.c')
| -rw-r--r-- | net/ipv6/exthdrs.c | 54 | 
1 files changed, 27 insertions, 27 deletions
diff --git a/net/ipv6/exthdrs.c b/net/ipv6/exthdrs.c index a3cded6a199..50ec95f9aee 100644 --- a/net/ipv6/exthdrs.c +++ b/net/ipv6/exthdrs.c @@ -96,14 +96,14 @@ EXPORT_SYMBOL_GPL(ipv6_find_tlv);  /*   *	Parsing tlv encoded headers.   * - *	Parsing function "func" returns 1, if parsing succeed - *	and 0, if it failed. + *	Parsing function "func" returns true, if parsing succeed + *	and false, if it failed.   *	It MUST NOT touch skb->h.   */  struct tlvtype_proc {  	int	type; -	int	(*func)(struct sk_buff *skb, int offset); +	bool	(*func)(struct sk_buff *skb, int offset);  };  /********************* @@ -112,11 +112,11 @@ struct tlvtype_proc {  /* An unknown option is detected, decide what to do */ -static int ip6_tlvopt_unknown(struct sk_buff *skb, int optoff) +static bool ip6_tlvopt_unknown(struct sk_buff *skb, int optoff)  {  	switch ((skb_network_header(skb)[optoff] & 0xC0) >> 6) {  	case 0: /* ignore */ -		return 1; +		return true;  	case 1: /* drop packet */  		break; @@ -129,18 +129,18 @@ static int ip6_tlvopt_unknown(struct sk_buff *skb, int optoff)  			break;  	case 2: /* send ICMP PARM PROB regardless and drop packet */  		icmpv6_param_prob(skb, ICMPV6_UNK_OPTION, optoff); -		return 0; +		return false;  	}  	kfree_skb(skb); -	return 0; +	return false;  }  /* Parse tlv encoded option header (hop-by-hop or destination) */ -static int ip6_parse_tlv(struct tlvtype_proc *procs, struct sk_buff *skb) +static bool ip6_parse_tlv(const struct tlvtype_proc *procs, struct sk_buff *skb)  { -	struct tlvtype_proc *curr; +	const struct tlvtype_proc *curr;  	const unsigned char *nh = skb_network_header(skb);  	int off = skb_network_header_len(skb);  	int len = (skb_transport_header(skb)[1] + 1) << 3; @@ -186,14 +186,14 @@ static int ip6_parse_tlv(struct tlvtype_proc *procs, struct sk_buff *skb)  					/* type specific length/alignment  					   checks will be performed in the  					   func(). */ -					if (curr->func(skb, off) == 0) -						return 0; +					if (curr->func(skb, off) == false) +						return false;  					break;  				}  			}  			if (curr->type < 0) {  				if (ip6_tlvopt_unknown(skb, off) == 0) -					return 0; +					return false;  			}  			break;  		} @@ -201,10 +201,10 @@ static int ip6_parse_tlv(struct tlvtype_proc *procs, struct sk_buff *skb)  		len -= optlen;  	}  	if (len == 0) -		return 1; +		return true;  bad:  	kfree_skb(skb); -	return 0; +	return false;  }  /***************************** @@ -212,7 +212,7 @@ bad:   *****************************/  #if defined(CONFIG_IPV6_MIP6) || defined(CONFIG_IPV6_MIP6_MODULE) -static int ipv6_dest_hao(struct sk_buff *skb, int optoff) +static bool ipv6_dest_hao(struct sk_buff *skb, int optoff)  {  	struct ipv6_destopt_hao *hao;  	struct inet6_skb_parm *opt = IP6CB(skb); @@ -266,15 +266,15 @@ static int ipv6_dest_hao(struct sk_buff *skb, int optoff)  	if (skb->tstamp.tv64 == 0)  		__net_timestamp(skb); -	return 1; +	return true;   discard:  	kfree_skb(skb); -	return 0; +	return false;  }  #endif -static struct tlvtype_proc tlvprocdestopt_lst[] = { +static const struct tlvtype_proc tlvprocdestopt_lst[] = {  #if defined(CONFIG_IPV6_MIP6) || defined(CONFIG_IPV6_MIP6_MODULE)  	{  		.type	= IPV6_TLV_HAO, @@ -579,23 +579,23 @@ static inline struct net *ipv6_skb_net(struct sk_buff *skb)  /* Router Alert as of RFC 2711 */ -static int ipv6_hop_ra(struct sk_buff *skb, int optoff) +static bool ipv6_hop_ra(struct sk_buff *skb, int optoff)  {  	const unsigned char *nh = skb_network_header(skb);  	if (nh[optoff + 1] == 2) {  		IP6CB(skb)->ra = optoff; -		return 1; +		return true;  	}  	LIMIT_NETDEBUG(KERN_DEBUG "ipv6_hop_ra: wrong RA length %d\n",  		       nh[optoff + 1]);  	kfree_skb(skb); -	return 0; +	return false;  }  /* Jumbo payload */ -static int ipv6_hop_jumbo(struct sk_buff *skb, int optoff) +static bool ipv6_hop_jumbo(struct sk_buff *skb, int optoff)  {  	const unsigned char *nh = skb_network_header(skb);  	struct net *net = ipv6_skb_net(skb); @@ -614,13 +614,13 @@ static int ipv6_hop_jumbo(struct sk_buff *skb, int optoff)  		IP6_INC_STATS_BH(net, ipv6_skb_idev(skb),  				 IPSTATS_MIB_INHDRERRORS);  		icmpv6_param_prob(skb, ICMPV6_HDR_FIELD, optoff+2); -		return 0; +		return false;  	}  	if (ipv6_hdr(skb)->payload_len) {  		IP6_INC_STATS_BH(net, ipv6_skb_idev(skb),  				 IPSTATS_MIB_INHDRERRORS);  		icmpv6_param_prob(skb, ICMPV6_HDR_FIELD, optoff); -		return 0; +		return false;  	}  	if (pkt_len > skb->len - sizeof(struct ipv6hdr)) { @@ -632,14 +632,14 @@ static int ipv6_hop_jumbo(struct sk_buff *skb, int optoff)  	if (pskb_trim_rcsum(skb, pkt_len + sizeof(struct ipv6hdr)))  		goto drop; -	return 1; +	return true;  drop:  	kfree_skb(skb); -	return 0; +	return false;  } -static struct tlvtype_proc tlvprochopopt_lst[] = { +static const struct tlvtype_proc tlvprochopopt_lst[] = {  	{  		.type	= IPV6_TLV_ROUTERALERT,  		.func	= ipv6_hop_ra,  |