all repos — hex @ a48690e3a68a3dff333dce00fe5d4169edcfc086

A tiny, minimalist, slightly-esoteric concatenative programming lannguage. 

Refactor memory allocation in hex_item_t functions to use calloc for zero-initialization; enhances safety and prevents uninitialized memory usage.
h3rald h3rald@h3rald.com
Wed, 10 Sep 2025 17:45:22 +0200
commit

a48690e3a68a3dff333dce00fe5d4169edcfc086

parent

0a0442f643bf6f9010035999d7112fed3ba024b8

4 files changed, 38 insertions(+), 20 deletions(-)

jump to
M src/hex.c → src/hex.c 
@@ -562,7 +562,7 @@ {
         hex_error(ctx, "[create string] Failed to allocate memory for string");
         return NULL;
     }
-    hex_item_t *item = malloc(sizeof(hex_item_t));
+    hex_item_t *item = calloc(1, sizeof(hex_item_t));
     if (item == NULL)
     {
         hex_error(ctx, "[create string] Failed to allocate memory for item");

@@ -571,13 +571,16 @@ return NULL;
     }
     item->type = HEX_TYPE_STRING;
     item->data.str_value = str;
+    item->is_operator = 0;
+    item->token = NULL;
+    item->quotation_size = 0;
     return item;
 }
 
 hex_item_t *hex_integer_item(hex_context_t *ctx, int value)
 {
     (void)(ctx);
-    hex_item_t *item = malloc(sizeof(hex_item_t));
+    hex_item_t *item = calloc(1, sizeof(hex_item_t));
     if (item == NULL)
     {
         hex_error(ctx, "[create integer] Failed to allocate memory for item");

@@ -585,12 +588,15 @@ return NULL;
     }
     item->type = HEX_TYPE_INTEGER;
     item->data.int_value = value;
+    item->is_operator = 0;
+    item->token = NULL;
+    item->quotation_size = 0;
     return item;
 }
 
 hex_item_t *hex_quotation_item(hex_context_t *ctx, hex_item_t **quotation, size_t size)
 {
-    hex_item_t *item = malloc(sizeof(hex_item_t));
+    hex_item_t *item = calloc(1, sizeof(hex_item_t));
     if (item == NULL)
     {
         hex_error(ctx, "[create quotation] Failed to allocate memory for item");

@@ -600,12 +606,13 @@ item->is_operator = 0;
     item->type = HEX_TYPE_QUOTATION;
     item->data.quotation_value = quotation;
     item->quotation_size = size;
+    item->token = NULL;
     return item;
 }
 
 hex_item_t *hex_symbol_item(hex_context_t *ctx, hex_token_t *token)
 {
-    hex_item_t *item = malloc(sizeof(hex_item_t));
+    hex_item_t *item = calloc(1, sizeof(hex_item_t));
     if (item == NULL)
     {
         hex_error(ctx, "[create symbol] Failed to allocate memory for item");

@@ -613,6 +620,8 @@ return NULL;
     }
     item->type = hex_valid_native_symbol(ctx, token->value) ? HEX_TYPE_NATIVE_SYMBOL : HEX_TYPE_USER_SYMBOL;
     item->token = token;
+    item->is_operator = 0;
+    item->quotation_size = 0;
     return item;
 }
 

@@ -861,7 +870,7 @@ return NULL;
     }
 
     // Allocate memory for the new hex_item_t structure
-    hex_item_t *copy = (hex_item_t *)malloc(sizeof(hex_item_t));
+    hex_item_t *copy = (hex_item_t *)calloc(1, sizeof(hex_item_t));
     if (!copy)
     {
         hex_error(ctx, "[copy item] Failed to allocate memory for item copy");

@@ -1175,7 +1184,7 @@ }
 
 void hex_set_native_symbol(hex_context_t *ctx, const char *name, int (*func)())
 {
-    hex_item_t *func_item = malloc(sizeof(hex_item_t));
+    hex_item_t *func_item = calloc(1, sizeof(hex_item_t));
     if (func_item == NULL)
     {
         hex_error(ctx, "[set native symbol] Memory allocation failed for native symbol '%s'", name);

@@ -1184,7 +1193,7 @@ }
     func_item->type = HEX_TYPE_NATIVE_SYMBOL;
     func_item->data.fn_value = func;
     // Need to create a fake token for native symbols as well.
-    func_item->token = malloc(sizeof(hex_token_t));
+    func_item->token = calloc(1, sizeof(hex_token_t));
     func_item->token->type = HEX_TOKEN_SYMBOL;
     func_item->token->value = strdup(name);
     func_item->token->position = NULL;

@@ -4064,7 +4073,7 @@ return 1;
             }
 
             // Allocate a new hex_item_t
-            hex_item_t *item = (hex_item_t *)malloc(sizeof(hex_item_t));
+            hex_item_t *item = (hex_item_t *)calloc(1, sizeof(hex_item_t));
             if (!item)
             {
                 hex_error(ctx, "[symbol symbols] Memory allocation failed for hex_item_t");

@@ -6443,7 +6452,7 @@ HEX_FREE(ctx, item);
         return 1;
     }
 
-    hex_item_t *quotation = (hex_item_t *)malloc(sizeof(hex_item_t));
+    hex_item_t *quotation = (hex_item_t *)calloc(1, sizeof(hex_item_t));
     if (!quotation)
     {
         hex_error(ctx, "[symbol '] Memory allocation failed");

@@ -6453,7 +6462,7 @@ }
 
     *quotation = *item;
 
