diff options
| author | Burt P <pburt0@gmail.com> | 2018-02-21 21:45:23 -0600 |
|---|---|---|
| committer | Leandro A. F. Pereira <leandro@hardinfo.org> | 2018-02-22 07:31:51 -0800 |
| commit | 6bc582266cda64d5870cca3cc45fb0b95c5ee33a (patch) | |
| tree | 799f837eea876189c119e2205b78b48c8ef2f361 /shell | |
| parent | dd4f8f186170e0532aea3126a7c50e95eb5a8d84 (diff) | |
report.c: don't try to use column values that weren't provided (html version)
Like 96980fbf494d9c453c6b558967c90c55e41abe2d, but the html version
of that function. Fixes https://github.com/lpereira/hardinfo/issues/217
Signed-off-by: Burt P <pburt0@gmail.com>
Diffstat (limited to 'shell')
| -rw-r--r-- | shell/report.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/shell/report.c b/shell/report.c index e59b00d4..6abb03a0 100644 --- a/shell/report.c +++ b/shell/report.c @@ -300,7 +300,7 @@ report_html_key_value(ReportContext * ctx, gchar * key, gchar * value) { gint columns = report_get_visible_columns(ctx); gchar **values; - gint i; + gint i, mc; if (columns == 2) { ctx->output = h_strdup_cprintf("<tr><td class=\"field\">%s</td>" @@ -309,10 +309,11 @@ report_html_key_value(ReportContext * ctx, gchar * key, gchar * value) key, value); } else { values = g_strsplit(value, "|", columns); + mc = g_strv_length(values) - 1; ctx->output = h_strdup_cprintf("\n<tr>\n<td class=\"field\">%s</td>", ctx->output, key); - for (i = columns - 2; i >= 0; i--) { + for (i = mc; i >= 0; i--) { ctx->output = h_strdup_cprintf("<td class=\"value\">%s</td>", ctx->output, values[i]); |