diff options
| author | Remi Gacogne <rgacogne-github@coredump.fr> | 2013-09-08 13:48:34 +0000 |
|---|---|---|
| committer | Vincent JARDIN <vincent.jardin@6wind.com> | 2014-02-11 10:28:03 +0100 |
| commit | a11e012e8661629d665e992e765741a5eaa7d017 (patch) | |
| tree | d6967d9c5044f8c924aa2f4b62bac8e3bda358e2 /ospfd | |
| parent | b52aef18a9f3acc8b24ab5c2631dc574b8e2ec70 (diff) | |
| download | quagga-a11e012e8661629d665e992e765741a5eaa7d017.tar.bz2 quagga-a11e012e8661629d665e992e765741a5eaa7d017.tar.xz | |
security: Fix some typos and potential NULL-deref
This patch against the git tree fixes minor typos, some of them possibily
leading to NULL-pointer dereference in rare conditions.
Signed-off-by: Remi Gacogne <rgacogne-github@coredump.fr>
Signed-off-by: Joachim Nilsson <troglobit@gmail.com>
Acked-by: Feng Lu <lu.feng@6wind.com>
Diffstat (limited to 'ospfd')
| -rw-r--r-- | ospfd/ospf_asbr.c | 3 | ||||
| -rw-r--r-- | ospfd/ospf_te.c | 3 |
2 files changed, 4 insertions, 2 deletions
diff --git a/ospfd/ospf_asbr.c b/ospfd/ospf_asbr.c index a23b4f2b..7e7c84fd 100644 --- a/ospfd/ospf_asbr.c +++ b/ospfd/ospf_asbr.c @@ -164,7 +164,8 @@ ospf_external_info_add (u_char type, struct prefix_ipv4 p, new->nexthop = nexthop; new->tag = 0; - rn->info = new; + if (rn) + rn->info = new; if (IS_DEBUG_OSPF (lsa, LSA_GENERATE)) zlog_debug ("Redistribute[%s]: %s/%d external info created.", diff --git a/ospfd/ospf_te.c b/ospfd/ospf_te.c index 587564a1..c605ce68 100644 --- a/ospfd/ospf_te.c +++ b/ospfd/ospf_te.c @@ -1036,7 +1036,8 @@ ospf_mpls_te_lsa_refresh (struct ospf_lsa *lsa) /* If the lsa's age reached to MaxAge, start flushing procedure. */ if (IS_LSA_MAXAGE (lsa)) { - lp->flags &= ~LPFLG_LSA_ENGAGED; + if (lp) + lp->flags &= ~LPFLG_LSA_ENGAGED; ospf_opaque_lsa_flush_schedule (lsa); goto out; } |