The branch stable/13 has been updated by melifaro:

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

commit 11304949deb36d2f9abbcda87d398496e892dcec
Author:     Alexander V. Chernikov <melif...@freebsd.org>
AuthorDate: 2021-02-12 20:56:52 +0000
Commit:     Alexander V. Chernikov <melif...@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);
        }
_______________________________________________
dev-commits-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/dev-commits-src-all
To unsubscribe, send any mail to "dev-commits-src-all-unsubscr...@freebsd.org"

Reply via email to