On Thu, Mar 29, 2012 at 02:27:36PM -0400, Stephen Gallagher wrote:
> On Tue, 2012-03-27 at 15:27 -0400, Jakub Hrozek wrote:
> > Just a one-liner but was not exactly easy to find..
> >
> >
https://fedorahosted.org/sssd/ticket/1269
>
> Nack.
>
> On close inspection, it looks like we're not properly truncating the
> 'primary_str' variable after this. We're setting the NULL-terminator to
> be after the asterisk, which is pretty much guaranteed to fail the
> strncmp() on line 426.
The strncmp worked because we were only comparing tmp_len-1 bytes and
never got to the asterisk. But it's confusing, I fixed the NULL
terminator.