On 5/3/2013 14:13, Gert Doering wrote: > Add ASSERT() check to route_list_add_vpn_gateway() to ensure that *rl > is valid (and if not, crash with a somewhat more meaningful message than > "segmentation violation"). For the actual bugfix, change init code to > always allocate a "struct route_list" for IPv4, even if no --route options > have been seen in the config. > > Fix trac#281 and trac#258. > > Signed-off-by: Gert Doering <g...@greenie.muc.de>
ACK to the fix. As briefly discussed online, in the long-term the route_list_add_vpn_gateway() call in forward.c should probably be moved to a better location; it sounds like that's already in the works for a future changeset (and not required to fix this crash.) -- Josh Cepek
