From: Alexandre Belloni <alexandre.belloni@bootlin.com>
To: Andrej Valek <andrej.valek@siemens.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [OE-core][PATCH] busybox: 1.35.0 -> 1.36.0
Date: Wed, 11 Jan 2023 11:54:02 +0100 [thread overview]
Message-ID: <Y76VSuZevS+O23sv@mail.local> (raw)
In-Reply-To: <20230106110505.443726-1-andrej.valek@siemens.com>
This generates a warning:
WARNING: busybox-1.36.0-r0 do_package_qa: QA Issue: busybox: ELF binary /bin/busybox.nosuid has relocations in .text [textrel]
https://autobuilder.yoctoproject.org/typhoon/#/builders/48/builds/6507/steps/11/logs/stdio
On 06/01/2023 12:05:05+0100, Andrej Valek wrote:
> - update to next (un)stable version 1.36.0
> - refresh defconfig
> - disable new applets (tree, tsort, seedrng)
> - use hw-accel for sha1/256 sums when available
> - remove and refresh already merged patches
>
> Signed-off-by: Andrej Valek <andrej.valek@siemens.com>
> ---
> ...ab_1.35.0.bb => busybox-inittab_1.36.0.bb} | 0
> .../0001-devmem-add-128-bit-width.patch | 128 ------------------
> .../busybox/busybox/CVE-2022-30065.patch | 29 ----
> meta/recipes-core/busybox/busybox/defconfig | 10 +-
> .../busybox/busybox/recognize_connmand.patch | 10 +-
> meta/recipes-core/busybox/busybox/sha1sum.cfg | 2 +
> .../busybox/busybox/sha256sum.cfg | 1 +
> .../{busybox_1.35.0.bb => busybox_1.36.0.bb} | 4 +-
> 8 files changed, 17 insertions(+), 167 deletions(-)
> rename meta/recipes-core/busybox/{busybox-inittab_1.35.0.bb => busybox-inittab_1.36.0.bb} (100%)
> delete mode 100644 meta/recipes-core/busybox/busybox/0001-devmem-add-128-bit-width.patch
> delete mode 100644 meta/recipes-core/busybox/busybox/CVE-2022-30065.patch
> rename meta/recipes-core/busybox/{busybox_1.35.0.bb => busybox_1.36.0.bb} (92%)
>
> diff --git a/meta/recipes-core/busybox/busybox-inittab_1.35.0.bb b/meta/recipes-core/busybox/busybox-inittab_1.36.0.bb
> similarity index 100%
> rename from meta/recipes-core/busybox/busybox-inittab_1.35.0.bb
> rename to meta/recipes-core/busybox/busybox-inittab_1.36.0.bb
> diff --git a/meta/recipes-core/busybox/busybox/0001-devmem-add-128-bit-width.patch b/meta/recipes-core/busybox/busybox/0001-devmem-add-128-bit-width.patch
> deleted file mode 100644
> index 985e2bf1d9..0000000000
> --- a/meta/recipes-core/busybox/busybox/0001-devmem-add-128-bit-width.patch
> +++ /dev/null
> @@ -1,128 +0,0 @@
> -From d432049f288c9acdc4a7caa729c68ceba3c5dca1 Mon Sep 17 00:00:00 2001
> -From: Aaro Koskinen <aaro.koskinen@nokia.com>
> -Date: Thu, 25 Aug 2022 18:47:02 +0300
> -Subject: [PATCH] devmem: add 128-bit width
> -
> -Add 128-bit width if the compiler provides the needed type.
> -
> -function old new delta
> -devmem_main 405 464 +59
> -.rodata 109025 109043 +18
> -------------------------------------------------------------------------------
> -(add/remove: 0/0 grow/shrink: 2/0 up/down: 77/0) Total: 77 bytes
> -
> -Upstream-Status: Backport [https://git.busybox.net/busybox/commit/?id=d432049f288c9acdc4a7caa729c68ceba3c5dca1]
> -
> -Signed-off-by: Aaro Koskinen <aaro.koskinen@nokia.com>
> -Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
> -Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
> -Signed-off-by: Mingli Yu <mingli.yu@windriver.com>
> ----
> - miscutils/devmem.c | 68 ++++++++++++++++++++++++++++++----------------
> - 1 file changed, 44 insertions(+), 24 deletions(-)
> -
> -diff --git a/miscutils/devmem.c b/miscutils/devmem.c
> -index f9f0276bc..f21621bd6 100644
> ---- a/miscutils/devmem.c
> -+++ b/miscutils/devmem.c
> -@@ -29,7 +29,6 @@ int devmem_main(int argc UNUSED_PARAM, char **argv)
> - {
> - void *map_base, *virt_addr;
> - uint64_t read_result;
> -- uint64_t writeval = writeval; /* for compiler */
> - off_t target;
> - unsigned page_size, mapped_size, offset_in_page;
> - int fd;
> -@@ -64,9 +63,6 @@ int devmem_main(int argc UNUSED_PARAM, char **argv)
> - width = strchrnul(bhwl, (argv[2][0] | 0x20)) - bhwl;
> - width = sizes[width];
> - }
> -- /* VALUE */
> -- if (argv[3])
> -- writeval = bb_strtoull(argv[3], NULL, 0);
> - } else { /* argv[2] == NULL */
> - /* make argv[3] to be a valid thing to fetch */
> - argv--;
> -@@ -96,28 +92,46 @@ int devmem_main(int argc UNUSED_PARAM, char **argv)
> - virt_addr = (char*)map_base + offset_in_page;
> -
> - if (!argv[3]) {
> -- switch (width) {
> -- case 8:
> -- read_result = *(volatile uint8_t*)virt_addr;
> -- break;
> -- case 16:
> -- read_result = *(volatile uint16_t*)virt_addr;
> -- break;
> -- case 32:
> -- read_result = *(volatile uint32_t*)virt_addr;
> -- break;
> -- case 64:
> -- read_result = *(volatile uint64_t*)virt_addr;
> -- break;
> -- default:
> -- bb_simple_error_msg_and_die("bad width");
> -+#ifdef __SIZEOF_INT128__
> -+ if (width == 128) {
> -+ unsigned __int128 rd =
> -+ *(volatile unsigned __int128 *)virt_addr;
> -+ printf("0x%016llX%016llX\n",
> -+ (unsigned long long)(uint64_t)(rd >> 64),
> -+ (unsigned long long)(uint64_t)rd
> -+ );
> -+ } else
> -+#endif
> -+ {
> -+ switch (width) {
> -+ case 8:
> -+ read_result = *(volatile uint8_t*)virt_addr;
> -+ break;
> -+ case 16:
> -+ read_result = *(volatile uint16_t*)virt_addr;
> -+ break;
> -+ case 32:
> -+ read_result = *(volatile uint32_t*)virt_addr;
> -+ break;
> -+ case 64:
> -+ read_result = *(volatile uint64_t*)virt_addr;
> -+ break;
> -+ default:
> -+ bb_simple_error_msg_and_die("bad width");
> -+ }
> -+// printf("Value at address 0x%"OFF_FMT"X (%p): 0x%llX\n",
> -+// target, virt_addr,
> -+// (unsigned long long)read_result);
> -+ /* Zero-padded output shows the width of access just done */
> -+ printf("0x%0*llX\n", (width >> 2), (unsigned long long)read_result);
> - }
> --// printf("Value at address 0x%"OFF_FMT"X (%p): 0x%llX\n",
> --// target, virt_addr,
> --// (unsigned long long)read_result);
> -- /* Zero-padded output shows the width of access just done */
> -- printf("0x%0*llX\n", (width >> 2), (unsigned long long)read_result);
> - } else {
> -+ /* parse VALUE */
> -+#ifdef __SIZEOF_INT128__
> -+ unsigned __int128 writeval = strtoumax(argv[3], NULL, 0);
> -+#else
> -+ uint64_t writeval = bb_strtoull(argv[3], NULL, 0);
> -+#endif
> - switch (width) {
> - case 8:
> - *(volatile uint8_t*)virt_addr = writeval;
> -@@ -135,6 +149,12 @@ int devmem_main(int argc UNUSED_PARAM, char **argv)
> - *(volatile uint64_t*)virt_addr = writeval;
> - // read_result = *(volatile uint64_t*)virt_addr;
> - break;
> -+#ifdef __SIZEOF_INT128__
> -+ case 128:
> -+ *(volatile unsigned __int128 *)virt_addr = writeval;
> -+// read_result = *(volatile uint64_t*)virt_addr;
> -+ break;
> -+#endif
> - default:
> - bb_simple_error_msg_and_die("bad width");
> - }
> ---
> -2.25.1
> -
> diff --git a/meta/recipes-core/busybox/busybox/CVE-2022-30065.patch b/meta/recipes-core/busybox/busybox/CVE-2022-30065.patch
> deleted file mode 100644
> index 25ad653b25..0000000000
> --- a/meta/recipes-core/busybox/busybox/CVE-2022-30065.patch
> +++ /dev/null
> @@ -1,29 +0,0 @@
> -Fix use-after-free in awk.
> -
> -CVE: CVE-2022-30065
> -Upstream-Status: Submitted [http://lists.busybox.net/pipermail/busybox/2022-June/089768.html]
> -Signed-off-by: Ross Burton <ross.burton@arm.com>
> -
> -fixes https://bugs.busybox.net/show_bug.cgi?id=14781
> -
> -Signed-off-by: Natanael Copa <ncopa at alpinelinux.org>
> ----
> - editors/awk.c | 3 +++
> - 1 file changed, 3 insertions(+)
> -
> -diff --git a/editors/awk.c b/editors/awk.c
> -index 079d0bde5..728ee8685 100644
> ---- a/editors/awk.c
> -+++ b/editors/awk.c
> -@@ -3128,6 +3128,9 @@ static var *evaluate(node *op, var *res)
> -
> - case XC( OC_MOVE ):
> - debug_printf_eval("MOVE\n");
> -+ /* make sure that we never return a temp var */
> -+ if (L.v == TMPVAR0)
> -+ L.v = res;
> - /* if source is a temporary string, jusk relink it to dest */
> - if (R.v == TMPVAR1
> - && !(R.v->type & VF_NUMBER)
> ---
> -2.36.1
> diff --git a/meta/recipes-core/busybox/busybox/defconfig b/meta/recipes-core/busybox/busybox/defconfig
> index 5e1e1f5638..3d36447c63 100644
> --- a/meta/recipes-core/busybox/busybox/defconfig
> +++ b/meta/recipes-core/busybox/busybox/defconfig
> @@ -1,7 +1,7 @@
> #
> # Automatically generated make config: don't edit
> -# Busybox version: 1.35.0
> -# Sun Dec 26 16:55:55 2021
> +# Busybox version: 1.36.0
> +# Tue Jan 3 14:17:01 2023
> #
> CONFIG_HAVE_DOT_CONFIG=y
>
> @@ -123,6 +123,9 @@ CONFIG_UNICODE_WIDE_WCHARS=y
> # CONFIG_UNICODE_BIDI_SUPPORT is not set
> # CONFIG_UNICODE_NEUTRAL_TABLE is not set
> # CONFIG_UNICODE_PRESERVE_BROKEN is not set
> +# CONFIG_LOOP_CONFIGURE is not set
> +# CONFIG_NO_LOOP_CONFIGURE is not set
> +CONFIG_TRY_LOOP_CONFIGURE=y
>
> #
> # Applets
> @@ -338,6 +341,7 @@ CONFIG_FEATURE_TR_CLASSES=y
> # CONFIG_FEATURE_TR_EQUIV is not set
> CONFIG_TRUE=y
> # CONFIG_TRUNCATE is not set
> +# CONFIG_TSORT is not set
> CONFIG_TTY=y
> CONFIG_UNAME=y
> CONFIG_UNAME_OSNAME="GNU/Linux"
> @@ -831,10 +835,12 @@ CONFIG_MICROCOM=y
> CONFIG_RFKILL=y
> # CONFIG_RUNLEVEL is not set
> # CONFIG_RX is not set
> +# CONFIG_SEEDRNG is not set
> # CONFIG_SETFATTR is not set
> # CONFIG_SETSERIAL is not set
> CONFIG_STRINGS=y
> CONFIG_TIME=y
> +# CONFIG_TREE is not set
> CONFIG_TS=y
> # CONFIG_TTYSIZE is not set
> # CONFIG_UBIATTACH is not set
> diff --git a/meta/recipes-core/busybox/busybox/recognize_connmand.patch b/meta/recipes-core/busybox/busybox/recognize_connmand.patch
> index f42c74caad..4f28beb556 100644
> --- a/meta/recipes-core/busybox/busybox/recognize_connmand.patch
> +++ b/meta/recipes-core/busybox/busybox/recognize_connmand.patch
> @@ -4,14 +4,14 @@ Upstream-Status: Inappropriate [OE-Core]
>
> Signed-off-by: Saul Wold <sgw@linux.intel.com>
>
> -Index: busybox-1.22.1/networking/ifupdown.c
> +Index: busybox-1.36.0/networking/ifupdown.c
> ===================================================================
> ---- busybox-1.22.1.orig/networking/ifupdown.c
> -+++ busybox-1.22.1/networking/ifupdown.c
> -@@ -521,6 +521,10 @@ struct dhcp_client_t {
> +--- busybox-1.36.0.orig/networking/ifupdown.c
> ++++ busybox-1.36.0/networking/ifupdown.c
> +@@ -628,6 +628,10 @@ struct dhcp_client_t {
> };
>
> - static const struct dhcp_client_t ext_dhcp_clients[] = {
> + static const struct dhcp_client_t ext_dhcp_clients[] ALIGN_PTR = {
> + { "connmand",
> + "true",
> + "true",
> diff --git a/meta/recipes-core/busybox/busybox/sha1sum.cfg b/meta/recipes-core/busybox/busybox/sha1sum.cfg
> index 20e72d9263..01137a2486 100644
> --- a/meta/recipes-core/busybox/busybox/sha1sum.cfg
> +++ b/meta/recipes-core/busybox/busybox/sha1sum.cfg
> @@ -1 +1,3 @@
> CONFIG_SHA1SUM=y
> +CONFIG_SHA1_SMALL=3
> +CONFIG_SHA1_HWACCEL=y
> diff --git a/meta/recipes-core/busybox/busybox/sha256sum.cfg b/meta/recipes-core/busybox/busybox/sha256sum.cfg
> index ce652ae4c6..a5b0ab22d1 100644
> --- a/meta/recipes-core/busybox/busybox/sha256sum.cfg
> +++ b/meta/recipes-core/busybox/busybox/sha256sum.cfg
> @@ -1 +1,2 @@
> CONFIG_SHA256SUM=y
> +CONFIG_SHA256_HWACCEL=y
> diff --git a/meta/recipes-core/busybox/busybox_1.35.0.bb b/meta/recipes-core/busybox/busybox_1.36.0.bb
> similarity index 92%
> rename from meta/recipes-core/busybox/busybox_1.35.0.bb
> rename to meta/recipes-core/busybox/busybox_1.36.0.bb
> index e9ca6fdb1a..e46d43e681 100644
> --- a/meta/recipes-core/busybox/busybox_1.35.0.bb
> +++ b/meta/recipes-core/busybox/busybox_1.36.0.bb
> @@ -49,9 +49,7 @@ SRC_URI = "https://busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \
> file://0001-sysctl-ignore-EIO-of-stable_secret-below-proc-sys-ne.patch \
> file://0001-libbb-sockaddr2str-ensure-only-printable-characters-.patch \
> file://0002-nslookup-sanitize-all-printed-strings-with-printable.patch \
> - file://CVE-2022-30065.patch \
> - file://0001-devmem-add-128-bit-width.patch \
> "
> SRC_URI:append:libc-musl = " file://musl.cfg "
>
> -SRC_URI[tarball.sha256sum] = "faeeb244c35a348a334f4a59e44626ee870fb07b6884d68c10ae8bc19f83a694"
> +SRC_URI[tarball.sha256sum] = "542750c8af7cb2630e201780b4f99f3dcceeb06f505b479ec68241c1e6af61a5"
> --
> 2.34.3
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#175575): https://lists.openembedded.org/g/openembedded-core/message/175575
> Mute This Topic: https://lists.openembedded.org/mt/96091213/3617179
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alexandre.belloni@bootlin.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
--
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
next prev parent reply other threads:[~2023-01-11 10:54 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-01-06 11:05 [OE-core][PATCH] busybox: 1.35.0 -> 1.36.0 Andrej Valek
2023-01-07 6:23 ` Khem Raj
2023-01-08 13:17 ` Valek, Andrej
2023-01-08 14:34 ` Khem Raj
2023-01-09 6:29 ` Valek, Andrej
2023-01-09 20:55 ` Khem Raj
2023-01-11 10:54 ` Alexandre Belloni [this message]
2023-01-11 19:33 ` Khem Raj
2023-01-13 8:30 ` Valek, Andrej
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Y76VSuZevS+O23sv@mail.local \
--to=alexandre.belloni@bootlin.com \
--cc=andrej.valek@siemens.com \
--cc=openembedded-core@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.