Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bfdd: 0 is a valid fd. #18125

Merged
merged 1 commit into from
Feb 12, 2025
Merged

bfdd: 0 is a valid fd. #18125

merged 1 commit into from
Feb 12, 2025

Conversation

donaldsharp
Copy link
Member

It's unlikely that this would ever get a 0 fd to use but let's just let the code assume it is ok.

It's unlikely that this would ever get a 0 fd to use
but let's just let the code assume it is ok.

Signed-off-by: Donald Sharp <[email protected]>
Copy link
Contributor

@mjstapp mjstapp left a comment

Choose a reason for hiding this comment

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

makes sense

@mjstapp mjstapp merged commit 6d14b16 into FRRouting:master Feb 12, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants