On Sun, Jan 8, 2023 at 5:17 AM Valek, Andrej <andrej.va...@siemens.com> wrote:
>
> Hello Raj,
>
> Which applets have the problem, klogd and syslogd, or? I see there some
> kind of libc segmentation.
>
> Maybe we should unset the CONFIG_STATIC_LIBGCC=y which could be an
> improvement at all.

so far its klogd, syslogd and sh

>
> Regards,
> Andrej
>
> On Fri, 2023-01-06 at 22:23 -0800, Khem Raj wrote:
> > some applets are segfaulting on qemux86/musl
> >
> > [    2.754687] klogd[202]: segfault at 56da7e ip b7f4d668 sp bfdb0300
> > error 7 in libc.so[b7eda000+76000]
> > [    2.759018] syslogd[203]: segfault at 506a7e ip b7f74668 sp
> > bf997dd0 error 7 in libc.so[b7f01000+76000]
> > [   61.264333] sh[279]: segfault at 4d4a7e ip b7f5d668 sp bfff6610
> > error 7 in libc.so[b7eea000+76000]
> >
> > and it bails out logging in.
> >
> > On Fri, Jan 6, 2023 at 3:05 AM Andrej Valek
> > <andrej.va...@siemens.com> 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.va...@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.koski...@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.koski...@nokia.com>
> > > -Signed-off-by: Aaro Koskinen <aaro.koski...@iki.fi>
> > > -Signed-off-by: Denys Vlasenko <vda.li...@googlemail.com>
> > > -Signed-off-by: Mingli Yu <mingli...@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.bur...@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 <s...@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 (#175630): 
https://lists.openembedded.org/g/openembedded-core/message/175630
Mute This Topic: https://lists.openembedded.org/mt/96091213/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to