Re: [PATCH] examples/ipsec-secgw: Fix Missed Headers For FreeBSD

[Sergio Gonzalez Monroy <sergio.gonzalez.monroy@intel.com>]

Commit title shouldn't contain capital letters.

On 31/03/2016 10:12, Daniel Mrzyglod wrote:
> There were misssed typedef for u_char - <sys/types.h>
> There were missed network definitions - <netinet/in.h>

Something a bit more descriptive along the lines of:
     In FreeBSD, sys/types.h and netinet/in.h need to be included before 
netinet/ip.h.
> Failure #13: http://dpdk.org/ml/archives/test-report/2016-March/001896.html
>
> Fixes: d299106e8e31 ("examples/ipsec-secgw: add IPsec sample application")
>
> Signed-off-by: Daniel Mrzyglod <danielx.t.mrzyglod@intel.com>
> ---
>   examples/ipsec-secgw/esp.c   | 4 +++-
>   examples/ipsec-secgw/ipsec.c | 2 +-
>   examples/ipsec-secgw/rt.c    | 1 +
>   examples/ipsec-secgw/sa.c    | 2 ++
>   examples/ipsec-secgw/sp.c    | 2 ++
>   5 files changed, 9 insertions(+), 2 deletions(-)
>
> diff --git a/examples/ipsec-secgw/esp.c b/examples/ipsec-secgw/esp.c
> index ca0fc56..7b0e04c 100644
> --- a/examples/ipsec-secgw/esp.c
> +++ b/examples/ipsec-secgw/esp.c
> @@ -33,8 +33,10 @@
>   
>   #include <stdint.h>
>   #include <stdlib.h>
> -#include <netinet/ip.h>
>   #include <sys/types.h>
> +#include <sys/socket.h>

Do we need sys/socket.h?

Sergio