Skip to content

CFE-4536: Fixed file descriptor leak in sys.cpusockets#5822

Merged
nickanderson merged 1 commit into
cfengine:masterfrom
nickanderson:CFE-4536/master
Jun 3, 2025
Merged

CFE-4536: Fixed file descriptor leak in sys.cpusockets#5822
nickanderson merged 1 commit into
cfengine:masterfrom
nickanderson:CFE-4536/master

Conversation

@nickanderson

Copy link
Copy Markdown
Member

Ticket: CFE-4536

Ticket: CFE-4536
Changelog: Title

@craigcomstock craigcomstock left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yep, looks like this was missed in code review for this commit: e4b86ab and this PR: #5722

FullRead() does NOT close the file handle so this close(f) is needed.

@nickanderson nickanderson merged commit 37e6231 into cfengine:master Jun 3, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants