Document TryCalloc compatibility

aviggiano · aviggiano · commit a4fef71e7b29 · 2026-01-21T17:31:15.000Z
diff --git a/src/data-structure.c b/src/data-structure.c
@@ -821,6 +821,8 @@ uint32_t* bitmap_range_int_array(const Bitmap* bitmap, size_t start_offset, size
     return NULL;
   }
 
+  // Avoid rm_calloc_try here: RedisModule_TryCalloc is missing on Redis 6.2/7.0 and crashes
+  // when invoked; see https://github.com/aviggiano/redis-roaring/issues/139#issuecomment-3776839439
   uint32_t* ans = rm_calloc(range_size, sizeof(uint32_t));
   if (ans == NULL) {
     return NULL;
@@ -859,6 +861,8 @@ uint64_t* bitmap64_range_int_array(const Bitmap64* bitmap, uint64_t start_offset
     return NULL;
   }
 
+  // Avoid rm_calloc_try here: RedisModule_TryCalloc is missing on Redis 6.2/7.0 and crashes
+  // when invoked; see https://github.com/aviggiano/redis-roaring/issues/139#issuecomment-3776839439
   uint64_t* ans = rm_calloc(range_size, sizeof(uint64_t));
 
   if (ans == NULL) {

Original file line number	Diff line number	Diff line change
`@@ -821,6 +821,8 @@ uint32_t* bitmap_range_int_array(const Bitmap* bitmap, size_t start_offset, size`
`821`	`821`	`return NULL;`
`822`	`822`	`}`
`823`	`823`
	`824`	`+ // Avoid rm_calloc_try here: RedisModule_TryCalloc is missing on Redis 6.2/7.0 and crashes`
	`825`	`+ // when invoked; see https://github.com/aviggiano/redis-roaring/issues/139#issuecomment-3776839439`
`824`	`826`	`uint32_t* ans = rm_calloc(range_size, sizeof(uint32_t));`
`825`	`827`	`if (ans == NULL) {`
`826`	`828`	`return NULL;`
`@@ -859,6 +861,8 @@ uint64_t* bitmap64_range_int_array(const Bitmap64* bitmap, uint64_t start_offset`
`859`	`861`	`return NULL;`
`860`	`862`	`}`
`861`	`863`
	`864`	`+ // Avoid rm_calloc_try here: RedisModule_TryCalloc is missing on Redis 6.2/7.0 and crashes`
	`865`	`+ // when invoked; see https://github.com/aviggiano/redis-roaring/issues/139#issuecomment-3776839439`
`862`	`866`	`uint64_t* ans = rm_calloc(range_size, sizeof(uint64_t));`
`863`	`867`
`864`	`868`	`if (ans == NULL) {`