summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSaagar Jha <saagar@saagarjha.com>2023-04-10 01:34:48 -0700
committerSaagar Jha <saagar@saagarjha.com>2023-04-10 01:34:48 -0700
commitea9c030249cb3db7a923c8e546df9897e0a39384 (patch)
treefa4bc80525da91bfe9da28704e21a4c9efe1c77d
parent632bbe475c2d948cdae4eaba3367b9e65c5ff141 (diff)
Fix heap overflow from curl-originating buffers
Most of xs.h seems to expect that buffers are rounded up to block size, so we should preserve that invariant here. (In particular, xs_expand will avoid calling xs_realloc if the new size fits in the same block, which means that if we don't pad out the data it will expand out of the memory we're allocated.)
-rw-r--r--xs_curl.h2
1 files changed, 1 insertions, 1 deletions
diff --git a/xs_curl.h b/xs_curl.h
index e880a0d..ca90f92 100644
--- a/xs_curl.h
+++ b/xs_curl.h
@@ -55,7 +55,7 @@ static int _data_callback(void *buffer, size_t size,
/* open space */
pd->size += sz;
- pd->data = xs_realloc(pd->data, pd->size + 1);
+ pd->data = xs_realloc(pd->data, _xs_blk_size(pd->size + 1));
/* copy data */
memcpy(pd->data + pd->offset, buffer, sz);