check null dsn

getsentry · Feb 5, 2025 · a2508cd · a2508cd
1 parent 46ad5b0
commit a2508cd
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/src/backends/sentry_backend_crashpad.cpp b/src/backends/sentry_backend_crashpad.cpp
@@ -439,8 +439,9 @@ crashpad_backend_startup(
     if (minidump_url) {
         SENTRY_DEBUGF("using minidump URL \"%s\"", minidump_url);
     }
-    const char *env_proxy
-        = getenv(options->dsn->is_secure ? "https_proxy" : "http_proxy");
+    const char *env_proxy = options->dsn
+        ? getenv(options->dsn->is_secure ? "https_proxy" : "http_proxy")
+        : nullptr;
     const char *proxy_url = options->proxy ? options->proxy
         : env_proxy                        ? env_proxy
                                            : "";

diff --git a/src/transports/sentry_transport_winhttp.c b/src/transports/sentry_transport_winhttp.c
@@ -93,8 +93,9 @@ sentry__winhttp_transport_start(
     state->user_agent = sentry__string_to_wstr(opts->user_agent);
     state->debug = opts->debug;
 
-    const char *env_proxy
-        = getenv(opts->dsn->is_secure ? "https_proxy" : "http_proxy");
+    const char *env_proxy = opts->dsn
+        ? getenv(opts->dsn->is_secure ? "https_proxy" : "http_proxy")
+        : NULL;
     const char *proxy = opts->proxy ? opts->proxy : env_proxy ? env_proxy : "";
 
     // ensure the proxy starts with `http://`, otherwise ignore it