<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