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

[bug] nil deref in CreateFollowReq #2098

Closed
ftrvxmtrx opened this issue Aug 11, 2023 · 1 comment · Fixed by #2099
Closed

[bug] nil deref in CreateFollowReq #2098

ftrvxmtrx opened this issue Aug 11, 2023 · 1 comment · Fixed by #2099
Labels
bug Something isn't working

Comments

@ftrvxmtrx
Copy link
Contributor

Describe the bug with a clear and concise description of what the bug is.

Aug 11 02:31:12 ftrv gotosocial[1493730]: timestamp="11/08/2023 02:31:12.026" func=workers.(*Processor).ProcessFromFediAPI level=INFO activityType=Create objectType=Follow toAccount=sigrid msg="processing from fedi API"
Aug 11 02:31:12 ftrv gotosocial[1493730]: worker_run: recovered panic: runtime error: invalid memory address or nil pointer dereference
Aug 11 02:31:12 ftrv gotosocial[1493730]: runtime.gopanic()
Aug 11 02:31:12 ftrv gotosocial[1493730]:         /nix/store/pljrwnv985xcsgbvbf3vm53dvzzszpc6-go-1.20.6/share/go/src/runtime/panic.go:884
Aug 11 02:31:12 ftrv gotosocial[1493730]: runtime.panicmem()
Aug 11 02:31:12 ftrv gotosocial[1493730]:         /nix/store/pljrwnv985xcsgbvbf3vm53dvzzszpc6-go-1.20.6/share/go/src/runtime/panic.go:260
Aug 11 02:31:12 ftrv gotosocial[1493730]: runtime.sigpanic()
Aug 11 02:31:12 ftrv gotosocial[1493730]:         /nix/store/pljrwnv985xcsgbvbf3vm53dvzzszpc6-go-1.20.6/share/go/src/runtime/signal_unix.go:841
Aug 11 02:31:12 ftrv gotosocial[1493730]: workers.(*fediAPI).CreateFollowReq()
Aug 11 02:31:12 ftrv gotosocial[1493730]:         /home/ftrvx/w/_/gotosocial/internal/processing/workers/fromfediapi.go:268
Aug 11 02:31:12 ftrv gotosocial[1493730]: workers.(*Processor).ProcessFromFediAPI()
Aug 11 02:31:12 ftrv gotosocial[1493730]:         /home/ftrvx/w/_/gotosocial/internal/processing/workers/fromfediapi.go:96
Aug 11 02:31:12 ftrv gotosocial[1493730]: workers.(*Processor).EnqueueFediAPI.func1()
Aug 11 02:31:12 ftrv gotosocial[1493730]:         /home/ftrvx/w/_/gotosocial/internal/processing/workers/fromfediapi.go:52
Aug 11 02:31:12 ftrv gotosocial[1493730]: go-runners.worker_run()
Aug 11 02:31:12 ftrv gotosocial[1493730]:         /home/ftrvx/w/_/gotosocial/vendor/codeberg.org/gruf/go-runners/pool.go:237
Aug 11 02:31:12 ftrv gotosocial[1493730]: go-runners.(*WorkerPool).Start.func1.2()
Aug 11 02:31:12 ftrv gotosocial[1493730]:         /home/ftrvx/w/_/gotosocial/vendor/codeberg.org/gruf/go-runners/pool.go:71

What's your GoToSocial Version?

5588d4e

GoToSocial Arch

amd64

What happened?

A nil pointer dereference.

What you expected to happen?

No response

How to reproduce it?

No response

Anything else we need to know?

No response

@ftrvxmtrx ftrvxmtrx added the bug Something isn't working label Aug 11, 2023
@NyaaaWhatsUpDoc NyaaaWhatsUpDoc added this to the v0.11.0 Sloppy Sloth milestone Aug 11, 2023
@tsmethurst
Copy link
Contributor

Ah thank you, missed that one in the refactor :)

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

Successfully merging a pull request may close this issue.

3 participants