Use g_clear_error(&err) instead of g_error_free(err); err = NULL

5 files changed, 25 insertions, 49 deletions

diff --git a/audio/a2dp.c b/audio/a2dp.c
index 49312a71..8c38a90b 100644
--- a/audio/a2dp.c
+++ b/audio/a2dp.c
@@ -1071,8 +1071,7 @@ int a2dp_register(DBusConnection *conn, const bdaddr_t *src, GKeyFile *config)
 
 	if (err) {
 		debug("audio.conf: %s", err->message);
-		g_error_free(err);
-		err = NULL;
+		g_clear_error(&err);
 	} else {
 		if (strstr(str, "Sink"))
 			source = FALSE;
@@ -1084,8 +1083,7 @@ int a2dp_register(DBusConnection *conn, const bdaddr_t *src, GKeyFile *config)
 	str = g_key_file_get_string(config, "A2DP", "SBCSources", &err);
 	if (err) {
 		debug("audio.conf: %s", err->message);
-		g_error_free(err);
-		err = NULL;
+		g_clear_error(&err);
 	} else {
 		sbc_srcs = atoi(str);
 		g_free(str);
@@ -1094,8 +1092,7 @@ int a2dp_register(DBusConnection *conn, const bdaddr_t *src, GKeyFile *config)
 	str = g_key_file_get_string(config, "A2DP", "MPEG12Sources", &err);
 	if (err) {
 		debug("audio.conf: %s", err->message);
-		g_error_free(err);
-		err = NULL;
+		g_clear_error(&err);
 	} else {
 		mpeg12_srcs = atoi(str);
 		g_free(str);
@@ -1104,8 +1101,7 @@ int a2dp_register(DBusConnection *conn, const bdaddr_t *src, GKeyFile *config)
 	str = g_key_file_get_string(config, "A2DP", "SBCSinks", &err);
 	if (err) {
 		debug("audio.conf: %s", err->message);
-		g_error_free(err);
-		err = NULL;
+		g_clear_error(&err);
 	} else {
 		sbc_sinks = atoi(str);
 		g_free(str);
@@ -1114,8 +1110,7 @@ int a2dp_register(DBusConnection *conn, const bdaddr_t *src, GKeyFile *config)
 	str = g_key_file_get_string(config, "A2DP", "MPEG12Sinks", &err);
 	if (err) {
 		debug("audio.conf: %s", err->message);
-		g_error_free(err);
-		err = NULL;
+		g_clear_error(&err);
 	} else {
 		mpeg12_sinks = atoi(str);
 		g_free(str);
diff --git a/audio/headset.c b/audio/headset.c
index 0a7f18ed..9ba2138a 100644
--- a/audio/headset.c
+++ b/audio/headset.c
@@ -2040,8 +2040,7 @@ uint32_t headset_config_init(GKeyFile *config)
 					&err);
 	if (err) {
 		debug("audio.conf: %s", err->message);
-		g_error_free(err);
-		err = NULL;
+		g_clear_error(&err);
 	} else {
 		if (strcmp(str, "PCM") == 0)
 			sco_hci = FALSE;
diff --git a/audio/manager.c b/audio/manager.c
index 4441b90f..816e6821 100644
--- a/audio/manager.c
+++ b/audio/manager.c
@@ -510,8 +510,7 @@ static int headset_server_init(struct audio_adapter *adapter)
 						&err);
 		if (err) {
 			debug("audio.conf: %s", err->message);
-			g_error_free(err);
-			err = NULL;
+			g_clear_error(&err);
 		} else
 			master = tmp;
 	}
@@ -595,8 +594,7 @@ static int gateway_server_init(struct audio_adapter *adapter)
 						&err);
 		if (err) {
 			debug("audio.conf: %s", err->message);
-			g_error_free(err);
-			err = NULL;
+			g_clear_error(&err);
 		} else
 			master = tmp;
 	}
@@ -918,8 +916,7 @@ int audio_manager_init(DBusConnection *conn, GKeyFile *conf)
 					&err);
 	if (err) {
 		debug("audio.conf: %s", err->message);
-		g_error_free(err);
-		err = NULL;
+		g_clear_error(&err);
 	} else
 		enabled.hfp = b;
 
@@ -928,8 +925,7 @@ int audio_manager_init(DBusConnection *conn, GKeyFile *conf)
 					&err);
 	if (err) {
 		debug("audio.conf: %s", err->message);
-		g_error_free(err);
-		err = NULL;
+		g_clear_error(&err);
 	} else
 		max_connected_headsets = i;
 
diff --git a/network/manager.c b/network/manager.c
index 564af2ad..1b68ea2a 100644
--- a/network/manager.c
+++ b/network/manager.c
@@ -95,7 +95,7 @@ static void read_config(const char *file)
 
 	if (!g_key_file_load_from_file(keyfile, file, 0, &err)) {
 		error("Parsing %s failed: %s", file, err->message);
-		g_error_free(err);
+		g_clear_error(&err);
 		goto done;
 	}
 
@@ -103,8 +103,7 @@ static void read_config(const char *file)
 						"Disable", NULL, &err);
 	if (err) {
 		debug("%s: %s", file, err->message);
-		g_error_free(err);
-		err = NULL;
+		g_clear_error(&err);
 	} else {
 		int i;
 		for (i = 0; disabled[i] != NULL; i++) {
@@ -120,56 +119,49 @@ static void read_config(const char *file)
 						"DisableSecurity", &err);
 	if (err) {
 		debug("%s: %s", file, err->message);
-		g_error_free(err);
-		err = NULL;
+		g_clear_error(&err);
 	}
 
 	conf.panu_script = g_key_file_get_string(keyfile, "PANU Role",
 						"Script", &err);
 	if (err) {
 		debug("%s: %s", file, err->message);
-		g_error_free(err);
-		err = NULL;
+		g_clear_error(&err);
 	}
 
 	conf.gn_script = g_key_file_get_string(keyfile, "GN Role",
 						"Script", &err);
 	if (err) {
 		debug("%s: %s", file, err->message);
-		g_error_free(err);
-		err = NULL;
+		g_clear_error(&err);
 	}
 
 	conf.nap_script = g_key_file_get_string(keyfile, "NAP Role",
 						"Script", &err);
 	if (err) {
 		debug("%s: %s", file, err->message);
-		g_error_free(err);
-		err = NULL;
+		g_clear_error(&err);
 	}
 
 	conf.iface_prefix = g_key_file_get_string(keyfile, "PANU Role",
 						"Interface", &err);
 	if (err) {
 		debug("%s: %s", file, err->message);
-		g_error_free(err);
-		err = NULL;
+		g_clear_error(&err);
 	}
 
 	conf.gn_iface = g_key_file_get_string(keyfile, "GN Role",
 						"Interface", &err);
 	if (err) {
 		debug("%s: %s", file, err->message);
-		g_error_free(err);
-		err = NULL;
+		g_clear_error(&err);
 	}
 
 	conf.nap_iface = g_key_file_get_string(keyfile, "NAP Role",
 						"Interface", &err);
 	if (err) {
 		debug("%s: %s", file, err->message);
-		g_error_free(err);
-		err = NULL;
+		g_clear_error(&err);
 	}
 
 done:
diff --git a/src/main.c b/src/main.c
index 9bfeb512..407a1122 100644
--- a/src/main.c
+++ b/src/main.c
@@ -106,8 +106,7 @@ static void parse_config(GKeyFile *config)
 					"OffMode", &err);
 	if (err) {
 		debug("%s", err->message);
-		g_error_free(err);
-		err = NULL;
+		g_clear_error(&err);
 	} else {
 		debug("offmode=%s", str);
 		if (g_str_equal(str, "DevDown"))
@@ -120,8 +119,7 @@ static void parse_config(GKeyFile *config)
 					&err);
 	if (err) {
 		debug("%s", err->message);
-		g_error_free(err);
-		err = NULL;
+		g_clear_error(&err);
 	} else {
 		debug("discovto=%d", val);
 		main_opts.discovto = val;
@@ -133,8 +131,7 @@ static void parse_config(GKeyFile *config)
 					&err);
 	if (err) {
 		debug("%s", err->message);
-		g_error_free(err);
-		err = NULL;
+		g_clear_error(&err);
 	} else {
 		debug("pageto=%d", val);
 		main_opts.pageto = val;
@@ -145,8 +142,7 @@ static void parse_config(GKeyFile *config)
 					"Name", &err);
 	if (err) {
 		debug("%s", err->message);
-		g_error_free(err);
-		err = NULL;
+		g_clear_error(&err);
 	} else {
 		debug("name=%s", str);
 		g_free(main_opts.name);
@@ -159,8 +155,7 @@ static void parse_config(GKeyFile *config)
 					"Class", &err);
 	if (err) {
 		debug("%s", err->message);
-		g_error_free(err);
-		err = NULL;
+		g_clear_error(&err);
 	} else {
 		debug("class=%s", str);
 		main_opts.class = strtol(str, NULL, 16);
@@ -173,8 +168,7 @@ static void parse_config(GKeyFile *config)
 					&err);
 	if (err) {
 		debug("%s", err->message);
-		g_error_free(err);
-		err = NULL;
+		g_clear_error(&err);
 	} else {
 		debug("inqmode=%d", val);
 		main_opts.inqmode = val;