diff options
| author | Lukas Slebodnik <lslebodn@redhat.com> | 2016-07-25 09:11:08 +0200 |
|---|---|---|
| committer | Lukas Slebodnik <lslebodn@redhat.com> | 2016-07-25 15:49:20 +0200 |
| commit | bc7991db97482eb2ac77f7105ee4bb3d329acff7 (patch) | |
| tree | 4c3b29cb36518cc8faa3b97d39841cabfc3017b4 /src/sbus/sssd_dbus_server.c | |
| parent | 66588a6241df42a04cb9ead75cf3afb38495d74a (diff) | |
| download | sssd-bc7991db97482eb2ac77f7105ee4bb3d329acff7.tar.gz sssd-bc7991db97482eb2ac77f7105ee4bb3d329acff7.tar.xz sssd-bc7991db97482eb2ac77f7105ee4bb3d329acff7.zip | |
Amend debug messages after failure of unlink
Some messages did not have errno or name of problematic file.
There was also improper use of negative value.
The function strerror was called with -1 instead of errno
Reviewed-by: Petr Čech <pcech@redhat.com>
Diffstat (limited to 'src/sbus/sssd_dbus_server.c')
| -rw-r--r-- | src/sbus/sssd_dbus_server.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/src/sbus/sssd_dbus_server.c b/src/sbus/sssd_dbus_server.c index 0a1cace41..6cc4172a0 100644 --- a/src/sbus/sssd_dbus_server.c +++ b/src/sbus/sssd_dbus_server.c @@ -103,8 +103,9 @@ create_socket_symlink(const char *filename, const char *symlink_filename) ret = unlink(symlink_filename); if (ret != 0) { ret = errno; - DEBUG(SSSDBG_CRIT_FAILURE, "Cannot remove old symlink: [%d][%s].\n", - ret, strerror(ret)); + DEBUG(SSSDBG_CRIT_FAILURE, + "Cannot remove old symlink '%s': [%d][%s].\n", + symlink_filename, ret, strerror(ret)); return EIO; } errno = 0; @@ -351,8 +352,8 @@ done: if (tmp_ret != EOK) { tmp_ret = errno; DEBUG(SSSDBG_MINOR_FAILURE, - "Failed to remove symbolic link: %d [%s]!\n", - tmp_ret, sss_strerror(tmp_ret)); + "Failed to remove symbolic link '%s': %d [%s]!\n", + symlink_filename, tmp_ret, sss_strerror(tmp_ret)); } } talloc_free(tmp_ctx); |