I would prefer this way of checking the NULL That would be more consistent with ip_gre.c and ipip.c any suggestions ? /Raj