Turn on -Wextra, and silence a warning that it brought about.

7042f584 · Geoff Simmons · ba187152 · 7042f584 · 7042f584
Commit 7042f584 authored May 23, 2017 by Geoff Simmons
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 2 deletions

Makefile.am src/Makefile.am +1 -1

vmod_blob.c src/vmod_blob.c +2 -1

No files found.
--- a/src/Makefile.am
+++ b/src/Makefile.am
-AM_CPPFLAGS = @VMOD_INCLUDES@ -Wall -Werror
+AM_CPPFLAGS = @VMOD_INCLUDES@ -Wall -Werror -Wextra
 CFLAGS += -std=c99
 vmoddir = @VMOD_DIR@

--- a/src/vmod_blob.c
+++ b/src/vmod_blob.c
@@ -92,7 +92,8 @@ vmod_integer(VRT_CTX, VCL_BLOB b, VCL_ENUM padding)
 {
 	VCL_INT i = 0;
-	if (b->len >= sizeof(VCL_INT))
+	assert(b->len >= 0);
+	if ((unsigned)b->len >= sizeof(VCL_INT))
 		return *((VCL_INT *) b->priv);
 	if (b->len == 0) {
 		ERR(ctx, "blob length is 0 in blob.integer()");