diff options
| author | Antenore Gatta <antenore@simbiosi.org> | 2018-02-12 11:37:55 +0300 |
|---|---|---|
| committer | Antenore Gatta <antenore@simbiosi.org> | 2018-02-12 11:37:55 +0300 |
| commit | 863e82c3257865e32944c73acfbb7ac9ef507c4a (patch) | |
| tree | 568803e43ac6f4a059a5f13f36d9160cbecf8f07 | |
| parent | 89a997f7f7d7dfc65d9962cbe6e4d9b2c16a2025 (diff) | |
Bad free for kernel values
| -rw-r--r-- | remmina/src/remmina_stats.c | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/remmina/src/remmina_stats.c b/remmina/src/remmina_stats.c index fb24670a0..57eca962e 100644 --- a/remmina/src/remmina_stats.c +++ b/remmina/src/remmina_stats.c @@ -171,7 +171,9 @@ JsonNode *remmina_stats_get_os_info() kernel_name = "n/a"; } json_builder_add_string_value(b, kernel_name); - g_free(kernel_name); + if (!kernel_name || kernel_name[0] == '\0') { + g_free(kernel_name); + } json_builder_set_member_name(b, "kernel_release"); kernel_release = g_strdup_printf("%s", remmina_utils_get_kernel_release()); @@ -181,7 +183,9 @@ JsonNode *remmina_stats_get_os_info() kernel_release = "n/a"; } json_builder_add_string_value(b, kernel_release); - g_free(kernel_release); + if (!kernel_release || kernel_release[0] == '\0') { + g_free(kernel_release); + } json_builder_set_member_name(b, "kernel_arch"); kernel_arch = g_strdup_printf("%s", remmina_utils_get_kernel_arch()); @@ -191,7 +195,9 @@ JsonNode *remmina_stats_get_os_info() kernel_arch = "n/a"; } json_builder_add_string_value(b, kernel_arch); - g_free(kernel_arch); + if (!kernel_arch || kernel_arch[0] == '\0') { + g_free(kernel_arch); + } id = remmina_utils_get_lsb_id(); if (!id || id[0] == '\0') { |