c/profile: Short circuit on errors

d-e-s-o · anakryiko · commit 00c99c0da19f · 2024-07-24T14:05:49.000-07:00
Report proper errors when symbolization fails and bail out early.

Signed-off-by: Daniel Müller &lt;deso@posteo.net&gt;
diff --git a/examples/c/profile.c b/examples/c/profile.c
@@ -80,6 +80,10 @@ static void show_stack_trace(__u64 *stack, int stack_sz, pid_t pid)
 		result = blaze_symbolize_kernel_abs_addrs(symbolizer, &src, (const uintptr_t *)stack, stack_sz);
 	}
 
+	if (result == NULL) {
+		printf("  failed to symbolize addresses: %s\n", blaze_err_str(blaze_err_last()));
+		return;
+	}
 
 	for (i = 0; i < stack_sz; i++) {
 		if (!result || result->cnt <= i || result->syms[i].name == NULL) {

Original file line number	Diff line number	Diff line change
`@@ -80,6 +80,10 @@ static void show_stack_trace(__u64 *stack, int stack_sz, pid_t pid)`
`80`	`80`	`result = blaze_symbolize_kernel_abs_addrs(symbolizer, &src, (const uintptr_t *)stack, stack_sz);`
`81`	`81`	`}`
`82`	`82`
	`83`	`+ if (result == NULL) {`
	`84`	`+ printf(" failed to symbolize addresses: %s\n", blaze_err_str(blaze_err_last()));`
	`85`	`+ return;`
	`86`	`+ }`
`83`	`87`
`84`	`88`	`for (i = 0; i < stack_sz; i++) {`
`85`	`89`	`if (!result \|\| result->cnt <= i \|\| result->syms[i].name == NULL) {`