git: 11304949deb3 - stable/13 - Fix ifa refcount leak during route addition.

Alexander V. Chernikov melifaro at FreeBSD.org
Sun Feb 14 10:18:19 UTC 2021


The branch stable/13 has been updated by melifaro:

URL: https://cgit.FreeBSD.org/src/commit/?id=11304949deb36d2f9abbcda87d398496e892dcec

commit 11304949deb36d2f9abbcda87d398496e892dcec
Author:     Alexander V. Chernikov <melifaro at FreeBSD.org>
AuthorDate: 2021-02-12 20:56:52 +0000
Commit:     Alexander V. Chernikov <melifaro at FreeBSD.org>
CommitDate: 2021-02-14 10:16:51 +0000

    Fix ifa refcount leak during route addition.
    
    Reported by:    rstone
    Reviewed by:    rstone
    
    (cherry picked from commit a375ec52a7b423133f66878ecf002efc3b6e9fca)
---
 sys/net/route/route_ctl.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/sys/net/route/route_ctl.c b/sys/net/route/route_ctl.c
index 6b0869196d12..9aedfb9d5855 100644
--- a/sys/net/route/route_ctl.c
+++ b/sys/net/route/route_ctl.c
@@ -594,14 +594,12 @@ create_rtentry(struct rib_head *rnh, struct rt_addrinfo *info,
 	}
 
 	error = nhop_create_from_info(rnh, info, &nh);
-	if (error != 0) {
-		ifa_free(info->rti_ifa);
+	ifa_free(info->rti_ifa);
+	if (error != 0)
 		return (error);
-	}
 
 	rt = uma_zalloc(V_rtzone, M_NOWAIT | M_ZERO);
 	if (rt == NULL) {
-		ifa_free(info->rti_ifa);
 		nhop_free(nh);
 		return (ENOBUFS);
 	}


More information about the dev-commits-src-all mailing list