Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 30 May 2025 23:30:04 GMT
From:      John Baldwin <jhb@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: 0d48d1ffe044 - main - nvmft: Fix a resource leak for SET_FEATURES/ASYNC_EVENT_CONFIGURATION
Message-ID:  <202505302330.54UNU4gm026143@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by jhb:

URL: https://cgit.FreeBSD.org/src/commit/?id=0d48d1ffe0446cd2f87ce02555e3d17772ae7284

commit 0d48d1ffe0446cd2f87ce02555e3d17772ae7284
Author:     John Baldwin <jhb@FreeBSD.org>
AuthorDate: 2025-05-30 23:18:19 +0000
Commit:     John Baldwin <jhb@FreeBSD.org>
CommitDate: 2025-05-30 23:29:57 +0000

    nvmft: Fix a resource leak for SET_FEATURES/ASYNC_EVENT_CONFIGURATION
    
    The received command capsule was not freed after sending the success
    response.
    
    Fixes:          a15f7c96a276 ("nvmft: The in-kernel NVMe over Fabrics controller")
    Sponsored by:   Chelsio Communications
---
 sys/dev/nvmf/controller/nvmft_controller.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/sys/dev/nvmf/controller/nvmft_controller.c b/sys/dev/nvmf/controller/nvmft_controller.c
index 96c9fee47357..390467534ca2 100644
--- a/sys/dev/nvmf/controller/nvmft_controller.c
+++ b/sys/dev/nvmf/controller/nvmft_controller.c
@@ -787,6 +787,7 @@ handle_set_features(struct nvmft_controller *ctrlr,
 		ctrlr->aer_mask = aer_mask;
 		mtx_unlock(&ctrlr->lock);
 		nvmft_send_success(ctrlr->admin, nc);
+		nvmf_free_capsule(nc);
 		return;
 	}
 	default:



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202505302330.54UNU4gm026143>