On Wed, May 21, 2014 at 03:58:52PM +0300, Alexander Bokovoy wrote:
On Tue, 20 May 2014, Jakub Hrozek wrote:
>On Tue, May 13, 2014 at 11:45:29AM +0200, Pavel Reichl wrote:
>>Hello,
>>
>>Alexander has prepared attached patches, but he was not able to post
>>them himself as he is currently having bad connection.
>>
>>Pavel Reichl
>
>I've tested the patches atop my patch called "AD: Initialize
>user_map_cnt in server mode" that I sent to master today. It's a good
>start but I would like to propose two amendments that I attach.
>
>One makes sure the search by SID also includes UID which we read later,
>the second makes sure ENOENT or an empty message is handled gracefully
>and doesn't pollute the logs with a DEBUG message.
>
>If you agree, we can apply these patches atop yours or squash them in, I
>don't mind either way.
>
>The second patch looks good to me.
ACK to both. I'm fine with squashing the patches together.
Thanks for the review, for posterity I have attached the patches I'm
going to push.