[GH-ISSUE #34] App get fatal error if container inspect fails #15

New issue

Closed

opened 2026-02-26 04:33:46 +03:00 by kerem · 1 comment

kerem commented

2026-02-26 04:33:46 +03:00

Owner

Originally created by @mageddo on GitHub (Sep 10, 2017).
Original GitHub issue: https://github.com/mageddo/dns-proxy-server/issues/34

At this line in this snippet of code

cInspection, err := cli.ContainerInspect(ctx, event.ID)
  if err != nil {
    logger.Errorf("status=inspect-error, container=%s, err=%v", cInspection.Name, err)

If the inspection get a error the application will get nil pointer when try to log the container name

Taks

💡 Fix the nil pointer
💡 add continue statement to start the next iteration

🆗

Originally created by @mageddo on GitHub (Sep 10, 2017). Original GitHub issue: https://github.com/mageddo/dns-proxy-server/issues/34 [At this line](https://github.com/mageddo/dns-proxy-server/blob/3e49e1cc06845f27d3f2443c3d053083cf1febff/src/github.com/mageddo/dns-proxy-server/events/docker/DockerEvents.go#L83) in this snippet of code ```go cInspection, err := cli.ContainerInspect(ctx, event.ID) if err != nil { logger.Errorf("status=inspect-error, container=%s, err=%v", cInspection.Name, err) ``` If the inspection get a error the application will get nil pointer when try to log the container name ### Taks :bulb: Fix the nil pointer :bulb: add continue statement to start the next iteration :ok:

kerem

2026-02-26 04:33:46 +03:00

closed this issue
added the
bug
label

kerem commented

2026-02-26 04:33:46 +03:00

Author

Owner

@mageddo commented on GitHub (Sep 10, 2017):

Evidence

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x5147eb]

goroutine 7 [running]:
panic(0x73cb60, 0xc42000c060)
	/usr/local/go/src/runtime/panic.go:500 +0x1a1
github.com/mageddo/dns-proxy-server/events/docker.HandleDockerEvents()
	/app/src/github.com/mageddo/dns-proxy-server/events/docker/DockerEvents.go:83 +0x10bb
created by main.main
	/app/src/dns.go:121 +0x144

@mageddo commented on GitHub (Sep 10, 2017): Evidence ``` panic: runtime error: invalid memory address or nil pointer dereference [signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x5147eb] goroutine 7 [running]: panic(0x73cb60, 0xc42000c060) /usr/local/go/src/runtime/panic.go:500 +0x1a1 github.com/mageddo/dns-proxy-server/events/docker.HandleDockerEvents() /app/src/github.com/mageddo/dns-proxy-server/events/docker/DockerEvents.go:83 +0x10bb created by main.main /app/src/dns.go:121 +0x144 ```

kerem referenced this issue

2026-02-26 04:34:31 +03:00

[PR #15] [MERGED] Feature/mg 212 #219

kerem referenced this issue

2026-02-26 04:35:00 +03:00

[PR #258] [MERGED] Migrating from Travis to Actions (by Addshore) #365

kerem referenced this issue

2026-02-26 04:35:00 +03:00

[PR #257] [MERGED] Convert CI to Github actions #366