[Swan-dev] cavp question

Andrew Cagney andrew.cagney at gmail.com
Mon Aug 21 16:59:42 UTC 2017


On 20 August 2017 at 00:02, D. Hugh Redelmeier <hugh at mimosa.com> wrote:
> This code in testing/cavp/cavp_ikev1.c looks fishy:
>
>        if (prf == NULL) {
>                 print_line(prf->key);
>                 return;
>         }
>
> My best guess is that the test is suposed to be:
>        if (prf->prf == NULL) {

Yes.  Here, prf is never NULL, but prf->prf could be.

If someone feeds SHA-224, which we don't support into CAVP, then it
will dump core instead of stumbling on.  The test facility shouldn't
be doing that anyway :-)

I'll change to code and get rid of the indirection.

Andrew


> But I am quite unsure.
>
> Can someone look at this?
> _______________________________________________
> Swan-dev mailing list
> Swan-dev at lists.libreswan.org
> https://lists.libreswan.org/mailman/listinfo/swan-dev


More information about the Swan-dev mailing list