From: Paul Tsochantaris <redacted>
Date: Mon, 29 Jan 2024 22:19:29 +0000 (+0000)
Subject: ggml alloc: Fix for null dereference on alloc failure (#5200)
X-Git-Tag: upstream/0.0.4488~2479
X-Git-Url: https://git.djapps.eu/?a=commitdiff_plain;h=ceebbb5b21b971941b2533210b74bf359981006c;p=pkg%2Fggml%2Fsources%2Fllama.cpp

ggml alloc: Fix for null dereference on alloc failure (#5200)

* Fix for a null pointer dereference if a metal GGML buffer fails to be allocated

* Freeing the allocated buffers rather than the pointer in ggml-alloc.c

* Fixed the fix of the fix
---

diff --git a/ggml-alloc.c b/ggml-alloc.c
index dfe5ba2e..f9be6e1c 100644
--- a/ggml-alloc.c
+++ b/ggml-alloc.c
@@ -791,7 +791,7 @@ static bool alloc_tensor_range(struct ggml_context * ctx,
         for (size_t i = 0; i < *n_buffers; i++) {
             ggml_backend_buffer_free(*buffers[i]);
         }
-        free(buffers);
+        free(*buffers);
         return false;
     }