[strongSwan-dev] route reinstallation test report and analysis

[Mirko Parthey]

On Mon, May 07, 2012 at 12:05:52PM +0200, Tobias Brunner wrote:
> > The change lookup code uses an uninitialized struct member
> 
> Actually, no.  For structs initialized like this all members not
> assigned are initialized to 0/NULL (this is a feature of C99 called
> designated initializers and GCC also supports it for C89).

You are right.

> Ok, how about the attached patch.  I changed the lookup so that the
> interface that has the preferred source address installed is also
> considered if it is different from the outgoing interface.  Also, no IP
> specific changes are queued now (I tried to avoid unnecessary updates
> but that was probably overkill).

The route management is working well for me now, thanks.

I still have an issue with how charon handles a certain sequence of
address updates, I'll send a separate email about that.

Regards,
Mirko