| From: Paul Wouters <[email protected]>

| Additionally, since the SPI in IKEv1 and IKEv2 is always 8 octets, all
| callers to this function pass a length of 8, so we might as well phase
| out the argument.

Yes.  Done.

| The while() seems to cover an extremely extremely rare case, which is
| just as rare as generating the same SPI twice, a condition that is never
| checked for.

Right, but a 0 is a disaster and a duplicate is only unfortunate.

| If there is still a reason to keep it, I would like to rename it to
| get_rnd_ikespi() or something, as "cookie" is a misleading term.

I added "v1" to the name.
_______________________________________________
Swan-dev mailing list
[email protected]
https://lists.libreswan.org/mailman/listinfo/swan-dev

Reply via email to