Skip to content

Comments

Fix nil pointer dereference on server shutdown#407

Open
KonishchevDmitry wants to merge 1 commit intomartin-helmich:masterfrom
KonishchevDmitry:npd-fix
Open

Fix nil pointer dereference on server shutdown#407
KonishchevDmitry wants to merge 1 commit intomartin-helmich:masterfrom
KonishchevDmitry:npd-fix

Conversation

@KonishchevDmitry
Copy link

2024-12-24T16:26:05.629+0300        info        prometheus-nginxlog-exporter/main.go:101        caught term terminated. exiting
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x881f0e]
goroutine 51 [running]:
github.com/martin-helmich/prometheus-nginxlog-exporter/pkg/syslog.Listen.func1()
        /home/runner/work/prometheus-nginxlog-exporter/prometheus-nginxlog-exporter/pkg/syslog/syslog.go:83 +0x6e
main.processNamespace.func2()
        /home/runner/work/prometheus-nginxlog-exporter/prometheus-nginxlog-exporter/main.go:233 +0x3f
created by main.processNamespace
        /home/runner/work/prometheus-nginxlog-exporter/prometheus-nginxlog-exporter/main.go:230 +0x23b

```
2024-12-24T16:26:05.629+0300        info        prometheus-nginxlog-exporter/main.go:101        caught term terminated. exiting
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x881f0e]
goroutine 51 [running]:
github.com/martin-helmich/prometheus-nginxlog-exporter/pkg/syslog.Listen.func1()
        /home/runner/work/prometheus-nginxlog-exporter/prometheus-nginxlog-exporter/pkg/syslog/syslog.go:83 +0x6e
main.processNamespace.func2()
        /home/runner/work/prometheus-nginxlog-exporter/prometheus-nginxlog-exporter/main.go:233 +0x3f
created by main.processNamespace
        /home/runner/work/prometheus-nginxlog-exporter/prometheus-nginxlog-exporter/main.go:230 +0x23b
```
@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit ba562ab and detected 0 issues on this pull request.

View more on Code Climate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant