From a1faa348cb92e7de012d942c04b2ec589ce4b1f1 Mon Sep 17 00:00:00 2001 From: Matthew Fernandez Date: Fri, 24 Jun 2022 07:13:41 -0700 Subject: [PATCH] gvplugin_package_record: replace unchecked 'strdup' with alloc wrapper This turns what was previously a crash on out-of-memory into a graceful exit. --- lib/gvc/gvconfig.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/lib/gvc/gvconfig.c b/lib/gvc/gvconfig.c index a1c9d4856..1b8c0af50 100644 --- a/lib/gvc/gvconfig.c +++ b/lib/gvc/gvconfig.c @@ -14,6 +14,7 @@ #define _GNU_SOURCE 1 #endif +#include #include #include @@ -94,8 +95,8 @@ static gvplugin_package_t * gvplugin_package_record(GVC_t * gvc, const char *package_path, const char *name) { gvplugin_package_t *package = gmalloc(sizeof(gvplugin_package_t)); - package->path = package_path ? strdup(package_path) : NULL; - package->name = strdup(name); + package->path = package_path ? gv_strdup(package_path) : NULL; + package->name = gv_strdup(name); package->next = gvc->packages; gvc->packages = package; return package; -- 2.40.0