nvmf_che: Don't leak a socket if an error occurs finalizing the socket
ClosedPublic
Actions

Authored by jhb on Feb 24 2026, 8:20 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Aug 26, 3:10 PM

	Unknown Object (File)
	Tue, Aug 25, 12:24 AM

	Unknown Object (File)
	Sat, Aug 22, 10:27 AM

	Unknown Object (File)
	Thu, Aug 20, 10:34 AM

	Unknown Object (File)
	Thu, Aug 13, 9:25 AM

	Unknown Object (File)
	Wed, Aug 12, 1:20 AM

	Unknown Object (File)
	Mon, Aug 10, 6:27 AM

	Unknown Object (File)
	Sat, Aug 8, 5:51 AM

View All 47 Files

Subscribers

imp

Details

Reviewers

Commits

rG505f381cb96e: nvmf_che: Don't leak a socket if an error occurs finalizing the socket

Summary

If soreserve() or sosetopt() (to set TCP_NODELAY) fails after claiming
the socket from the file descriptor, explicitly close the socket before
returning failure.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jhb created this revision.Feb 24 2026, 8:20 PM

Herald added subscribers: ae, imp. · View Herald TranscriptFeb 24 2026, 8:20 PM

jhb requested review of this revision.Feb 24 2026, 8:20 PM

Harbormaster completed remote builds in B70971: Diff 172623.Feb 24 2026, 8:20 PM

This revision was not accepted when it landed; it landed in state Needs Review.Apr 20 2026, 5:20 PM

Closed by commit rG505f381cb96e: nvmf_che: Don't leak a socket if an error occurs finalizing the socket (authored by jhb). · Explain Why

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rG505f381cb96e: nvmf_che: Don't leak a socket if an error occurs finalizing the socket.

Revision Contents
Changeset List

Path

Size

sys/

dev/

cxgbe/

nvmf/

nvmf_che.c

1 line

Diff 175923

View Options

sys/dev/cxgbe/nvmf/nvmf_che.c

nvmf_che: Don't leak a socket if an error occurs finalizing the socketClosedPublicActions