Although this can be merged into the 1st patch, the patch itself
Looks good.
On Mon, Dec 24, 2012 at 11:35:15AM +0900, Simon Horman wrote:
> Use NULL instead of 0 to denote a NULL pointer.
>
> Signed-off-by: Simon Horman <ho...@verge.net.au>
> ---
> datapath/datapath.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/datapath/datapath.c b/datapath/datapath.c
> index e537b6d..2a36fb0 100644
> --- a/datapath/datapath.c
> +++ b/datapath/datapath.c
> @@ -103,7 +103,7 @@ void skb_cb_set_mpls_bos(struct sk_buff *skb)
> unsigned char *skb_cb_mpls_bos(const struct sk_buff *skb)
> {
> return OVS_CB(skb)->mpls_bos ?
> - skb_mac_header(skb) + OVS_CB(skb)->mpls_bos : 0;
> + skb_mac_header(skb) + OVS_CB(skb)->mpls_bos : NULL;
> }
>
> ptrdiff_t skb_cb_mpls_bos_offset(const struct sk_buff *skb)
> --
> 1.7.10.4
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>
--
yamahata
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
