On Wed, May 21, 2014 at 03:15:04PM +0200, Jakub Hrozek wrote:
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.
Pushed to master:
58092eb1acfefefef9148789062aa9114e00372e
33d70148eac28c548b10eb3aa36ab23169cedc5a