mirror of
https://gitlab.com/qemu-project/qemu
synced 2024-11-05 20:35:44 +00:00
tests/unit/test-char: Fix qemu_socket(), make_udp_socket() check
qemu_socket() and make_udp_socket() return a file descriptor on success, -1 on failure. The check misinterprets 0 as failure. Fix that. Signed-off-by: Markus Armbruster <armbru@redhat.com> Message-ID: <20240203080228.2766159-3-armbru@redhat.com> Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com> Reviewed-by: Eric Blake <eblake@redhat.com>
This commit is contained in:
parent
a58c439a2d
commit
f396da77ca
1 changed files with 2 additions and 2 deletions
|
@ -556,7 +556,7 @@ static int make_udp_socket(int *port)
|
|||
socklen_t alen = sizeof(addr);
|
||||
int ret, sock = qemu_socket(PF_INET, SOCK_DGRAM, 0);
|
||||
|
||||
g_assert_cmpint(sock, >, 0);
|
||||
g_assert_cmpint(sock, >=, 0);
|
||||
addr.sin_family = AF_INET ;
|
||||
addr.sin_addr.s_addr = htonl(INADDR_ANY);
|
||||
addr.sin_port = 0;
|
||||
|
@ -1407,7 +1407,7 @@ static void char_hotswap_test(void)
|
|||
|
||||
int port;
|
||||
int sock = make_udp_socket(&port);
|
||||
g_assert_cmpint(sock, >, 0);
|
||||
g_assert_cmpint(sock, >=, 0);
|
||||
|
||||
chr_args = g_strdup_printf("udp:127.0.0.1:%d", port);
|
||||
|
||||
|
|
Loading…
Reference in a new issue