[master] 243d815 At least one platform has a backtrace(3) which returns -1 in a size_t :-(

Poul-Henning Kamp phk at FreeBSD.org
Sun May 29 22:28:05 CEST 2016


commit 243d81567caebaf22f726bf1dea3eadadc07de84
Author: Poul-Henning Kamp <phk at FreeBSD.org>
Date:   Sun May 29 20:27:09 2016 +0000

    At least one platform has a backtrace(3) which returns -1 in a size_t :-(

diff --git a/bin/varnishd/cache/cache_panic.c b/bin/varnishd/cache/cache_panic.c
index 53ef3cd..5657535 100644
--- a/bin/varnishd/cache/cache_panic.c
+++ b/bin/varnishd/cache/cache_panic.c
@@ -516,14 +516,16 @@ static void
 pan_backtrace(struct vsb *vsb)
 {
 	void *array[10];
-	size_t size;
-	size_t i;
+	ssize_t size;
+	ssize_t i;
 	char **strings;
 	char *p;
 
 	size = backtrace (array, 10);
-	if (size == 0)
+	if (size <= 0) {
+		VSB_printf(vsb, "Backtrace not available (%zd)\n", size);
 		return;
+	}
 	VSB_printf(vsb, "Backtrace:\n");
 	VSB_indent(vsb, 2);
 	for (i = 0; i < size; i++) {



More information about the varnish-commit mailing list