From 84b7a37869ac87aea595b57075d895d0035026d0 Mon Sep 17 00:00:00 2001 From: "Lubomir I. Ivanov" Date: Fri, 13 Nov 2015 15:20:48 +0200 Subject: subsurfacestartup.c: leave free_prefs() empty There is a bug on OSX where free() is called on non-allocated memory in free_prefs(). Most of the preferences are not freed in free_prefs() while copy_string() is used on them, so let's not free() any pointers in free_prefs() and leave them as one-time leaks. Signed-off-by: Lubomir I. Ivanov Signed-off-by: Dirk Hohndel --- subsurface-core/subsurfacestartup.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) (limited to 'subsurface-core/subsurfacestartup.c') diff --git a/subsurface-core/subsurfacestartup.c b/subsurface-core/subsurfacestartup.c index 5b3dfa8c1..c39c89c17 100644 --- a/subsurface-core/subsurfacestartup.c +++ b/subsurface-core/subsurfacestartup.c @@ -313,11 +313,5 @@ void copy_prefs(struct preferences *src, struct preferences *dest) */ void free_prefs(void) { - free((void*)prefs.default_cylinder); - free((void*)prefs.divelist_font); - free((void*)prefs.cloud_storage_password); - free(prefs.proxy_host); - free(prefs.proxy_user); - free(prefs.proxy_pass); - free(prefs.userid); + // nop } -- cgit v1.2.3-70-g09d2