diff options
author | Jani Suonpera <jani.suonpera@qt.io> | 2020-11-25 15:38:29 +0200 |
---|---|---|
committer | Jani Suonpera <jani.suonpera@qt.io> | 2020-12-04 09:43:52 +0200 |
commit | 19d99b10df41b7875b3c788bc7a9d1a88734e644 (patch) | |
tree | dc987a79b9ad1810bedfa8221533b550cf1662cc /recipes-qt/qt5/qtwebengine/chromium/0029-chromium-musl-initialize-msghdr-in-a-compatible-mann.patch | |
parent | 545be38961a2e287c64f2ae607569e88da4bfa8b (diff) | |
parent | 7a9f639fbd632dbe23af95c7b8123a0b907c6179 (diff) | |
download | meta-qt5-19d99b10df41b7875b3c788bc7a9d1a88734e644.tar.gz |
Merge remote-tracking branch 'qt/upstream/master' into 5.15
Task-number: QTBUG-88755
Change-Id: I7adf9d6bb5fb8a2b0893d1c4b142db73996f20b9
Diffstat (limited to 'recipes-qt/qt5/qtwebengine/chromium/0029-chromium-musl-initialize-msghdr-in-a-compatible-mann.patch')
-rw-r--r-- | recipes-qt/qt5/qtwebengine/chromium/0029-chromium-musl-initialize-msghdr-in-a-compatible-mann.patch | 45 |
1 files changed, 45 insertions, 0 deletions
diff --git a/recipes-qt/qt5/qtwebengine/chromium/0029-chromium-musl-initialize-msghdr-in-a-compatible-mann.patch b/recipes-qt/qt5/qtwebengine/chromium/0029-chromium-musl-initialize-msghdr-in-a-compatible-mann.patch new file mode 100644 index 00000000..72adf1cb --- /dev/null +++ b/recipes-qt/qt5/qtwebengine/chromium/0029-chromium-musl-initialize-msghdr-in-a-compatible-mann.patch | |||
@@ -0,0 +1,45 @@ | |||
1 | From 0b04cbd5b93908bad4dfc6d829048fe90a09b868 Mon Sep 17 00:00:00 2001 | ||
2 | From: Khem Raj <raj.khem@gmail.com> | ||
3 | Date: Sun, 10 May 2020 08:16:01 -0700 | ||
4 | Subject: [PATCH] chromium: musl: initialize msghdr in a compatible manner | ||
5 | |||
6 | initialize msghdr in a compatible manner | ||
7 | |||
8 | msghdr stuct from socket.h is not same between musl and glibc | ||
9 | where musl claims to be more posix compliant where as glibc seems | ||
10 | to fill whats needed for linux sizewise and chooses long enough types | ||
11 | which maybe questionable, therefore constructing a structure with explicit | ||
12 | constructor is not going to work correctly for musl and glibc at same time | ||
13 | |||
14 | see | ||
15 | https://git.musl-libc.org/cgit/musl/commit/arch/x86_64/bits/socket.h?id=7168790763cdeb794df52be6e3b39fbb021c5a64 | ||
16 | |||
17 | This fix initialized the struct to 0 first and then sets the struct elements | ||
18 | by name, so we dont have to hard code the positions of elements when initializing | ||
19 | structure | ||
20 | |||
21 | Upstream-Status: Pending | ||
22 | Signed-off-by: Khem Raj <raj.khem@gmail.com> | ||
23 | --- | ||
24 | chromium/net/socket/udp_socket_posix.cc | 8 ++++++-- | ||
25 | 1 file changed, 6 insertions(+), 2 deletions(-) | ||
26 | |||
27 | diff --git a/chromium/net/socket/udp_socket_posix.cc b/chromium/net/socket/udp_socket_posix.cc | ||
28 | index 7df6892d67e..030ff8896e4 100644 | ||
29 | --- a/chromium/net/socket/udp_socket_posix.cc | ||
30 | +++ b/chromium/net/socket/udp_socket_posix.cc | ||
31 | @@ -1199,8 +1199,12 @@ SendResult UDPSocketPosixSender::InternalSendmmsgBuffers( | ||
32 | for (auto& buffer : buffers) | ||
33 | msg_iov->push_back({const_cast<char*>(buffer->data()), buffer->length()}); | ||
34 | msgvec->reserve(buffers.size()); | ||
35 | - for (size_t j = 0; j < buffers.size(); j++) | ||
36 | - msgvec->push_back({{nullptr, 0, &msg_iov[j], 1, nullptr, 0, 0}, 0}); | ||
37 | + for (size_t j = 0; j < buffers.size(); j++) { | ||
38 | + struct msghdr m = {0}; | ||
39 | + m.msg_iov = &msg_iov[j]; | ||
40 | + m.msg_iovlen = 1; | ||
41 | + msgvec->push_back({m, 0}); | ||
42 | + } | ||
43 | int result = HANDLE_EINTR(Sendmmsg(fd, &msgvec[0], buffers.size(), 0)); | ||
44 | SendResult send_result(0, 0, std::move(buffers)); | ||
45 | if (result < 0) { | ||