diff options
author | Dirk Hohndel <dirk@hohndel.org> | 2012-09-11 20:52:54 -0700 |
---|---|---|
committer | Dirk Hohndel <dirk@hohndel.org> | 2012-09-11 20:52:54 -0700 |
commit | 1afe36840ddbc2d139c1f7e2b3607aecc88d9890 (patch) | |
tree | 8917ae5dd3b886000a141073539bb23e35db0a76 /macos.c | |
parent | be941e00b20cc0b8b3b46e04200ce15c3023ad23 (diff) | |
download | subsurface-1afe36840ddbc2d139c1f7e2b3607aecc88d9890.tar.gz |
Fix memory handling error on MacOS
We are not allowed to free a string that we get back from the config APIs. So
strdup it instead to be compatible with Linux in that respect.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'macos.c')
-rw-r--r-- | macos.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -54,7 +54,7 @@ const void *subsurface_get_conf(char *name, pref_type_t type) strpref = CFPreferencesCopyAppValue(CFSTR_VAR(name), SUBSURFACE_PREFERENCES); if (!strpref) return NULL; - return CFStringGetCStringPtr(strpref, kCFStringEncodingMacRoman); + return strdup(CFStringGetCStringPtr(strpref, kCFStringEncodingMacRoman)); } /* we shouldn't get here, but having this line makes the compiler happy */ return NULL; |