fix: Fix crashing when SDK is disabled via options (#253)

limbonaut · web-flow · commit 8bff94d3a38c · 2025-07-17T19:01:43.000-04:00
* Fix null pointer dereference in SentrySDK::get_user

* Update CHANGELOG.md
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,7 @@
 ### Fixes
 
 - Check options.enabled during SDK initialization ([#250](https://github.com/getsentry/sentry-godot/pull/250))
+- Fix crashing when SDK is disabled via options ([#253](https://github.com/getsentry/sentry-godot/pull/253))
 
 ## 1.0.0-alpha.0
 
diff --git a/src/sentry_sdk.cpp b/src/sentry_sdk.cpp
@@ -139,7 +139,7 @@ void SentrySDK::set_user(const Ref<SentryUser> &p_user) {
 
 Ref<SentryUser> SentrySDK::get_user() const {
 	MutexLock lock(*user_mutex.ptr());
-	return user->duplicate();
+	return user.is_valid() ? user->duplicate() : nullptr;
 }
 
 void SentrySDK::remove_user() {

Original file line number	Diff line number	Diff line change
`@@ -139,7 +139,7 @@ void SentrySDK::set_user(const Ref<SentryUser> &p_user) {`
`139`	`139`
`140`	`140`	`Ref<SentryUser> SentrySDK::get_user() const {`
`141`	`141`	`MutexLock lock(*user_mutex.ptr());`
`142`		`- return user->duplicate();`
	`142`	`+ return user.is_valid() ? user->duplicate() : nullptr;`
`143`	`143`	`}`
`144`	`144`
`145`	`145`	`void SentrySDK::remove_user() {`