Certain consumers of rtalloc like gif(4) and if_stf(4) lookup the
route and once they are done with it, call rtfree(). rtfree() should only be used when we are certain we hold the last reference to the route. This bug results in console messages like the following: rtfree: 0xc40f7000 has 1 refs This patch switches the rtfree() to use RTFREE_LOCKED() instead, which should handle the reference counting on the route better. Approved by: re@ (gnn) Reviewed by: bms Reported by: many via net@ and current@ Tested by: many
This commit is contained in:
parent
9f9cc8d8af
commit
5328bd11db
@ -607,10 +607,10 @@ stf_checkaddr4(sc, in, inifp)
|
||||
(u_int32_t)ntohl(sin.sin_addr.s_addr));
|
||||
#endif
|
||||
if (rt)
|
||||
rtfree(rt);
|
||||
RTFREE_LOCKED(rt);
|
||||
return -1;
|
||||
}
|
||||
rtfree(rt);
|
||||
RTFREE_LOCKED(rt);
|
||||
}
|
||||
|
||||
return 0;
|
||||
|
@ -374,10 +374,10 @@ gif_validate4(const struct ip *ip, struct gif_softc *sc, struct ifnet *ifp)
|
||||
(u_int32_t)ntohl(sin.sin_addr.s_addr));
|
||||
#endif
|
||||
if (rt)
|
||||
rtfree(rt);
|
||||
RTFREE_LOCKED(rt);
|
||||
return 0;
|
||||
}
|
||||
rtfree(rt);
|
||||
RTFREE_LOCKED(rt);
|
||||
}
|
||||
|
||||
return 32 * 2;
|
||||
|
Loading…
Reference in New Issue
Block a user