* [PATCH] systemd: fix build with gcc8
@ 2018-06-01 15:42 Martin Jansa
2018-06-02 2:03 ` akuster808
2018-06-02 9:00 ` [PATCHv2] " Martin Jansa
0 siblings, 2 replies; 4+ messages in thread
From: Martin Jansa @ 2018-06-01 15:42 UTC (permalink / raw)
To: openembedded-core
Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
...ild-with-gcc8-Werror-format-truncati.patch | 53 +++++++++++++++++++
meta/recipes-core/systemd/systemd_237.bb | 1 +
2 files changed, 54 insertions(+)
create mode 100644 meta/recipes-core/systemd/systemd/0036-time-util-fix-build-with-gcc8-Werror-format-truncati.patch
diff --git a/meta/recipes-core/systemd/systemd/0036-time-util-fix-build-with-gcc8-Werror-format-truncati.patch b/meta/recipes-core/systemd/systemd/0036-time-util-fix-build-with-gcc8-Werror-format-truncati.patch
new file mode 100644
index 0000000000..e44f431073
--- /dev/null
+++ b/meta/recipes-core/systemd/systemd/0036-time-util-fix-build-with-gcc8-Werror-format-truncati.patch
@@ -0,0 +1,53 @@
+From f9fd4546c25a15629544bc4642a74d7909cc0840 Mon Sep 17 00:00:00 2001
+From: Martin Jansa <Martin.Jansa@gmail.com>
+Date: Fri, 1 Jun 2018 15:22:28 +0000
+Subject: [PATCH] time-util: fix build with gcc8 -Werror=format-truncation=
+
+* it fails with gcc8 when -O1 or -Os is used (and -ftree-vrp which is added by -O2 and higher isn't used)
+
+../git/src/basic/time-util.c: In function 'format_timespan':
+../git/src/basic/time-util.c:508:46: error: '%0*llu' directive output between 1 and 2147483647 bytes may cause result to exceed 'INT_MAX' [-Werror=format-truncation=]
+ "%s"USEC_FMT".%0*"PRI_USEC"%s",
+ ^~~~
+../git/src/basic/time-util.c:508:60: note: format string is defined here
+ "%s"USEC_FMT".%0*"PRI_USEC"%s",
+../git/src/basic/time-util.c:508:46: note: directive argument in the range [0, 18446744073709551614]
+ "%s"USEC_FMT".%0*"PRI_USEC"%s",
+ ^~~~
+../git/src/basic/time-util.c:507:37: note: 'snprintf' output 4 or more bytes (assuming 2147483651) into a destination of size 4294967295
+ k = snprintf(p, l,
+ ^~~~~~~~~~~~~~
+ "%s"USEC_FMT".%0*"PRI_USEC"%s",
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ p > buf ? " " : "",
+ ~~~~~~~~~~~~~~~~~~~
+ a,
+ ~~
+ j,
+ ~~
+ b,
+ ~~
+ table[i].suffix);
+ ~~~~~~~~~~~~~~~~
+cc1: some warnings being treated as errors
+
+Upstream-Status: Submitted https://github.com/systemd/systemd/pull/9156
+
+Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
+---
+ src/basic/time-util.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/src/basic/time-util.c b/src/basic/time-util.c
+index 0601d4fa9..db5a9cd78 100644
+--- a/src/basic/time-util.c
++++ b/src/basic/time-util.c
+@@ -484,7 +484,7 @@ char *format_timespan(char *buf, size_t l, usec_t t, usec_t accuracy) {
+ /* Let's see if we should shows this in dot notation */
+ if (t < USEC_PER_MINUTE && b > 0) {
+ usec_t cc;
+- int j;
++ char j;
+
+ j = 0;
+ for (cc = table[i].usec; cc > 1; cc /= 10)
diff --git a/meta/recipes-core/systemd/systemd_237.bb b/meta/recipes-core/systemd/systemd_237.bb
index 2e6558ded1..0113c2dc72 100644
--- a/meta/recipes-core/systemd/systemd_237.bb
+++ b/meta/recipes-core/systemd/systemd_237.bb
@@ -54,6 +54,7 @@ SRC_URI += "file://touchscreen.rules \
file://libmount.patch \
file://0034-Fix-format-truncation-compile-failure-by-typecasting.patch \
file://0035-Define-glibc-compatible-basename-for-non-glibc-syste.patch \
+ file://0036-time-util-fix-build-with-gcc8-Werror-format-truncati.patch \
"
SRC_URI_append_qemuall = " file://0001-core-device.c-Change-the-default-device-timeout-to-2.patch"
--
2.17.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] systemd: fix build with gcc8
2018-06-01 15:42 [PATCH] systemd: fix build with gcc8 Martin Jansa
@ 2018-06-02 2:03 ` akuster808
2018-06-02 8:58 ` Martin Jansa
2018-06-02 9:00 ` [PATCHv2] " Martin Jansa
1 sibling, 1 reply; 4+ messages in thread
From: akuster808 @ 2018-06-02 2:03 UTC (permalink / raw)
To: Martin Jansa, openembedded-core
On 06/01/2018 08:42 AM, Martin Jansa wrote:
> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> ---
> ...ild-with-gcc8-Werror-format-truncati.patch | 53 +++++++++++++++++++
> meta/recipes-core/systemd/systemd_237.bb | 1 +
> 2 files changed, 54 insertions(+)
I am seeing this same issue in Sumo and I have applied this patch. Its
currently running a test build.
- armin
> create mode 100644 meta/recipes-core/systemd/systemd/0036-time-util-fix-build-with-gcc8-Werror-format-truncati.patch
>
> diff --git a/meta/recipes-core/systemd/systemd/0036-time-util-fix-build-with-gcc8-Werror-format-truncati.patch b/meta/recipes-core/systemd/systemd/0036-time-util-fix-build-with-gcc8-Werror-format-truncati.patch
> new file mode 100644
> index 0000000000..e44f431073
> --- /dev/null
> +++ b/meta/recipes-core/systemd/systemd/0036-time-util-fix-build-with-gcc8-Werror-format-truncati.patch
> @@ -0,0 +1,53 @@
> +From f9fd4546c25a15629544bc4642a74d7909cc0840 Mon Sep 17 00:00:00 2001
> +From: Martin Jansa <Martin.Jansa@gmail.com>
> +Date: Fri, 1 Jun 2018 15:22:28 +0000
> +Subject: [PATCH] time-util: fix build with gcc8 -Werror=format-truncation=
> +
> +* it fails with gcc8 when -O1 or -Os is used (and -ftree-vrp which is added by -O2 and higher isn't used)
> +
> +../git/src/basic/time-util.c: In function 'format_timespan':
> +../git/src/basic/time-util.c:508:46: error: '%0*llu' directive output between 1 and 2147483647 bytes may cause result to exceed 'INT_MAX' [-Werror=format-truncation=]
> + "%s"USEC_FMT".%0*"PRI_USEC"%s",
> + ^~~~
> +../git/src/basic/time-util.c:508:60: note: format string is defined here
> + "%s"USEC_FMT".%0*"PRI_USEC"%s",
> +../git/src/basic/time-util.c:508:46: note: directive argument in the range [0, 18446744073709551614]
> + "%s"USEC_FMT".%0*"PRI_USEC"%s",
> + ^~~~
> +../git/src/basic/time-util.c:507:37: note: 'snprintf' output 4 or more bytes (assuming 2147483651) into a destination of size 4294967295
> + k = snprintf(p, l,
> + ^~~~~~~~~~~~~~
> + "%s"USEC_FMT".%0*"PRI_USEC"%s",
> + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> + p > buf ? " " : "",
> + ~~~~~~~~~~~~~~~~~~~
> + a,
> + ~~
> + j,
> + ~~
> + b,
> + ~~
> + table[i].suffix);
> + ~~~~~~~~~~~~~~~~
> +cc1: some warnings being treated as errors
> +
> +Upstream-Status: Submitted https://github.com/systemd/systemd/pull/9156
> +
> +Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> +---
> + src/basic/time-util.c | 2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/src/basic/time-util.c b/src/basic/time-util.c
> +index 0601d4fa9..db5a9cd78 100644
> +--- a/src/basic/time-util.c
> ++++ b/src/basic/time-util.c
> +@@ -484,7 +484,7 @@ char *format_timespan(char *buf, size_t l, usec_t t, usec_t accuracy) {
> + /* Let's see if we should shows this in dot notation */
> + if (t < USEC_PER_MINUTE && b > 0) {
> + usec_t cc;
> +- int j;
> ++ char j;
> +
> + j = 0;
> + for (cc = table[i].usec; cc > 1; cc /= 10)
> diff --git a/meta/recipes-core/systemd/systemd_237.bb b/meta/recipes-core/systemd/systemd_237.bb
> index 2e6558ded1..0113c2dc72 100644
> --- a/meta/recipes-core/systemd/systemd_237.bb
> +++ b/meta/recipes-core/systemd/systemd_237.bb
> @@ -54,6 +54,7 @@ SRC_URI += "file://touchscreen.rules \
> file://libmount.patch \
> file://0034-Fix-format-truncation-compile-failure-by-typecasting.patch \
> file://0035-Define-glibc-compatible-basename-for-non-glibc-syste.patch \
> + file://0036-time-util-fix-build-with-gcc8-Werror-format-truncati.patch \
> "
> SRC_URI_append_qemuall = " file://0001-core-device.c-Change-the-default-device-timeout-to-2.patch"
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] systemd: fix build with gcc8
2018-06-02 2:03 ` akuster808
@ 2018-06-02 8:58 ` Martin Jansa
0 siblings, 0 replies; 4+ messages in thread
From: Martin Jansa @ 2018-06-02 8:58 UTC (permalink / raw)
To: akuster808; +Cc: Patches and discussions about the oe-core layer
[-- Attachment #1: Type: text/plain, Size: 5017 bytes --]
I'm surprised that this is reproducible with gcc7 used in Sumo.
Khem suggested to use signed char instead, the pull-request in upstream was
updated with signed char as well:
https://github.com/systemd/systemd/pull/9156
I'll send v2.
On Sat, Jun 2, 2018 at 4:03 AM akuster808 <akuster808@gmail.com> wrote:
>
>
> On 06/01/2018 08:42 AM, Martin Jansa wrote:
> > Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> > ---
> > ...ild-with-gcc8-Werror-format-truncati.patch | 53 +++++++++++++++++++
> > meta/recipes-core/systemd/systemd_237.bb | 1 +
> > 2 files changed, 54 insertions(+)
>
> I am seeing this same issue in Sumo and I have applied this patch. Its
> currently running a test build.
> - armin
> > create mode 100644
> meta/recipes-core/systemd/systemd/0036-time-util-fix-build-with-gcc8-Werror-format-truncati.patch
> >
> > diff --git
> a/meta/recipes-core/systemd/systemd/0036-time-util-fix-build-with-gcc8-Werror-format-truncati.patch
> b/meta/recipes-core/systemd/systemd/0036-time-util-fix-build-with-gcc8-Werror-format-truncati.patch
> > new file mode 100644
> > index 0000000000..e44f431073
> > --- /dev/null
> > +++
> b/meta/recipes-core/systemd/systemd/0036-time-util-fix-build-with-gcc8-Werror-format-truncati.patch
> > @@ -0,0 +1,53 @@
> > +From f9fd4546c25a15629544bc4642a74d7909cc0840 Mon Sep 17 00:00:00 2001
> > +From: Martin Jansa <Martin.Jansa@gmail.com>
> > +Date: Fri, 1 Jun 2018 15:22:28 +0000
> > +Subject: [PATCH] time-util: fix build with gcc8
> -Werror=format-truncation=
> > +
> > +* it fails with gcc8 when -O1 or -Os is used (and -ftree-vrp which is
> added by -O2 and higher isn't used)
> > +
> > +../git/src/basic/time-util.c: In function 'format_timespan':
> > +../git/src/basic/time-util.c:508:46: error: '%0*llu' directive output
> between 1 and 2147483647 bytes may cause result to exceed 'INT_MAX'
> [-Werror=format-truncation=]
> > +
> "%s"USEC_FMT".%0*"PRI_USEC"%s",
> > + ^~~~
> > +../git/src/basic/time-util.c:508:60: note: format string is defined here
> > +
> "%s"USEC_FMT".%0*"PRI_USEC"%s",
> > +../git/src/basic/time-util.c:508:46: note: directive argument in the
> range [0, 18446744073709551614]
> > +
> "%s"USEC_FMT".%0*"PRI_USEC"%s",
> > + ^~~~
> > +../git/src/basic/time-util.c:507:37: note: 'snprintf' output 4 or more
> bytes (assuming 2147483651) into a destination of size 4294967295
> > + k = snprintf(p, l,
> > + ^~~~~~~~~~~~~~
> > +
> "%s"USEC_FMT".%0*"PRI_USEC"%s",
> > +
> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > + p > buf ? " " : "",
> > + ~~~~~~~~~~~~~~~~~~~
> > + a,
> > + ~~
> > + j,
> > + ~~
> > + b,
> > + ~~
> > + table[i].suffix);
> > + ~~~~~~~~~~~~~~~~
> > +cc1: some warnings being treated as errors
> > +
> > +Upstream-Status: Submitted https://github.com/systemd/systemd/pull/9156
> > +
> > +Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> > +---
> > + src/basic/time-util.c | 2 +-
> > + 1 file changed, 1 insertion(+), 1 deletion(-)
> > +
> > +diff --git a/src/basic/time-util.c b/src/basic/time-util.c
> > +index 0601d4fa9..db5a9cd78 100644
> > +--- a/src/basic/time-util.c
> > ++++ b/src/basic/time-util.c
> > +@@ -484,7 +484,7 @@ char *format_timespan(char *buf, size_t l, usec_t
> t, usec_t accuracy) {
> > + /* Let's see if we should shows this in dot notation */
> > + if (t < USEC_PER_MINUTE && b > 0) {
> > + usec_t cc;
> > +- int j;
> > ++ char j;
> > +
> > + j = 0;
> > + for (cc = table[i].usec; cc > 1; cc /= 10)
> > diff --git a/meta/recipes-core/systemd/systemd_237.bb
> b/meta/recipes-core/systemd/systemd_237.bb
> > index 2e6558ded1..0113c2dc72 100644
> > --- a/meta/recipes-core/systemd/systemd_237.bb
> > +++ b/meta/recipes-core/systemd/systemd_237.bb
> > @@ -54,6 +54,7 @@ SRC_URI += "file://touchscreen.rules \
> > file://libmount.patch \
> >
> file://0034-Fix-format-truncation-compile-failure-by-typecasting.patch \
> >
> file://0035-Define-glibc-compatible-basename-for-non-glibc-syste.patch \
> > +
> file://0036-time-util-fix-build-with-gcc8-Werror-format-truncati.patch \
> > "
> > SRC_URI_append_qemuall = "
> file://0001-core-device.c-Change-the-default-device-timeout-to-2.patch"
> >
>
>
[-- Attachment #2: Type: text/html, Size: 7389 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCHv2] systemd: fix build with gcc8
2018-06-01 15:42 [PATCH] systemd: fix build with gcc8 Martin Jansa
2018-06-02 2:03 ` akuster808
@ 2018-06-02 9:00 ` Martin Jansa
1 sibling, 0 replies; 4+ messages in thread
From: Martin Jansa @ 2018-06-02 9:00 UTC (permalink / raw)
To: openembedded-core
Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
...ild-with-gcc8-Werror-format-truncati.patch | 53 +++++++++++++++++++
meta/recipes-core/systemd/systemd_237.bb | 1 +
2 files changed, 54 insertions(+)
create mode 100644 meta/recipes-core/systemd/systemd/0036-time-util-fix-build-with-gcc8-Werror-format-truncati.patch
diff --git a/meta/recipes-core/systemd/systemd/0036-time-util-fix-build-with-gcc8-Werror-format-truncati.patch b/meta/recipes-core/systemd/systemd/0036-time-util-fix-build-with-gcc8-Werror-format-truncati.patch
new file mode 100644
index 0000000000..fcc7c500ab
--- /dev/null
+++ b/meta/recipes-core/systemd/systemd/0036-time-util-fix-build-with-gcc8-Werror-format-truncati.patch
@@ -0,0 +1,53 @@
+From f9fd4546c25a15629544bc4642a74d7909cc0840 Mon Sep 17 00:00:00 2001
+From: Martin Jansa <Martin.Jansa@gmail.com>
+Date: Fri, 1 Jun 2018 15:22:28 +0000
+Subject: [PATCH] time-util: fix build with gcc8 -Werror=format-truncation=
+
+* it fails with gcc8 when -O1 or -Os is used (and -ftree-vrp which is added by -O2 and higher isn't used)
+
+../git/src/basic/time-util.c: In function 'format_timespan':
+../git/src/basic/time-util.c:508:46: error: '%0*llu' directive output between 1 and 2147483647 bytes may cause result to exceed 'INT_MAX' [-Werror=format-truncation=]
+ "%s"USEC_FMT".%0*"PRI_USEC"%s",
+ ^~~~
+../git/src/basic/time-util.c:508:60: note: format string is defined here
+ "%s"USEC_FMT".%0*"PRI_USEC"%s",
+../git/src/basic/time-util.c:508:46: note: directive argument in the range [0, 18446744073709551614]
+ "%s"USEC_FMT".%0*"PRI_USEC"%s",
+ ^~~~
+../git/src/basic/time-util.c:507:37: note: 'snprintf' output 4 or more bytes (assuming 2147483651) into a destination of size 4294967295
+ k = snprintf(p, l,
+ ^~~~~~~~~~~~~~
+ "%s"USEC_FMT".%0*"PRI_USEC"%s",
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ p > buf ? " " : "",
+ ~~~~~~~~~~~~~~~~~~~
+ a,
+ ~~
+ j,
+ ~~
+ b,
+ ~~
+ table[i].suffix);
+ ~~~~~~~~~~~~~~~~
+cc1: some warnings being treated as errors
+
+Upstream-Status: Submitted https://github.com/systemd/systemd/pull/9156
+
+Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
+---
+ src/basic/time-util.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/src/basic/time-util.c b/src/basic/time-util.c
+index 0601d4fa9..db5a9cd78 100644
+--- a/src/basic/time-util.c
++++ b/src/basic/time-util.c
+@@ -484,7 +484,7 @@ char *format_timespan(char *buf, size_t l, usec_t t, usec_t accuracy) {
+ /* Let's see if we should shows this in dot notation */
+ if (t < USEC_PER_MINUTE && b > 0) {
+ usec_t cc;
+- int j;
++ signed char j;
+
+ j = 0;
+ for (cc = table[i].usec; cc > 1; cc /= 10)
diff --git a/meta/recipes-core/systemd/systemd_237.bb b/meta/recipes-core/systemd/systemd_237.bb
index 2e6558ded1..0113c2dc72 100644
--- a/meta/recipes-core/systemd/systemd_237.bb
+++ b/meta/recipes-core/systemd/systemd_237.bb
@@ -54,6 +54,7 @@ SRC_URI += "file://touchscreen.rules \
file://libmount.patch \
file://0034-Fix-format-truncation-compile-failure-by-typecasting.patch \
file://0035-Define-glibc-compatible-basename-for-non-glibc-syste.patch \
+ file://0036-time-util-fix-build-with-gcc8-Werror-format-truncati.patch \
"
SRC_URI_append_qemuall = " file://0001-core-device.c-Change-the-default-device-timeout-to-2.patch"
--
2.17.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-06-02 9:00 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-06-01 15:42 [PATCH] systemd: fix build with gcc8 Martin Jansa
2018-06-02 2:03 ` akuster808
2018-06-02 8:58 ` Martin Jansa
2018-06-02 9:00 ` [PATCHv2] " Martin Jansa
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.