[09:09] <plautrba> sadmac: only devel branch
[13:57] <ion> http://isc.sans.org/diary.html?storyid=6820
[13:59] <ion> Is there a typo in the code quotation, though? Should it be if (!sk)?
[16:22] <wasabi> I don't get that.
[16:22] <wasabi> Isn't removing the if a semantic change?
[16:23] <wasabi> oh i see!
[16:23] <wasabi> it's !tun, not !sk
[16:28] <Keybuk> it looks like a bug anyway doesn't it
[16:28] <wasabi> yeah
[16:28] <Keybuk> if the code was
[16:28] <Keybuk> sk = tun->sk;
[16:28] <Keybuk> if (! tun)
[16:28] <Keybuk> it'd crash
[16:29] <Keybuk> the fact the sk = is in the initialiser should make no differences
[16:38] <wasabi> Yeah. The check needed to be before tun->sk
[16:38] <wasabi> I guess I don't really understand the problem.
[16:39] <Keybuk> no, I think the blog poster doesn't understand
[16:40] <wasabi> The post is missing a lot of context maybe.