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

Crash after running a few days #108

Closed
lapo-luchini opened this issue Feb 28, 2022 · 12 comments
Closed

Crash after running a few days #108

lapo-luchini opened this issue Feb 28, 2022 · 12 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@lapo-luchini
Copy link

ERRO[357497]
ERRO[358096]
ERRO[358096]
ERRO[358096]
ERRO[358098]
ERRO[358696]
ERRO[359597]
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x46f1d3]

goroutine 213683 [running]:
github.com/Luzilla/dnsbl_exporter/collector.(*Rbl).lookup(0xc0002c0030, {0xc000230420, 0xc}, {0xc000230550, 0xc})
        /home/lapo/dnsbl_exporter/collector/rbl.go:163 +0x4b9
github.com/Luzilla/dnsbl_exporter/collector.(*Rbl).Update.func1(0xc0000b2000, 0xc0002c0030, {0xc000230420, 0xc}, {0xc000230550, 0xc})
        /home/lapo/dnsbl_exporter/collector/rbl.go:181 +0x16d
created by github.com/Luzilla/dnsbl_exporter/collector.(*Rbl).Update
        /home/lapo/dnsbl_exporter/collector/rbl.go:176 +0x105
@till
Copy link
Contributor

till commented Feb 28, 2022

Can you share your config?

@lapo-luchini
Copy link
Author

I used the full file from matteocorti as rbls.ini and I have two servers in my targets.ini.

@till
Copy link
Contributor

till commented May 14, 2022

@lapo-luchini looking at this again too. I am assuming the crash still happens. I'll see if I can re-do the code and make it more easier to handle failures.

till added a commit that referenced this issue May 14, 2022
When no IPs are returned, we should stop this dance. This could be
a temporary problem with dns resolution.

For: #108
@till
Copy link
Contributor

till commented May 14, 2022

@lapo-luchini again, sorry for the delay. I made a branch and hopefully included a fix, not sure if you want/can test: #119

I will merge it later and release a new version. If you happen to run into other issues, please run the process with --log.debug. It should help me catch what the problem is. My guess here is that some kind of (temporary?) failure in DNS resolution or record lookup made it crash.

@till
Copy link
Contributor

till commented May 15, 2022

@lapo-luchini
Copy link
Author

I'm testing 9088e8b, thanks!

Will report if I find bugs (or it crashes again).

@till till self-assigned this May 16, 2022
@till till added the bug Something isn't working label May 16, 2022
@lapo-luchini
Copy link
Author

Happened again (after quite a few straight days of running):

ERRO[430346]
ERRO[431246]
ERRO[431846]
ERRO[432147]
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x465473]

goroutine 257028 [running]:
github.com/Luzilla/dnsbl_exporter/collector.(*Rbl).lookup(0xc0001f76e0, {0xc00002a5d0, 0x12}, {0xc0000293d0, 0xc})
        /home/andxor/dnsbl_exporter/collector/rbl.go:163 +0x36f
github.com/Luzilla/dnsbl_exporter/collector.(*Rbl).Update.func1({0xc00002a5d0, 0x12}, {0xc0000293d0, 0xc})
        /home/andxor/dnsbl_exporter/collector/rbl.go:181 +0x12f
created by github.com/Luzilla/dnsbl_exporter/collector.(*Rbl).Update
        /home/andxor/dnsbl_exporter/collector/rbl.go:176 +0x18a
% git describe
0.5.0-53-g9088e8b

@lapo-luchini
Copy link
Author

lapo-luchini commented May 21, 2022

Whops, sorry, you said to try the branch and I was trying head… switching now to this:

% git checkout origin/fix-nil-pointer
% git describe
v0.6.0-alpha3
% dist/dnsbl_exporter_freebsd_amd64/dnsbl-exporter --log.debug

@till
Copy link
Contributor

till commented May 21, 2022

@lapo-luchini Appreciate the persistence! 💪

@till
Copy link
Contributor

till commented Jun 23, 2022

@lapo-luchini Hope no news is good news? 😅

@till till closed this as completed Jul 26, 2022
@lapo-luchini
Copy link
Author

Yeah sorry, it seems it never happened again.

@till
Copy link
Contributor

till commented Aug 21, 2022

Yeah sorry, it seems it never happened again.

Thank you!

@till till added this to the 1.0.0 milestone Sep 5, 2022
till added a commit that referenced this issue Sep 6, 2022
When no IPs are returned, we should stop this dance. This could be
a temporary problem with dns resolution.

For: #108
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants