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

nsca can exit due to ECONNABORTED on FreeBSD #20

Closed
leres opened this issue Jan 28, 2017 · 7 comments
Closed

nsca can exit due to ECONNABORTED on FreeBSD #20

leres opened this issue Jan 28, 2017 · 7 comments
Assignees
Labels

Comments

@leres
Copy link

leres commented Jan 28, 2017

We've had a lot of problems on our FreeBSD systems with the nsca daemon exiting due to ECONNABORTED. This is probably caused by cyber security scanning tickling a race condition in nsca itself.

The FreeBSD accept(2) man page says:

 [ECONNABORTED] A connection arrived, but it was closed while waiting on the listen queue.

Here is a minimal patch. A better fix would be to never treat accept(2) errors as fatal.

@jfrickson jfrickson self-assigned this Mar 9, 2017
@jfrickson jfrickson added the Bug label Mar 9, 2017
@jfrickson
Copy link
Contributor

Looking into it.

@ghost
Copy link

ghost commented Apr 20, 2017

I may be hitting this at work as well. Patched locally, will see if this solves the unexpected NSCA daemon shutdowns.

@hedenface
Copy link
Contributor

@feld What did you find? Did this solve your issue?

@hedenface hedenface assigned hedenface and unassigned jfrickson Jun 16, 2017
@ghost
Copy link

ghost commented Aug 16, 2017

I can confirm this fixed the issue and is no longer crashing. We are shipping this patch with NSCA on FreeBSD

@xavierba
Copy link
Contributor

@hedenface, the patch above looks like it has been tested, fixes the issue, and FreeBSD ships it for more than 2 years. This is likely valid for other platforms as well.
Can you please consider committing it ?

xavierba added a commit to xavierba/nsca that referenced this issue Oct 31, 2019
Originally submitted by Craig Leres in
NagiosEnterprises#20
@xavierba
Copy link
Contributor

I filed #39 with the patch rebased against maint branch.

@sawolf
Copy link
Member

sawolf commented Mar 25, 2020

PR was merged - closing

@sawolf sawolf closed this as completed Mar 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants