[PATCH] lib, acpi, hotkey: remove redundant null checks before free()

[Colin King]

From: Colin Ian King <colin.king at canonical.com>

Since free() on a null is a no-op, the null checks before free()
are redundant and a waste of code.  So remove the null checks.
Cleans up a bunch of smatch warnings too.

Signed-off-by: Colin Ian King <colin.king at canonical.com>
---
 src/acpi/crsdump/crsdump.c   |  4 +---
 src/acpi/method/method.c     |  3 +--
 src/acpi/pcc/pcc.c           |  4 +---
 src/acpi/wmi/wmi.c           |  4 +---
 src/hotkey/hotkey/hotkey.c   |  6 ++----
 src/lib/src/fwts_framework.c |  3 +--
 src/lib/src/fwts_klog.c      |  3 +--
 src/lib/src/fwts_log.c       | 10 +++-------
 8 files changed, 11 insertions(+), 26 deletions(-)

diff --git a/src/acpi/crsdump/crsdump.c b/src/acpi/crsdump/crsdump.c
index d6dd6df..94fbf0c 100644
--- a/src/acpi/crsdump/crsdump.c
+++ b/src/acpi/crsdump/crsdump.c
@@ -869,9 +869,7 @@ static int crsdump_test1(fwts_framework *fw)
 				else
 					crsdump_small_resource_items(fw, name, data, obj->Buffer.Length);
 			}
-
-			if (buf.Length && buf.Pointer)
-				free(buf.Pointer);
+			free(buf.Pointer);
 		}
 	}
 	return FWTS_OK;
diff --git a/src/acpi/method/method.c b/src/acpi/method/method.c
index 906ebc0..6afa864 100644
--- a/src/acpi/method/method.c
+++ b/src/acpi/method/method.c
@@ -439,8 +439,7 @@ static void method_evaluate_found_method(
 			check_func(fw, name, &buf, obj, private);
 		}
 	}
-	if (buf.Length && buf.Pointer)
-		free(buf.Pointer);
+	free(buf.Pointer);
 
 	fwts_acpica_sem_count_get(&sem_acquired, &sem_released);
 	if (sem_acquired != sem_released) {
diff --git a/src/acpi/pcc/pcc.c b/src/acpi/pcc/pcc.c
index 36eb654..4e4dd8a 100644
--- a/src/acpi/pcc/pcc.c
+++ b/src/acpi/pcc/pcc.c
@@ -433,9 +433,7 @@ static int pcc_test1(fwts_framework *fw)
 				if (ACPI_FAILURE(ret) == AE_OK) {
 					pcc_check_buffer(fw, pcc_name, &buf);
 					count++;
-
-					if (buf.Length && buf.Pointer)
-	        				free(buf.Pointer);
+	        			free(buf.Pointer);
 				}
 			}
 		}
diff --git a/src/acpi/wmi/wmi.c b/src/acpi/wmi/wmi.c
index 2425be1..510e9d0 100644
--- a/src/acpi/wmi/wmi.c
+++ b/src/acpi/wmi/wmi.c
@@ -380,9 +380,7 @@ static int wmi_test1(fwts_framework *fw)
 					(uint8_t*)obj->Buffer.Pointer);
 				wdg_found = true;
 			}
-
-			if (buf.Length && buf.Pointer)
-				free(buf.Pointer);
+			free(buf.Pointer);
 		}
 	}
 
diff --git a/src/hotkey/hotkey/hotkey.c b/src/hotkey/hotkey/hotkey.c
index 381f029..70fcc78 100644
--- a/src/hotkey/hotkey/hotkey.c
+++ b/src/hotkey/hotkey/hotkey.c
@@ -231,10 +231,8 @@ static int hotkey_deinit(fwts_framework *fw)
 	FWTS_UNUSED(fw);
 
 	fwts_keymap_free(hotkeys);
-	if (hotkey_dev)
-		free(hotkey_dev);
-	if (hotkey_keymap)
-		free(hotkey_keymap);
+	free(hotkey_dev);
+	free(hotkey_keymap);
 	return FWTS_OK;
 }
 
diff --git a/src/lib/src/fwts_framework.c b/src/lib/src/fwts_framework.c
index ff1e3b3..8042cf9 100644
--- a/src/lib/src/fwts_framework.c
+++ b/src/lib/src/fwts_framework.c
@@ -788,8 +788,7 @@ static void fwts_framework_strdup(char **ptr, const char *str)
 	if (ptr == NULL)
 		return;
 
-	if (*ptr)
-		free(*ptr);
+	free(*ptr);
 	*ptr = strdup(str);
 }
 
diff --git a/src/lib/src/fwts_klog.c b/src/lib/src/fwts_klog.c
index cd247e2..0472136 100644
--- a/src/lib/src/fwts_klog.c
+++ b/src/lib/src/fwts_klog.c
@@ -469,8 +469,7 @@ static void fwts_klog_regex_find_callback(fwts_framework *fw, char *line, int re
 			return;
 
 		rc = pcre_exec(re, extra, line, strlen(line), 0, 0, vector, 1);
-		if (extra)
-			free(extra);
+		free(extra);
 		pcre_free(re);
 		if (rc == 0)
 			(*match)++;
diff --git a/src/lib/src/fwts_log.c b/src/lib/src/fwts_log.c
index b4a36ec..d8b7cc9 100644
--- a/src/lib/src/fwts_log.c
+++ b/src/lib/src/fwts_log.c
@@ -482,8 +482,7 @@ int fwts_log_set_owner(fwts_log *log, const char *owner)
 	if (log && (log->magic == LOG_MAGIC)) {
 		char *newowner = strdup(owner);
 		if (newowner) {
-			if (log->owner)
-				free(log->owner);
+			free(log->owner);
 			log->owner = newowner;
 			return FWTS_OK;
 		}
@@ -580,8 +579,7 @@ char *fwts_log_get_filenames(const char *filename, const fwts_log_type type)
 		fwts_log_type mask = 1 << i;
 		if (type & mask) {
 			if ((tmp = fwts_log_filename(filename, mask)) == NULL) {
-				if (filenames)
-					free(filenames);
+				free(filenames);
 				return NULL;
 			}
 
@@ -726,9 +724,7 @@ int fwts_log_close(fwts_log *log)
 		/* ..and free log files */
 		fwts_list_free_items(&log->log_files, free);
 
-		if (log->owner)
-			free(log->owner);
-
+		free(log->owner);
 		free(log);
 	}
 	return FWTS_OK;
-- 
1.8.3.1