8000 Crash: SIGSEGV: segmentation violation (config-related) · Issue #65 · oz/tz · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Crash: SIGSEGV: segmentation violation (config-related) #65

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.

Sign up for GitHub

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

Jump to bottom
Closed
jnd-au opened this issue Oct 16, 2024 · 0 comments
Closed

Crash: SIGSEGV: segmentation violation (config-related) #65

jnd-au opened this issue Oct 16, 2024 · 0 comments

Comments

@jnd-au
Copy link
< 8000 div class="d-none d-sm-flex"> Contributor
jnd-au commented Oct 16, 2024

Config now relies on a Logger (since 88dec44), but the Logger is not initialised for Unit Testing (SetupLogger), causing the following failure when writing unit tests involving configuration:

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

goroutine 14 [running]:
testing.tRunner.func1.2({0x1046e2ee0, 0x104892040})
        ./go/1.23.2/libexec/src/testing/testing.go:1632 +0x1bc
testing.tRunner.func1()
        ./go/1.23.2/libexec/src/testing/testing.go:1635 +0x334
panic({0x1046e2ee0?, 0x104892040?})
        ./go/1.23.2/libexec/src/runtime/panic.go:785 +0x124
log.(*Logger).output(0x14000019200?, 0x21?, 0x14000104488?, 0x1045190f4?)
        ./go/1.23.2/libexec/src/log/log.go:203 +0x30
log.(*Logger).Printf(...)
        ./go/1.23.2/libexec/src/log/log.go:268
github.com/oz/tz.LoadConfigFile()
        ./tz/config_file.go:999 +0x280

Furthermore, the configuration behaviour was added to tz without unit tests.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Config bugfixes jnd-au/tz
1 participant
@jnd-au
0