summaryrefslogtreecommitdiff
path: root/plugins/hddtemp
diff options
context:
space:
mode:
authorinfirit <[email protected]>2015-07-03 18:41:53 +0200
committerinfirit <[email protected]>2015-07-07 15:36:36 +0200
commite688080690005e5508db95d5f19c351c457302de (patch)
tree19c25832c942a75a05da44255a19829bf2d70b5b /plugins/hddtemp
parenta9fe9c4239c98f08a8815985cbcc485d729b0c2c (diff)
downloadmate-sensors-applet-e688080690005e5508db95d5f19c351c457302de.tar.bz2
mate-sensors-applet-e688080690005e5508db95d5f19c351c457302de.tar.xz
hddtemp: use g_strcmp0 instead of strcmp
Passing NULL to strcmp could be unsafe or at the minimum result in undefined behaviour. g_strcmp0 handles NULL gracefully and returns 0.
Diffstat (limited to 'plugins/hddtemp')
-rw-r--r--plugins/hddtemp/hddtemp-plugin.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/plugins/hddtemp/hddtemp-plugin.c b/plugins/hddtemp/hddtemp-plugin.c
index 220c830..f7d36f5 100644
--- a/plugins/hddtemp/hddtemp-plugin.c
+++ b/plugins/hddtemp/hddtemp-plugin.c
@@ -151,9 +151,9 @@ static void hddtemp_plugin_get_sensors(GList **sensors) {
pv = output_vector = g_strsplit(hddtemp_output, "|", -1);
while(pv[1] != NULL) {
- if (strcmp(pv[2], "") != 0 &&
- strcmp(pv[3], "") != 0 &&
- strcmp(pv[4], "") != 0 &&
+ if (g_strcmp0(pv[2], "") != 0 &&
+ g_strcmp0(pv[3], "") != 0 &&
+ g_strcmp0(pv[4], "") != 0 &&
(!(g_ascii_strcasecmp(pv[2], "???") == 0 ||
g_ascii_strcasecmp(pv[3], "ERR") == 0 ||
g_ascii_strcasecmp(pv[4], "*") == 0))) {