[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 08/11] sockets: Fix socket_sockaddr_to_address_unix() for abstract
From: |
Daniel P . Berrangé |
Subject: |
[PULL 08/11] sockets: Fix socket_sockaddr_to_address_unix() for abstract sockets |
Date: |
Tue, 3 Nov 2020 14:02:22 +0000 |
From: Markus Armbruster <armbru@redhat.com>
Commit 776b97d360 "qemu-sockets: add abstract UNIX domain socket
support" neglected to update socket_sockaddr_to_address_unix(). The
function returns a non-abstract socket address for abstract
sockets (wrong) with a null @path (also wrong; a non-optional QAPI str
member must never be null).
The null @path is due to confused code going back all the way to
commit 17c55decec "sockets: add helpers for creating SocketAddress
from a socket".
Add the required special case, and simplify the confused code.
Fixes: 776b97d3605ed0fc94443048fdf988c7725e38a9
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---
util/qemu-sockets.c | 14 ++++++++++++--
1 file changed, 12 insertions(+), 2 deletions(-)
diff --git a/util/qemu-sockets.c b/util/qemu-sockets.c
index 3ceaa81226..a578c434c2 100644
--- a/util/qemu-sockets.c
+++ b/util/qemu-sockets.c
@@ -1270,10 +1270,20 @@ socket_sockaddr_to_address_unix(struct sockaddr_storage
*sa,
addr = g_new0(SocketAddress, 1);
addr->type = SOCKET_ADDRESS_TYPE_UNIX;
- if (su->sun_path[0]) {
- addr->u.q_unix.path = g_strndup(su->sun_path, sizeof(su->sun_path));
+#ifdef CONFIG_LINUX
+ if (!su->sun_path[0]) {
+ /* Linux abstract socket */
+ addr->u.q_unix.path = g_strndup(su->sun_path + 1,
+ sizeof(su->sun_path) - 1);
+ addr->u.q_unix.has_abstract = true;
+ addr->u.q_unix.abstract = true;
+ addr->u.q_unix.has_tight = true;
+ addr->u.q_unix.tight = salen < sizeof(*su);
+ return addr;
}
+#endif
+ addr->u.q_unix.path = g_strndup(su->sun_path, sizeof(su->sun_path));
return addr;
}
#endif /* WIN32 */
--
2.28.0
- [PULL 00/11] Sock next patches, Daniel P . Berrangé, 2020/11/03
- [PULL 02/11] test-util-sockets: Correct to set has_abstract, has_tight, Daniel P . Berrangé, 2020/11/03
- [PULL 01/11] test-util-sockets: Plug file descriptor leak, Daniel P . Berrangé, 2020/11/03
- [PULL 03/11] test-util-sockets: Clean up SocketAddress construction, Daniel P . Berrangé, 2020/11/03
- [PULL 04/11] test-util-sockets: Factor out test_socket_unix_abstract_one(), Daniel P . Berrangé, 2020/11/03
- [PULL 05/11] test-util-sockets: Synchronize properly, don't sleep(1), Daniel P . Berrangé, 2020/11/03
- [PULL 06/11] test-util-sockets: Test the complete abstract socket matrix, Daniel P . Berrangé, 2020/11/03
- [PULL 07/11] sockets: Fix default of UnixSocketAddress member @tight, Daniel P . Berrangé, 2020/11/03
- [PULL 08/11] sockets: Fix socket_sockaddr_to_address_unix() for abstract sockets,
Daniel P . Berrangé <=
- [PULL 10/11] sockets: Bypass "replace empty @path" for abstract unix sockets, Daniel P . Berrangé, 2020/11/03
- [PULL 09/11] char-socket: Fix qemu_chr_socket_address() for abstract sockets, Daniel P . Berrangé, 2020/11/03
- [PULL 11/11] sockets: Make abstract UnixSocketAddress depend on CONFIG_LINUX, Daniel P . Berrangé, 2020/11/03
- Re: [PULL 00/11] Sock next patches, Peter Maydell, 2020/11/03