From: Colin Ian King <colin.k...@canonical.com> afinfo is being null checked before a call afinfo->init_tempsel so afinfo may be potentially null. ifinfo may still be null in the case were it is not updated when family == tmpl->encap_family, hence we may hit a null pointer dereference in the call to afinfo->init_temprop.
Fix this by adding a null ptr check before calling init_temprop. Signed-off-by: Colin Ian King <colin.k...@canonical.com> --- net/xfrm/xfrm_state.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/net/xfrm/xfrm_state.c b/net/xfrm/xfrm_state.c index a62097e..5083418 100644 --- a/net/xfrm/xfrm_state.c +++ b/net/xfrm/xfrm_state.c @@ -656,7 +656,8 @@ xfrm_init_tempstate(struct xfrm_state *x, const struct flowi *fl, if (!afinfo) return; } - afinfo->init_temprop(x, tmpl, daddr, saddr); + if (afinfo) + afinfo->init_temprop(x, tmpl, daddr, saddr); } static struct xfrm_state *__xfrm_state_lookup(struct net *net, u32 mark, -- 2.10.2