From adf87f3a36328b949ed777068b14d975b429f9ad Mon Sep 17 00:00:00 2001 From: Jo-Philipp Wich Date: Tue, 13 Jan 2015 12:46:37 +0100 Subject: [PATCH] redirects: fix possible null pointer access Signed-off-by: Jo-Philipp Wich --- redirects.c | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/redirects.c b/redirects.c index a30c540..50ed44d 100644 --- a/redirects.c +++ b/redirects.c @@ -312,20 +312,20 @@ fw3_load_redirects(struct fw3_state *state, struct uci_package *p) set(redir->_src->flags, FW3_FAMILY_V4, redir->target); redir->_src->conntrack = true; valid = true; - } - if (!check_local(e, redir, state) && !redir->dest.set && - resolve_dest(e, redir, state)) - { - warn_elem(e, "does not specify a destination, assuming '%s'", - redir->dest.name); - } + if (!check_local(e, redir, state) && !redir->dest.set && + resolve_dest(e, redir, state)) + { + warn_elem(e, "does not specify a destination, assuming '%s'", + redir->dest.name); + } - if (redir->reflection && redir->_dest && redir->_src->masq) - { - set(redir->_dest->flags, FW3_FAMILY_V4, FW3_FLAG_ACCEPT); - set(redir->_dest->flags, FW3_FAMILY_V4, FW3_FLAG_DNAT); - set(redir->_dest->flags, FW3_FAMILY_V4, FW3_FLAG_SNAT); + if (redir->reflection && redir->_dest && redir->_src->masq) + { + set(redir->_dest->flags, FW3_FAMILY_V4, FW3_FLAG_ACCEPT); + set(redir->_dest->flags, FW3_FAMILY_V4, FW3_FLAG_DNAT); + set(redir->_dest->flags, FW3_FAMILY_V4, FW3_FLAG_SNAT); + } } } else -- 2.30.2