-    hex_item_t *result = (hex_item_t *)malloc(sizeof(hex_item_t));
+    hex_item_t *result = (hex_item_t *)calloc(1, sizeof(hex_item_t));
     if (!result)
     {
         hex_error(ctx, "[symbol '] Memory allocation failed");
M src/registry.c → src/registry.c 
@@ -206,7 +206,7 @@ }
 
 void hex_set_native_symbol(hex_context_t *ctx, const char *name, int (*func)())
 {
-    hex_item_t *func_item = malloc(sizeof(hex_item_t));
+    hex_item_t *func_item = calloc(1, sizeof(hex_item_t));
     if (func_item == NULL)
     {
         hex_error(ctx, "[set native symbol] Memory allocation failed for native symbol '%s'", name);

@@ -215,7 +215,7 @@ }
     func_item->type = HEX_TYPE_NATIVE_SYMBOL;
     func_item->data.fn_value = func;
     // Need to create a fake token for native symbols as well.
-    func_item->token = malloc(sizeof(hex_token_t));
+    func_item->token = calloc(1, sizeof(hex_token_t));
     func_item->token->type = HEX_TOKEN_SYMBOL;
     func_item->token->value = strdup(name);
     func_item->token->position = NULL;
M src/stack.c → src/stack.c 
@@ -130,7 +130,7 @@ {
         hex_error(ctx, "[create string] Failed to allocate memory for string");
         return NULL;
     }
-    hex_item_t *item = malloc(sizeof(hex_item_t));
+    hex_item_t *item = calloc(1, sizeof(hex_item_t));
     if (item == NULL)
     {
         hex_error(ctx, "[create string] Failed to allocate memory for item");

@@ -139,13 +139,16 @@ return NULL;
     }
     item->type = HEX_TYPE_STRING;
     item->data.str_value = str;
+    item->is_operator = 0;
+    item->token = NULL;
+    item->quotation_size = 0;
     return item;
 }
 
 hex_item_t *hex_integer_item(hex_context_t *ctx, int value)
 {
     (void)(ctx);
-    hex_item_t *item = malloc(sizeof(hex_item_t));
+    hex_item_t *item = calloc(1, sizeof(hex_item_t));
     if (item == NULL)
     {
         hex_error(ctx, "[create integer] Failed to allocate memory for item");

@@ -153,12 +156,15 @@ return NULL;
     }
     item->type = HEX_TYPE_INTEGER;
     item->data.int_value = value;
+    item->is_operator = 0;
+    item->token = NULL;
+    item->quotation_size = 0;
     return item;
 }
 
 hex_item_t *hex_quotation_item(hex_context_t *ctx, hex_item_t **quotation, size_t size)
 {
-    hex_item_t *item = malloc(sizeof(hex_item_t));
+    hex_item_t *item = calloc(1, sizeof(hex_item_t));
     if (item == NULL)
     {
         hex_error(ctx, "[create quotation] Failed to allocate memory for item");

@@ -168,12 +174,13 @@ item->is_operator = 0;
     item->type = HEX_TYPE_QUOTATION;
     item->data.quotation_value = quotation;
     item->quotation_size = size;
+    item->token = NULL;
     return item;
 }
 
 hex_item_t *hex_symbol_item(hex_context_t *ctx, hex_token_t *token)
 {
-    hex_item_t *item = malloc(sizeof(hex_item_t));
+    hex_item_t *item = calloc(1, sizeof(hex_item_t));
     if (item == NULL)
     {
         hex_error(ctx, "[create symbol] Failed to allocate memory for item");

@@ -181,6 +188,8 @@ return NULL;
     }
     item->type = hex_valid_native_symbol(ctx, token->value) ? HEX_TYPE_NATIVE_SYMBOL : HEX_TYPE_USER_SYMBOL;
     item->token = token;
+    item->is_operator = 0;
+    item->quotation_size = 0;
     return item;
 }
 

@@ -429,7 +438,7 @@ return NULL;
     }
 
     // Allocate memory for the new hex_item_t structure
-    hex_item_t *copy = (hex_item_t *)malloc(sizeof(hex_item_t));
+    hex_item_t *copy = (hex_item_t *)calloc(1, sizeof(hex_item_t));
     if (!copy)
     {
         hex_error(ctx, "[copy item] Failed to allocate memory for item copy");
M src/symbols.c → src/symbols.c 
@@ -153,7 +153,7 @@ return 1;
             }
 
             // Allocate a new hex_item_t
-            hex_item_t *item = (hex_item_t *)malloc(sizeof(hex_item_t));
+            hex_item_t *item = (hex_item_t *)calloc(1, sizeof(hex_item_t));
             if (!item)
             {
                 hex_error(ctx, "[symbol symbols] Memory allocation failed for hex_item_t");

@@ -2532,7 +2532,7 @@ HEX_FREE(ctx, item);
         return 1;
     }
 
-    hex_item_t *quotation = (hex_item_t *)malloc(sizeof(hex_item_t));
+    hex_item_t *quotation = (hex_item_t *)calloc(1, sizeof(hex_item_t));
     if (!quotation)
     {
         hex_error(ctx, "[symbol '] Memory allocation failed");

@@ -2542,7 +2542,7 @@ }
 
     *quotation = *item;
 
-    hex_item_t *result = (hex_item_t *)malloc(sizeof(hex_item_t));
+    hex_item_t *result = (hex_item_t *)calloc(1, sizeof(hex_item_t));
     if (!result)
     {
         hex_error(ctx, "[symbol '] Memory allocation failed");