On 07/16/2015 02:54 PM, Lukas Slebodnik wrote:
On (16/07/15 14:35), Pavel Reichl wrote:
On 07/14/2015 01:21 PM, Lukas Slebodnik wrote:
+++ b/src/db/sysdb_search.c
@@ -1613,10 +1613,11 @@ errno_t sysdb_get_real_name(TALLOC_CTX *mem_ctx,
                                                   &res);
                 if (ret == EOK && res->count == 1) {
                     msg = res->msgs[0];
-                } else {
+                } else if (ret != ENOENT) {
                     DEBUG(SSSDBG_OP_FAILURE,
-                          "sysdb_search_object_by_uuid did not return a " \
-                          "single result.\n");
+                          "sysdb_search_object_by_uuid failed or returned "
+                          "more than one result [%d][%s].\n",
+                          ret, strerror(ret));
Hello,

I asked you to replace 'strerror' by 'sss_strerror'. But I don't think it's
too important. Local CI passed, so ACK.

Just one message contained sss_strerror. It might have been caused by
improper merging after rebase.

Thank you for a careful review and updated version is attached.

LS

ACK

_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel