logind: change to -1+errno
authorDavid Herrmann <dh.herrmann@gmail.com>
Thu, 21 Nov 2013 08:14:46 +0000 (09:14 +0100)
committerKristian Høgsberg <krh@bitplanet.net>
Fri, 22 Nov 2013 00:28:08 +0000 (16:28 -0800)
commitfcb6bf43a4af27d94f34bc00ca46678aa5d12d46
treef8fa77bc964e9d7def886af9b7b988e809a7d503
parentf86c39058aa20de2c2ca352b27b35ddd14c811e0
logind: change to -1+errno

Set errno and return -1 in public API calls like all other weston code
does. Most systemd+dbus calls return negative error-codes instead of -1
and setting errno. Thus, we need to explicitly set errno before returning.

Also note that we must set errno _after_ the cleanup path. Calling
functions like close() in the cleanup path might overwrite errno (which is
not what we want). So protect errno until the final return -1;
src/logind-util.c