Fix crash daemon envelope ownership and event_json cleanup

cursoragent · cursoragent · commit e794f41a23bf · 2026-03-23T12:34:50.000Z
diff --git a/src/backends/native/sentry_crash_daemon.c b/src/backends/native/sentry_crash_daemon.c
@@ -2280,6 +2280,7 @@ write_envelope_with_minidump(const sentry_options_t *options,
     if (fd < 0) {
         SENTRY_WARN("Failed to open envelope file for writing");
         sentry_free(event_id);
+        sentry_free(event_json);
         return false;
     }
 
@@ -2333,8 +2334,8 @@ write_envelope_with_minidump(const sentry_options_t *options,
             _write(fd, "\n", 1);
 #endif
         }
-        sentry_free(event_json);
     }
+    sentry_free(event_json);
 
     // Add minidump as attachment
 #if defined(SENTRY_PLATFORM_UNIX)
@@ -2750,8 +2751,6 @@ sentry__process_crash(const sentry_options_t *options, sentry_crash_ipc_t *ipc)
             SENTRY_WARN("No transport available for sending envelope");
             sentry_envelope_free(envelope);
         }
-    } else {
-        sentry_envelope_free(envelope);
     }
 
     // Clean up temporary envelope file (keep minidump for

Original file line number	Diff line number	Diff line change
`@@ -2280,6 +2280,7 @@ write_envelope_with_minidump(const sentry_options_t *options,`
`2280`	`2280`	`if (fd < 0) {`
`2281`	`2281`	`SENTRY_WARN("Failed to open envelope file for writing");`
`2282`	`2282`	`sentry_free(event_id);`
	`2283`	`+ sentry_free(event_json);`
`2283`	`2284`	`return false;`
`2284`	`2285`	`}`
`2285`	`2286`
`@@ -2333,8 +2334,8 @@ write_envelope_with_minidump(const sentry_options_t *options,`
`2333`	`2334`	`_write(fd, "\n", 1);`
`2334`	`2335`	`#endif`
`2335`	`2336`	`}`
`2336`		`- sentry_free(event_json);`
`2337`	`2337`	`}`
	`2338`	`+ sentry_free(event_json);`
`2338`	`2339`
`2339`	`2340`	`// Add minidump as attachment`
`2340`	`2341`	`#if defined(SENTRY_PLATFORM_UNIX)`
`@@ -2750,8 +2751,6 @@ sentry__process_crash(const sentry_options_t options, sentry_crash_ipc_t ipc)`
`2750`	`2751`	`SENTRY_WARN("No transport available for sending envelope");`
`2751`	`2752`	`sentry_envelope_free(envelope);`
`2752`	`2753`	`}`
`2753`		`- } else {`
`2754`		`- sentry_envelope_free(envelope);`
`2755`	`2754`	`}`
`2756`	`2755`
`2757`	`2756`	`// Clean up temporary envelope file (keep minidump for`