From a5e78dca9aaca20886e0176f6a5a5899144266b3 Mon Sep 17 00:00:00 2001 From: Eric-Paul Ickhorn Date: Thu, 23 May 2024 00:58:02 +0200 Subject: [PATCH] bugfix(arena): Fixed crash-bug which is triggered by cloning a string into a full arena. When allocating in a full arena, the function 'ufn_arena_alloc' returns NULL. Up to now, the string-clone function would try to copy to there. Now it returs NULL as string-copy when that would've happened. --- inc-c/ufn/ufn_arena.h | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/inc-c/ufn/ufn_arena.h b/inc-c/ufn/ufn_arena.h index 909359c..7fce8fb 100644 --- a/inc-c/ufn/ufn_arena.h +++ b/inc-c/ufn/ufn_arena.h @@ -131,6 +131,10 @@ char * ufn_arena_clone_string(ufn_arena_s *arena, const char *string) { uint32_t len_string = strlen(string); char *string_copy = ufn_arena_alloc(arena, len_string + 1); + if(string_copy == NULL) + { + return NULL; + } memcpy(string_copy, string, len_string + 1); return string_copy; }