Ben Greear wrote:
        /* Copy neighbout for reachability confirmation */
        dst_prev->neighbour    = neigh_clone(rt->u.dst.neighbour)

This code in the method xfrm4_bundle_create appears to over-write
the dst_prev->neighbour member without ever checking to see if it
needed to release the old neighbour pointer...

If there is some reason dst_prev->neighbour is always NULL, please
let me know.  Otherwise, I think it's a leak.

dst_prev is newly allocated in the loop a couple of lines above.
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to