<listen> element of the configuration file

Havoc Pennington hp at redhat.com
Fri Mar 9 14:34:03 PST 2007


Ralf Habacker wrote:
> Index: dbus/dbus-transport-socket.c
> ===================================================================
> RCS file: /cvs/dbus/dbus/dbus/dbus-transport-socket.c,v
> retrieving revision 1.4
> diff -u -r1.4 dbus-transport-socket.c
> --- dbus/dbus-transport-socket.c	20 Oct 2006 03:05:00 -0000	1.4
> +++ dbus/dbus-transport-socket.c	9 Mar 2007 20:40:30 -0000
> @@ -1294,7 +1294,7 @@
>        sresult = _dbus_string_parse_int (&str, 0, &lport, NULL);
>        _dbus_string_free (&str);
>            
> -      if (sresult == FALSE || lport <= 0 || lport > 65535)
> +      if (sresult == FALSE || lport < 0 || lport > 65535)
>          {

I don't think a port of 0 makes sense on the client side, does it?

Otherwise the patch looks fine to me, thanks to both of you.

Havoc



More information about the dbus mailing list