[PATCH] babel: Add support for dual-stack v4/v6 operation

[Toke Høiland-Jørgensen]

Ondrej Zajicek <santiago at crfreenet.org> writes:

> On Tue, May 30, 2017 at 01:37:56PM +0200, Toke Høiland-Jørgensen wrote:
>> > Also channel_is_active() is called but channel may not even exist?
>> 
>> Ah, proto_configure_channel can leave the channel var entirely unset?
>
> Yes, it could add/remove/reconfigure the channel, based on whether there
> is or is not the configuration and the old channel.
>
>> was assuming it was just leaving it in place but inactive. Can I just
>> check for NULL, then?
>
> Yes, that should be enough, as all channels goes up together with the
> protocol and channel cannot be deconfigured when up (protocol will be
> restarted in that case).

Right, so a new channel can appear on reconfigure, but existing ones
cannot go away?

-Toke