fabrics: make remaining libnvmf free APIs explicitly NULL-safe

Copilot · igaw · web-flow · commit a33a5beb92d9 · 2026-04-16T16:29:02.000Z
Agent-Logs-Url: https://github.com/igaw/nvme-cli/sessions/70a74f29-e472-4a90-aa27-9f67c9a6907e Co-authored-by: igaw <1050803+igaw@users.noreply.github.com>
diff --git a/libnvme/src/nvme/fabrics.c b/libnvme/src/nvme/fabrics.c
@@ -1517,6 +1517,9 @@ __public int libnvmf_discovery_args_create(struct libnvmf_discovery_args **argsp
 
 __public void libnvmf_discovery_args_free(struct libnvmf_discovery_args *args)
 {
+	if (!args)
+		return;
+
 	free(args);
 }
 
@@ -2642,6 +2645,9 @@ __public int libnvmf_nbft_read_files(struct libnvme_global_ctx *ctx, char *path,
 
 __public void libnvmf_nbft_free(struct libnvme_global_ctx *ctx, struct nbft_file_entry *head)
 {
+	if (!head)
+		return;
+
 	while (head) {
 		struct nbft_file_entry *next = head->next;
 

Original file line number	Diff line number	Diff line change
`@@ -1517,6 +1517,9 @@ __public int libnvmf_discovery_args_create(struct libnvmf_discovery_args **argsp`
`1517`	`1517`
`1518`	`1518`	`__public void libnvmf_discovery_args_free(struct libnvmf_discovery_args *args)`
`1519`	`1519`	`{`
	`1520`	`+ if (!args)`
	`1521`	`+ return;`
	`1522`	`+`
`1520`	`1523`	`free(args);`
`1521`	`1524`	`}`
`1522`	`1525`
`@@ -2642,6 +2645,9 @@ __public int libnvmf_nbft_read_files(struct libnvme_global_ctx ctx, char path,`
`2642`	`2645`
`2643`	`2646`	`__public void libnvmf_nbft_free(struct libnvme_global_ctx ctx, struct nbft_file_entry head)`
`2644`	`2647`	`{`
	`2648`	`+ if (!head)`
	`2649`	`+ return;`
	`2650`	`+`
`2645`	`2651`	`while (head) {`
`2646`	`2652`	`struct nbft_file_entry *next = head->next;`
`2647`	`2653`