[oe] [meta-networking][PATCH 1/4] ippool: Fix build errors found with hardening flags

[Martin Jansa]

I've dropped this from master-next to check if the build failures we were
discussing were caused by this. Did you change anything in here (is it v2
or just resend)?

On Wed, Jul 19, 2017 at 5:41 PM, Khem Raj <raj.khem at gmail.com> wrote:

> Signed-off-by: Khem Raj <raj.khem at gmail.com>
> Signed-off-by: Martin Jansa <Martin.Jansa at gmail.com>
> ---
>  ...timer-Check-for-return-value-of-write-API.patch | 28
> ++++++++++++++++++++++
>  .../recipes-daemons/ippool/ippool_1.3.bb           | 22 ++++++++---------
>  2 files changed, 39 insertions(+), 11 deletions(-)
>  create mode 100644 meta-networking/recipes-daemons/ippool/ippool/0001-
> usl_timer-Check-for-return-value-of-write-API.patch
>
> diff --git a/meta-networking/recipes-daemons/ippool/ippool/0001-
> usl_timer-Check-for-return-value-of-write-API.patch
> b/meta-networking/recipes-daemons/ippool/ippool/0001-
> usl_timer-Check-for-return-value-of-write-API.patch
> new file mode 100644
> index 000000000..6fb7cc5c4
> --- /dev/null
> +++ b/meta-networking/recipes-daemons/ippool/ippool/0001-
> usl_timer-Check-for-return-value-of-write-API.patch
> @@ -0,0 +1,28 @@
> +From 5d7f20c045b3c74dad2c53d65e30bd4840250082 Mon Sep 17 00:00:00 2001
> +From: Khem Raj <raj.khem at gmail.com>
> +Date: Tue, 27 Jun 2017 15:17:19 -0700
> +Subject: [PATCH] usl_timer: Check for return value of write() API
> +
> +Signed-off-by: Khem Raj <raj.khem at gmail.com>
> +---
> + usl/usl_timer.c | 4 +++-
> + 1 file changed, 3 insertions(+), 1 deletion(-)
> +
> +diff --git a/usl/usl_timer.c b/usl/usl_timer.c
> +index fda752b..d8414a6 100644
> +--- a/usl/usl_timer.c
> ++++ b/usl/usl_timer.c
> +@@ -94,7 +94,9 @@ void usl_timer_tick(void)
> +
> +       if (!usl_tick_pending) {
> +               usl_tick_pending = 1;
> +-              write(usl_tick_pipe[1], &msg, sizeof(msg));
> ++              if (write(usl_tick_pipe[1], &msg, sizeof(msg)) !=
> sizeof(msg)) {
> ++                      fprintf(stderr, "write to fd %i failed: %s\n",
> usl_tick_pipe[1], strerror(errno));
> ++              }
> +       }
> + }
> +
> +--
> +2.13.2
> +
> diff --git a/meta-networking/recipes-daemons/ippool/ippool_1.3.bb
> b/meta-networking/recipes-daemons/ippool/ippool_1.3.bb
> index 969f434cc..e1c9215ca 100644
> --- a/meta-networking/recipes-daemons/ippool/ippool_1.3.bb
> +++ b/meta-networking/recipes-daemons/ippool/ippool_1.3.bb
> @@ -11,16 +11,16 @@ HOMEPAGE = "http://www.openl2tp.org/"
>  SECTION = "console/network"
>  LICENSE = "GPLv2+"
>
> -SRC_URI = "\
> -        https://sourceforge.net/projects/openl2tp/files/${BPN}
> /${PV}/${BPN}-${PV}.tar.gz \
> -        file://ippool_usl_timer.patch \
> -        file://ippool_parallel_make_and_pic.patch \
> -        file://ippool_init.d.patch \
> -        file://always_syslog.patch \
> -        file://makefile-add-ldflags.patch \
> -        file://runtest.sh \
> -        file://ippool.service \
> -        "
> +SRC_URI = "https://sourceforge.net/projects/openl2tp/files/${BPN}
> /${PV}/${BPN}-${PV}.tar.gz \
> +           file://runtest.sh \
> +           file://ippool.service \
> +           file://ippool_usl_timer.patch \
> +           file://ippool_parallel_make_and_pic.patch \
> +           file://ippool_init.d.patch \
> +           file://always_syslog.patch \
> +           file://makefile-add-ldflags.patch \
> +           file://0001-usl_timer-Check-for-return-value-of-write-API.patch
> \
> +           "
>
>  LIC_FILES_CHKSUM = "file://LICENSE;md5=4c59283b82fc2b166455e0fc23c71c6f"
>  SRC_URI[md5sum] = "e2401e65db26a3764585b97212888fae"
> @@ -50,7 +50,7 @@ do_compile_prepend() {
>
>      # ignore the OPT_CFLAGS?= in Makefile,
>      # it should be in CFLAGS from env
> -    export OPT_CFLAGS=
> +    export OPT_CFLAGS="${SELECTED_OPTIMIZATION}"
>  }
>
>
> --
> 2.13.3
>
>