From 9d01d090c48c74a29b4ef67e0cd204772a2193c3 Mon Sep 17 00:00:00 2001
From: Simon Howard
Date: Fri, 24 Oct 2014 20:29:56 -0400
Subject: Replace strdup() with M_StringDuplicate().

strdup() can theoretically fail and return NULL. This could lead to
a crash or undesirable behavior. Add M_StringDuplicate() which does
the same thing but exits with an error if a string cannot be
allocated.

This fixes #456. Thanks to Quasar for the suggestion.
---
 src/heretic/g_game.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src/heretic')

diff --git a/src/heretic/g_game.c b/src/heretic/g_game.c
index da96537a..389c0e1d 100644
--- a/src/heretic/g_game.c
+++ b/src/heretic/g_game.c
@@ -1429,7 +1429,7 @@ static char *savename = NULL;
 
 void G_LoadGame(char *name)
 {
-    savename = strdup(name);
+    savename = M_StringDuplicate(name);
     gameaction = ga_loadgame;
 }
 
-- 
cgit v1.2.3