diff options
| author | Simon Howard | 2014-10-24 20:29:56 -0400 |
|---|---|---|
| committer | Simon Howard | 2014-10-24 20:29:56 -0400 |
| commit | 9d01d090c48c74a29b4ef67e0cd204772a2193c3 (patch) | |
| tree | 3a8dce09c81c6cd9db8adf266d5cc6eae366af98 /src/m_misc.h | |
| parent | b42b5269e0ad5b22acd6043429ec4013a4e76ddd (diff) | |
| download | chocolate-doom-9d01d090c48c74a29b4ef67e0cd204772a2193c3.tar.gz chocolate-doom-9d01d090c48c74a29b4ef67e0cd204772a2193c3.tar.bz2 chocolate-doom-9d01d090c48c74a29b4ef67e0cd204772a2193c3.zip | |
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.
Diffstat (limited to 'src/m_misc.h')
| -rw-r--r-- | src/m_misc.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/m_misc.h b/src/m_misc.h index 99151f56..844b4859 100644 --- a/src/m_misc.h +++ b/src/m_misc.h @@ -35,6 +35,7 @@ boolean M_StrToInt(const char *str, int *result); void M_ExtractFileBase(char *path, char *dest); void M_ForceUppercase(char *text); char *M_StrCaseStr(char *haystack, char *needle); +char *M_StringDuplicate(const char *orig); boolean M_StringCopy(char *dest, const char *src, size_t dest_size); boolean M_StringConcat(char *dest, const char *src, size_t dest_size); char *M_StringReplace(const char *haystack, const char *needle, |