public inbox for openembedded-core@lists.openembedded.org
 help / color / mirror / Atom feed
From: Steve Sakoman <steve@sakoman.com>
To: openembedded-core@lists.openembedded.org
Subject: [OE-core][kirkstone 07/31] systemd: Fix build regression with latest update
Date: Tue, 17 May 2022 08:23:53 -1000	[thread overview]
Message-ID: <53b7b68a7b84e6feef3d78e25dbdd8d053e921db.1652811454.git.steve@sakoman.com> (raw)
In-Reply-To: <cover.1652811454.git.steve@sakoman.com>

From: Khem Raj <raj.khem@gmail.com>

This happens when ptest is enabled with clang compiler

Signed-off-by: Khem Raj <raj.khem@gmail.com>
Signed-off-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit a1f51bae8e4717da2375b9a476c368554a795487)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 ...e-Use-sockaddr-pointer-type-for-bind.patch | 46 +++++++++++++++++++
 meta/recipes-core/systemd/systemd_250.5.bb    |  1 +
 2 files changed, 47 insertions(+)
 create mode 100644 meta/recipes-core/systemd/systemd/0001-resolve-Use-sockaddr-pointer-type-for-bind.patch

diff --git a/meta/recipes-core/systemd/systemd/0001-resolve-Use-sockaddr-pointer-type-for-bind.patch b/meta/recipes-core/systemd/systemd/0001-resolve-Use-sockaddr-pointer-type-for-bind.patch
new file mode 100644
index 0000000000..8567283537
--- /dev/null
+++ b/meta/recipes-core/systemd/systemd/0001-resolve-Use-sockaddr-pointer-type-for-bind.patch
@@ -0,0 +1,46 @@
+From ad1428f29196bcc88ae382ee67ff705928e2be24 Mon Sep 17 00:00:00 2001
+From: Khem Raj <raj.khem@gmail.com>
+Date: Thu, 5 May 2022 20:25:37 -0700
+Subject: [PATCH] resolve: Use sockaddr pointer type for bind()
+
+bind() expects sockaddr* but SERVER_ADDRESS is sockaddr_in type struct
+
+Fixes errors with clang e.g.
+
+../git/src/resolve/test-resolved-stream.c:112:32: error: incompatible pointer types passing 'struct sockaddr_in *' to parameter of type 'const struct sockaddr *' [-Werror,-Wincompatible-pointer-types]
+        assert_se(bind(bindfd, &SERVER_ADDRESS, sizeof(SERVER_ADDRESS)) >= 0);
+                               ^~~~~~~~~~~~~~~
+../git/src/resolve/test-resolved-stream.c:251:39: error: incompatible pointer types passing 'struct sockaddr_in *' to parameter of type 'const struct sockaddr *' [-Werror,-Wincompatible-pointer-types]
+                r = connect(clientfd, &SERVER_ADDRESS, sizeof(SERVER_ADDRESS));
+
+Upstream-Status: Submitted [https://github.com/systemd/systemd/pull/23281]
+Signed-off-by: Khem Raj <raj.khem@gmail.com>
+---
+ src/resolve/test-resolved-stream.c | 4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/src/resolve/test-resolved-stream.c b/src/resolve/test-resolved-stream.c
+index f12c729e50..504b532002 100644
+--- a/src/resolve/test-resolved-stream.c
++++ b/src/resolve/test-resolved-stream.c
+@@ -109,7 +109,7 @@ static void *tcp_dns_server(void *p) {
+ 
+         assert_se((bindfd = socket(AF_INET, SOCK_STREAM | SOCK_CLOEXEC, 0)) >= 0);
+         assert_se(setsockopt(bindfd, SOL_SOCKET, SO_REUSEADDR, &(int){1}, sizeof(int)) >= 0);
+-        assert_se(bind(bindfd, &SERVER_ADDRESS, sizeof(SERVER_ADDRESS)) >= 0);
++        assert_se(bind(bindfd, (struct sockaddr*)&SERVER_ADDRESS, sizeof(SERVER_ADDRESS)) >= 0);
+         assert_se(listen(bindfd, 1) >= 0);
+         assert_se((acceptfd = accept(bindfd, NULL, NULL)) >= 0);
+         server_handle(acceptfd);
+@@ -248,7 +248,7 @@ static void test_dns_stream(bool tls) {
+         assert_se((clientfd = socket(AF_INET, SOCK_STREAM | SOCK_CLOEXEC, 0)) >= 0);
+ 
+         for (int i = 0; i < 100; i++) {
+-                r = connect(clientfd, &SERVER_ADDRESS, sizeof(SERVER_ADDRESS));
++                r = connect(clientfd, (struct sockaddr*)&SERVER_ADDRESS, sizeof(SERVER_ADDRESS));
+                 if (r >= 0)
+                         break;
+                 usleep(EVENT_TIMEOUT_USEC / 100);
+-- 
+2.36.0
+
diff --git a/meta/recipes-core/systemd/systemd_250.5.bb b/meta/recipes-core/systemd/systemd_250.5.bb
index 2cf3b7ca88..e5a2f65202 100644
--- a/meta/recipes-core/systemd/systemd_250.5.bb
+++ b/meta/recipes-core/systemd/systemd_250.5.bb
@@ -25,6 +25,7 @@ SRC_URI += "file://touchscreen.rules \
            file://0003-implment-systemd-sysv-install-for-OE.patch \
            file://0001-systemd.pc.in-use-ROOTPREFIX-without-suffixed-slash.patch \
            file://0001-test-parse-argument-Include-signal.h.patch \
+           file://0001-resolve-Use-sockaddr-pointer-type-for-bind.patch \
            "
 
 # patches needed by musl
-- 
2.25.1



  parent reply	other threads:[~2022-05-17 18:24 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-17 18:23 [OE-core][kirkstone 00/31] Patch review Steve Sakoman
2022-05-17 18:23 ` [OE-core][kirkstone 01/31] freetype: backport patch for CVE-2022-27404 Steve Sakoman
2022-05-17 18:23 ` [OE-core][kirkstone 02/31] freetype: backport patch for CVE-2022-27405 Steve Sakoman
2022-05-17 18:23 ` [OE-core][kirkstone 03/31] freetype: backport patch for CVE-2022-27406 Steve Sakoman
2022-05-17 18:23 ` [OE-core][kirkstone 04/31] qemu: backport patch for CVE-2021-4206 Steve Sakoman
2022-05-17 18:23 ` [OE-core][kirkstone 05/31] qemu: backport patch for CVE-2021-4207 Steve Sakoman
2022-05-17 18:23 ` [OE-core][kirkstone 06/31] systemd: upgrade 250.4 -> 250.5 Steve Sakoman
2022-05-17 18:23 ` Steve Sakoman [this message]
2022-05-17 18:23 ` [OE-core][kirkstone 08/31] mesa: upgrade 22.0.0 -> 22.0.2 Steve Sakoman
2022-05-17 18:23 ` [OE-core][kirkstone 09/31] bind: upgrade 9.18.1 -> 9.18.2 Steve Sakoman
2022-05-17 18:23 ` [OE-core][kirkstone 10/31] cronie: upgrade 1.6.0 -> 1.6.1 Steve Sakoman
2022-05-17 18:23 ` [OE-core][kirkstone 11/31] epiphany: upgrade 42.0 -> 42.2 Steve Sakoman
2022-05-17 18:23 ` [OE-core][kirkstone 12/31] ffmpeg: upgrade 5.0 -> 5.0.1 Steve Sakoman
2022-05-17 18:23 ` [OE-core][kirkstone 13/31] fribidi: upgrade 1.0.11 -> 1.0.12 Steve Sakoman
2022-05-17 18:24 ` [OE-core][kirkstone 14/31] libinput: upgrade 1.19.3 -> 1.19.4 Steve Sakoman
2022-05-17 18:24 ` [OE-core][kirkstone 15/31] sqlite3: upgrade 3.38.2 -> 3.38.3 Steve Sakoman
2022-05-17 18:24 ` [OE-core][kirkstone 16/31] webkitgtk: upgrade 2.36.0 -> 2.36.1 Steve Sakoman
2022-05-17 18:24 ` [OE-core][kirkstone 17/31] xwayland: upgrade 22.1.0 -> 22.1.1 Steve Sakoman
2022-05-17 18:24 ` [OE-core][kirkstone 18/31] libxml2: Upgrade 2.9.13 -> 2.9.14 Steve Sakoman
2022-05-17 18:24 ` [OE-core][kirkstone 19/31] vim: Upgrade 8.2.4681 -> 8.2.4912 Steve Sakoman
2022-05-17 18:24 ` [OE-core][kirkstone 20/31] linux-firmware: replace mkdir by install Steve Sakoman
2022-05-17 18:24 ` [OE-core][kirkstone 21/31] linux-firmware: upgrade 20220411 -> 20220509 Steve Sakoman
2022-05-17 18:24 ` [OE-core][kirkstone 22/31] cairo: Add missing GPLv3 license checksum entry Steve Sakoman
2022-05-17 18:24 ` [OE-core][kirkstone 23/31] pypi.bbclass: Set CVE_PRODUCT to PYPI_PACKAGE Steve Sakoman
2022-05-17 18:24 ` [OE-core][kirkstone 24/31] wic/plugins/rootfs: Fix permissions when splitting rootfs folders across partitions Steve Sakoman
2022-05-17 18:24 ` [OE-core][kirkstone 25/31] e2fsprogs: update upstream status Steve Sakoman
2022-05-17 18:24 ` [OE-core][kirkstone 26/31] overlayfs: add docs about skipping QA check & service dependencies Steve Sakoman
2022-05-17 18:24 ` [OE-core][kirkstone 27/31] image.bbclass: allow overriding dependency on virtual/kernel:do_deploy Steve Sakoman
2022-05-17 18:24 ` [OE-core][kirkstone 28/31] sanity: Don't warn about make 4.2.1 for mint Steve Sakoman
2022-05-17 18:24 ` [OE-core][kirkstone 29/31] sed: Specify shell for "nobody" user in run-ptest Steve Sakoman
2022-05-17 18:24 ` [OE-core][kirkstone 30/31] strace: Don't run ptest as "nobody" Steve Sakoman
2022-05-17 18:24 ` [OE-core][kirkstone 31/31] base-passwd: Disable shell for default users Steve Sakoman

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=53b7b68a7b84e6feef3d78e25dbdd8d053e921db.1652811454.git.steve@sakoman.com \
    --to=steve@sakoman.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox