On Fri, Nov 18, 2005 at 08:05:09AM -0500, Greg Troxel wrote:
>         if (pa->type == iph1->natt_options->payload_nat_d)
> 
> There seems to be an assumption that iph1->natt_options is always
> non-NULL.  I suggest adding "iph1->natt_options != NULL &&" to the
> beginning of the if, but I haven't even read the next line.

I had tried that, and while it keeps racoon from crashing, it doesn't
actually make it work, although I forget the details. I'm guessing
that since I'm using NAT-T, natt_options should contain some useful
and needed info, so the question is why isn't racoon filling it n?