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

sentry_set_user() crashes when called after sentry_init() #96

Closed
matok opened this issue Nov 21, 2019 · 2 comments · Fixed by #98
Closed

sentry_set_user() crashes when called after sentry_init() #96

matok opened this issue Nov 21, 2019 · 2 comments · Fixed by #98

Comments

@matok
Copy link

matok commented Nov 21, 2019

Important details

  • Saas (sentry.io)
  • mac (10.15)
  • built from release 0.1.2
  • sentry with crashpad backend

Description

Invoking sentry_set_user() before sentry_init() works fine, all following events are tagged with provided user details, but when it's called after sentry_init, it crashes on BAD_ACCESS because of NULL FILE* pointer writer->context in mpack_file_writer_flush.

I don't know whether this is an expected behaviour, but it would be nice if we could change user details after initialization.

sentry_crash_vars
sentry_crash_stack
sentry_crash_code

@matok
Copy link
Author

matok commented Nov 21, 2019

I just compiled the same code on windows, also with crashpad backend and it doesn't crash.

@mitsuhiko
Copy link
Member

This might be the same core issues as #84. It fails to open the event file. I'm adding a fix for the crash in but it won't make the user data show up yet.

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 a pull request may close this issue.

2 participants