Skip to content

Commit e7a5ac6

Browse files
committed
DEBUG
1 parent 1979e99 commit e7a5ac6

4 files changed

Lines changed: 382 additions & 13 deletions

File tree

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -467,7 +467,7 @@ jobs:
467467
shell: bash
468468
run: |
469469
pip install --upgrade --requirement tests/requirements.txt
470-
pytest --capture=no --verbose tests
470+
pytest --capture=no --verbose tests/test_integration_native.py
471471
472472
- name: Check NDK
473473
if: ${{ env['ANDROID_API'] }}

src/backends/native/sentry_crash_daemon.c

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,21 @@
5757
# include <sys/stat.h>
5858
# include <windows.h>
5959

60+
static void
61+
daemon_debugf(const char *fmt, ...)
62+
{
63+
char msg[1024];
64+
va_list args;
65+
va_start(args, fmt);
66+
int n = vsnprintf(msg, sizeof(msg), fmt, args);
67+
va_end(args);
68+
if (n < 0) {
69+
return;
70+
}
71+
msg[sizeof(msg) - 1] = '\0';
72+
OutputDebugStringA(msg);
73+
}
74+
6075
// Forward declaration for StackWalk64-based stack unwinding (defined later)
6176
static size_t walk_stack_with_dbghelp(HANDLE hProcess, DWORD crashed_tid,
6277
const CONTEXT *ctx_record, sentry_frame_info_t *frames, size_t max_frames);
@@ -2126,6 +2141,7 @@ static bool
21262141
add_wer_context(sentry_value_t event, const sentry_crash_context_t *ctx)
21272142
{
21282143
if (!ctx->platform.wer_enabled) {
2144+
daemon_debugf("### DAEMON: add_wer_context skipped wer_enabled=0\n");
21292145
return false;
21302146
}
21312147

@@ -2134,12 +2150,23 @@ add_wer_context(sentry_value_t event, const sentry_crash_context_t *ctx)
21342150
wer_report_id, ctx->platform.wer_report_id, sizeof(wer_report_id) - 1);
21352151
wer_report_id[sizeof(wer_report_id) - 1] = '\0';
21362152

2153+
daemon_debugf(
2154+
"### DAEMON: add_wer_context start event_id=%s report_id=%s\n",
2155+
ctx->crash_event_id,
2156+
sentry__string_empty(wer_report_id) ? "(empty)" : wer_report_id);
2157+
21372158
sentry_value_t wer_context = sentry__wer_report_lookup(ctx->crash_event_id);
21382159
if (sentry_value_is_null(wer_context)) {
2160+
daemon_debugf("### DAEMON: WER metadata xml not found yet\n");
21392161
if (sentry__string_empty(wer_report_id)) {
2162+
daemon_debugf(
2163+
"### DAEMON: no fallback report_id, skipping wer context\n");
21402164
return false;
21412165
}
2166+
daemon_debugf("### DAEMON: using fallback report_id from callback\n");
21422167
wer_context = sentry_value_new_object();
2168+
} else {
2169+
daemon_debugf("### DAEMON: WER metadata xml found\n");
21432170
}
21442171

21452172
sentry_value_set_by_key(
@@ -2157,6 +2184,7 @@ add_wer_context(sentry_value_t event, const sentry_crash_context_t *ctx)
21572184
sentry_value_set_by_key(event, "contexts", contexts);
21582185
}
21592186
sentry_value_set_by_key(contexts, "wer", wer_context);
2187+
daemon_debugf("### DAEMON: add_wer_context attached wer context\n");
21602188
return true;
21612189
}
21622190
#endif

0 commit comments

Comments
 (0)