diff options
author | glebius <glebius@FreeBSD.org> | 2005-08-11 08:14:53 +0000 |
---|---|---|
committer | glebius <glebius@FreeBSD.org> | 2005-08-11 08:14:53 +0000 |
commit | fa253399af72f4968a7eddd5210066700b92d2c0 (patch) | |
tree | decf2e9a9a44b246a6d6dd15b016ed79b3b91a61 /sys/net/if_atmsubr.c | |
parent | e0688f129366b4f89acf47c45af55135dc0609ae (diff) | |
download | FreeBSD-src-fa253399af72f4968a7eddd5210066700b92d2c0.zip FreeBSD-src-fa253399af72f4968a7eddd5210066700b92d2c0.tar.gz |
o Make rt_check() function more strict:
- rt0 passed to rt_check() must not be NULL, assert this.
- rt returned by rt_check() must be valid locked rtentry,
if no error occured.
o Modify callers, so that they never pass NULL rt0
to rt_check().
Reviewed by: sam, ume (nd6.c)
Diffstat (limited to 'sys/net/if_atmsubr.c')
-rw-r--r-- | sys/net/if_atmsubr.c | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/sys/net/if_atmsubr.c b/sys/net/if_atmsubr.c index 6e662a0..a0e84ea 100644 --- a/sys/net/if_atmsubr.c +++ b/sys/net/if_atmsubr.c @@ -152,14 +152,16 @@ atm_output(struct ifnet *ifp, struct mbuf *m0, struct sockaddr *dst, case AF_INET: case AF_INET6: { - struct rtentry *rt; + struct rtentry *rt = NULL; /* * check route */ - error = rt_check(&rt, &rt0, dst); - if (error) - goto bad; - RT_UNLOCK(rt); + if (rt0 != NULL) { + error = rt_check(&rt, &rt0, dst); + if (error) + goto bad; + RT_UNLOCK(rt); + } if (dst->sa_family == AF_INET6) etype = ETHERTYPE_IPV6; |