Attachment 38282 Details for Bug 29881 – Verify that getsockname succeeded in _dbus_listen_tcp_socket

[patch] Verify that getsockname succeeded in _dbus_listen_tcp_socket

0004-Verfiy-that-getsockname-succeeded-in-_dbus_listen_tcp.diff (text/plain), 1.58 KB, created by Kristian Rietveld on 2010-08-30 03:44:39 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kristian Rietveld

Created: 2010-08-30 03:44:39 UTC

Size: 1.58 KB

patch

obsolete

>>From 2de5c93a8dbda0196c7a34420fa2a11967ff4230 Mon Sep 17 00:00:00 2001
>From: Christian Dywan <christian.dywan@lanedo.com>
>Date: Mon, 19 Jul 2010 13:05:29 +0200
>Subject: Verify that getsockname succeeded in _dbus_listen_tcp_socket
>
>---
> dbus/dbus-sysdeps-unix.c |   10 ++++++----
> 1 files changed, 6 insertions(+), 4 deletions(-)
>
>diff --git a/dbus/dbus-sysdeps-unix.c b/dbus/dbus-sysdeps-unix.c
>index 0a656da..f4d382d 100644
>--- a/dbus/dbus-sysdeps-unix.c
>+++ b/dbus/dbus-sysdeps-unix.c
>@@ -1271,16 +1271,18 @@ _dbus_listen_tcp_socket (const char     *host,
>              to use the same port */
>           if (!port || !strcmp(port, "0"))
>             {
>+              int result;
>               struct sockaddr_storage addr;
>               socklen_t addrlen;
>               char portbuf[50];
> 
>               addrlen = sizeof(addr);
>-              getsockname(fd, (struct sockaddr*) &addr, &addrlen);
>+              result = getsockname(fd, (struct sockaddr*) &addr, &addrlen);
> 
>-              if ((res = getnameinfo((struct sockaddr*)&addr, addrlen, NULL, 0,
>-                                     portbuf, sizeof(portbuf),
>-                                     NI_NUMERICHOST)) != 0)
>+              if (result != 0 ||
>+                  (res = getnameinfo ((struct sockaddr*)&addr, addrlen, NULL, 0,
>+                                      portbuf, sizeof(portbuf),
>+                                      NI_NUMERICHOST)) != 0)
>                 {
>                   dbus_set_error (error, _dbus_error_from_errno (errno),
>                                   "Failed to resolve port \"%s:%s\": %s (%s)",
>-- 
>1.7.1.1
>

Actions: View | Splinter Review

Attachments on bug 29881: 38279 | 38280 | 38281 | 38282 | 38840 | 38841 | 38906 | 38907 | 38908 | 39582 | 42166 | 42167 | 42904

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.