[master] 6394421d8 forgotten (char *) cast for %s format

Nils Goroll nils.goroll at uplex.de
Sat Jan 16 11:14:06 UTC 2021


commit 6394421d877005f1a0d01c0ff556e2b6bb150362
Author: Nils Goroll <nils.goroll at uplex.de>
Date:   Sat Jan 16 12:11:39 2021 +0100

    forgotten (char *) cast for %s format
    
    seems to be a real benefit of FreeBSD 13 vtest over all the other
    vtesters. Not even the compulsively obsessive Flexelint complained.

diff --git a/vmod/vmod_debug.c b/vmod/vmod_debug.c
index e92a4671a..4a29a8049 100644
--- a/vmod/vmod_debug.c
+++ b/vmod/vmod_debug.c
@@ -234,7 +234,7 @@ priv_ ## name ## _fini(void *ptr)					\
 	const char * const fmt = "priv_" #name "_fini(%p)";		\
 									\
 	AN ## assert (ptr);						\
-	VSL(SLT_Debug, 0, fmt, ptr);					\
+	VSL(SLT_Debug, 0, fmt, (char *)ptr);				\
 	free(ptr);							\
 }									\
 									\
diff --git a/vmod/vmod_debug_obj.c b/vmod/vmod_debug_obj.c
index 8c0e60070..25ec4733d 100644
--- a/vmod/vmod_debug_obj.c
+++ b/vmod/vmod_debug_obj.c
@@ -159,7 +159,7 @@ obj_priv_ ## name ## _fini(void *ptr)					\
 	const char * const fmt = "obj_priv_" #name "_fini(%p = \"%s\")"; \
 									\
 	AN(ptr);							\
-	VSL(SLT_Debug, 0, fmt, ptr, ptr);				\
+	VSL(SLT_Debug, 0, fmt, ptr, (char *)ptr);			\
 }									\
 									\
 static const struct vmod_priv_methods					\


More information about the varnish-commit mailing list