git: 6bc3535519f7 - stable/11 - netgraph/ng_checksum: Fix double free error

Lutz Donnerhacke donner at FreeBSD.org
Sun May 23 13:02:08 UTC 2021


The branch stable/11 has been updated by donner:

URL: https://cgit.FreeBSD.org/src/commit/?id=6bc3535519f7206f844c3ffd0ee282e8875dceb4

commit 6bc3535519f7206f844c3ffd0ee282e8875dceb4
Author:     Lutz Donnerhacke <donner at FreeBSD.org>
AuthorDate: 2021-05-15 09:32:57 +0000
Commit:     Lutz Donnerhacke <donner at FreeBSD.org>
CommitDate: 2021-05-23 13:01:34 +0000

    netgraph/ng_checksum: Fix double free error
    
    m_pullup(9) frees the mbuf(9) chain in the case of an allocation error.
    The mbuf chain must not be freed again in this case.
    
    PR:             255874
    Submitted by:   <lylgood at foxmail.com>
    Approved by:    markj
    Differential Revision: https://reviews.freebsd.org/D30273
    
    (cherry picked from commit 687e510e5ce32fddf46a9dc1d517ccc8a8e25581)
---
 sys/netgraph/ng_checksum.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sys/netgraph/ng_checksum.c b/sys/netgraph/ng_checksum.c
index c7fecb0ab78d..0100b7ef3260 100644
--- a/sys/netgraph/ng_checksum.c
+++ b/sys/netgraph/ng_checksum.c
@@ -683,9 +683,9 @@ bypass:
 	return (error);
 
 done:
+	NG_FREE_M(m);
 drop:
 	NG_FREE_ITEM(item);
-	NG_FREE_M(m);
 
 	priv->stats.dropped++;
 


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