Skip to content

Commit 8b5525c

Browse files
committed
mi: call nvme_mi_close on endpoint error path
We need to do more than free() on endpoint close, so call nvme_mi_close on endpoint creation failure. Signed-off-by: Jeremy Kerr <[email protected]>
1 parent 826d023 commit 8b5525c

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

src/nvme/mi-mctp.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -516,7 +516,7 @@ nvme_mi_ep_t nvme_mi_open_mctp(nvme_root_t root, unsigned int netid, __u8 eid)
516516

517517
err_free_ep:
518518
errno_save = errno;
519-
free(ep);
519+
nvme_mi_close(ep);
520520
free(mctp);
521521
errno = errno_save;
522522
return NULL;

src/nvme/mi.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1517,7 +1517,7 @@ void nvme_mi_close(nvme_mi_ep_t ep)
15171517
nvme_mi_for_each_ctrl_safe(ep, ctrl, tmp)
15181518
nvme_mi_close_ctrl(ctrl);
15191519

1520-
if (ep->transport->close)
1520+
if (ep->transport && ep->transport->close)
15211521
ep->transport->close(ep);
15221522
list_del(&ep->root_entry);
15231523
free(ep);

0 commit comments

Comments
 (0)