[master] 5036291c7 gzip: Check buffers arguments

Dridi Boukelmoune dridi.boukelmoune at gmail.com
Mon Apr 14 15:49:05 UTC 2025


commit 5036291c75fcc0d4c6292d90192cea6469a233ba
Author: Dridi Boukelmoune <dridi.boukelmoune at gmail.com>
Date:   Thu Mar 13 09:54:21 2025 +0100

    gzip: Check buffers arguments

diff --git a/bin/varnishd/cache/cache_gzip.c b/bin/varnishd/cache/cache_gzip.c
index 1bc717bf5..06eb692ec 100644
--- a/bin/varnishd/cache/cache_gzip.c
+++ b/bin/varnishd/cache/cache_gzip.c
@@ -175,8 +175,11 @@ VGZ_Ibuf(struct vgz *vg, const void *ptr, ssize_t len)
 {
 
 	CHECK_OBJ_NOTNULL(vg, VGZ_MAGIC);
-
 	AZ(vg->vz.avail_in);
+	assert(len >= 0);
+	if (len > 0)
+		AN(ptr);
+
 	vg->vz.next_in = TRUST_ME(ptr);
 	vg->vz.avail_in = len;
 }
@@ -196,6 +199,8 @@ VGZ_Obuf(struct vgz *vg, void *ptr, ssize_t len)
 {
 
 	CHECK_OBJ_NOTNULL(vg, VGZ_MAGIC);
+	AN(ptr);
+	assert(len > 0);
 
 	vg->vz.next_out = ptr;
 	vg->vz.avail_out = len;


More information about the varnish-commit mailing list