[SSSD] [PATCH] sdap-tests: Fix off by one
Jakub Hrozek
jhrozek at redhat.com
Wed Jul 9 15:34:07 UTC 2014
On Wed, Jul 09, 2014 at 03:56:16PM +0200, Pavel Reichl wrote:
> On Wed, 2014-07-09 at 13:02 +0200, Lukas Slebodnik wrote:
> > ehlo,
> >
> > 1st patch fix valgrind warning
> > 2nd patch fix linking of sdap-tests on platfrms with disabled link_all_deplibs
> >
> > LS
> > _______________________________________________
> > sssd-devel mailing list
> > sssd-devel at lists.fedorahosted.org
> > https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
>
> Hi,
>
> I was just thinking if in the first patch we should also test the length
> of strings: "strlen("1234") == key_len". But on the second thought I
> don't think this is a good idea...we should simply assume that
> sss_base64_decode returns correct size, right?
Yes, we should have (and I'm not sure we do actually) a test for
sss_base64_{encode,decode} which should be ran separately and each test
should exercise just one part of the code.
>
> Anyway patches seems OK to me.
> ACK, ACK.
master:
842f83f8db513214241a0fea076ac160b180e1dd
d528e4960bf233bc002f9676919ede196c91564c
>
> Regards,
>
> Pavel Reichl
>
> _______________________________________________
> sssd-devel mailing list
> sssd-devel at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
More information about the sssd-devel
mailing list