On Thu, Jul 16, 2015 at 04:38:03PM +0200, Pavel Reichl wrote:
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
* master: 890ae77c52e36f222655d710439d2fe25f15101b