diff --git a/bugfix-ui-ui.c-always-use-s-style-format-for-printf-style-f.patch b/bugfix-ui-ui.c-always-use-s-style-format-for-printf-style-f.patch new file mode 100644 index 0000000..305c42d --- /dev/null +++ b/bugfix-ui-ui.c-always-use-s-style-format-for-printf-style-f.patch @@ -0,0 +1,80 @@ +From f8bdd0e64284d841544fd3ebe22f4652902ba8d2 Mon Sep 17 00:00:00 2001 +From: Sergei Trofimovich +Date: Tue, 9 Nov 2021 22:24:17 +0000 +Subject: [PATCH] ui/ui.c: always use "%s"-style format for printf()-style + functions + +`ncuses-6.3` added printf-style function attributes and now makes +it easier to catch cases when user input is used in palce of format +string when built with CFLAGS=-Werror=format-security: + + ui/ui.c:714:16: error: format not a string literal and no format arguments [-Werror=format-security] + 714 | printw(copy_to); + | ^~~~~~~ + +Let's wrap all the missing places with "%s" format. +--- + ui/ui.c | 12 ++++++------ + 1 file changed, 6 insertions(+), 6 deletions(-) + +diff --git a/ui/ui.c b/ui/ui.c +index 1be8f95..6ff3305 100644 +--- a/ui/ui.c ++++ b/ui/ui.c +@@ -26,7 +26,7 @@ void show_frame() + while(strlen(top) != (size_t)COLS - 1) { + snprintf(top + strlen(top), COLS - strlen(top), " "); + } +- mvprintw(0, 0, top); ++ mvprintw(0, 0, "%s", top); + for(i = 0; i < LINES; i++) { + mvprintw(i, 0, " "); + mvprintw(i, COLS - 1, " "); +@@ -42,7 +42,7 @@ void show_footer() + snprintf(footer + strlen(footer), COLS - strlen(footer), " "); + } + attrset(COLOR_PAIR(4)); +- mvprintw(LINES - 1, 0, footer); ++ mvprintw(LINES - 1, 0, "%s", footer); + } + + char * check_control_in_sleep_input(int max_len, int column_offest, int line_offset) +@@ -331,7 +331,7 @@ void print_assigned_objects_string(irq_t *irq, int *line_offset) + char assigned_to[128] = "\0"; + for_each_int(irq->assigned_to, copy_assigned_obj, assigned_to); + assigned_to[strlen(assigned_to) - 2] = '\0'; +- mvprintw(*line_offset, 36, assigned_to); ++ mvprintw(*line_offset, 36, "%s", assigned_to); + } + + void print_irq_line(irq_t *irq, void *data) +@@ -566,7 +566,7 @@ void settings() + uint8_t sleep_input_offset = strlen(info) + 3; + snprintf(info + strlen(info), 128 - strlen(info), "%" PRIu64 "\n", setup.sleep); + attrset(COLOR_PAIR(1)); +- mvprintw(2, 3, info); ++ mvprintw(2, 3, "%s", info); + print_all_cpus(); + + int user_input = 1; +@@ -664,7 +664,7 @@ void display_tree_node_irqs(irq_t *irq, void *data) + char indent[32] = " \0"; + snprintf(indent + strlen(indent), 32 - strlen(indent), "%s", (char *)data); + attrset(COLOR_PAIR(3)); +- printw("%sIRQ %lu, IRQs since last rebalance %lu\n", ++ printw("%sIRQ %u, IRQs since last rebalance %lu\n", + indent, irq->vector, irq->diff); + } + +@@ -711,7 +711,7 @@ void display_tree_node(cpu_node_t *node, void *data) + default: + break; + } +- printw(copy_to); ++ printw("%s", copy_to); + if(g_list_length(node->irqs) > 0) { + for_each_irq(node->irqs, display_tree_node_irqs, indent); + } +-- +2.23.0 + diff --git a/irqbalance.spec b/irqbalance.spec index 3cec42b..70704df 100644 --- a/irqbalance.spec +++ b/irqbalance.spec @@ -1,7 +1,7 @@ Summary: A dynamic adaptive IRQ balancing daemon Name: irqbalance Version: 1.8.0 -Release: 4 +Release: 5 Epoch: 3 License: GPLv2 Source0: https://github.com/Irqbalance/irqbalance/archive/v%{version}.tar.gz#/irqbalance-%{version}.tar.gz @@ -24,6 +24,7 @@ Requires: numactl-libs %define _hardened_build 1 Patch6000: bugfix-fix-unsigned-integer-subtraction-sign-overflow.patch +Patch6001: bugfix-ui-ui.c-always-use-s-style-format-for-printf-style-f.patch Patch9000: feature-aarch64-add-the-regular-to-get-the-correct-i.patch Patch9001: feature-add-new-user-irq-policy-config-rule.patch @@ -122,6 +123,12 @@ fi /sbin/chkconfig --del %{name} >/dev/null 2>&1 || : %changelog +* Tue Apr 12 2021 Liu Chao - 3:1.8.0-5 +- Type:bugfix +- ID:NA +- SUG:restart +- DESC: fix irqbalance-ui build error when ncuses upgrades to 6.3 + * Mon Nov 1 2021 Liu Chao - 3:1.8.0-4 - Type:bugfix - ID:NA