Created attachment 105916 [details] [review]
Consistently save and restore errno
Some functions in dbus-transport-socket.c make a (wrapped)
socket syscall, then call other APIs, then test the result and
errno of the socket syscall.
This would break horribly if those "other APIs" overwrote errno with
their own value (... and this is part of why errno is an awful API).
Notably, if running under DBUS_VERBOSE, _dbus_verbose() is basically
fprintf(), which sets errno; and our Unix fd-passing support
makes calls of the form _dbus_verbose ("Read/wrote %i unix fds\n", n)
between the syscall and the result processing.
Maybe one day we'll convert all of dbus' syscall wrappers to either
raise a DBusError, or use the "negative errno" convention that systemd
borrowed from the Linux kernel, and in particular, we would need to
do that if we ever ported it to a platform where socket error reporting
was not basically errno. However, in practice everyone uses something
derived from BSD sockets, so "this sets errno, you know what errno is"
is a good enough internal API if we make sure to use it correctly.
Nothing calls _dbus_get_is_errno_nonzero(), so I just removed it instead
of converting it to the new calling convention.
As well as reviews, I'd appreciate maintainer opinions on whether this should go to the 1.8 or master branch. It's a bugfix, but we've never actually observed the bug, and the fix is intrusive; my personal opinion would be 1.9.
This has now been here for a month with no objections. If nobody reviews it in the next 2 weeks, I intend to go for "easier to ask forgiveness than permission" and commit it to 1.9.
(In reply to Simon McVittie from comment #1)
> This has now been here for a month with no objections. If nobody reviews it
> in the next 2 weeks, I intend to go for "easier to ask forgiveness than
> permission" and commit it to 1.9.
Done. Fixed in git for 1.9.2