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(a)lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel