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 _______________________________________________ Dev mailing list [email protected] https://lists.strongswan.org/mailman/listinfo/dev
