Merge pull request #194 from trofi/master

ui/ui.c: always use "%s"-style format for printf()-style functions
This commit is contained in:
Neil Horman 2021-11-10 07:26:06 -05:00 committed by GitHub
commit 7b1fc48549
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

12
ui/ui.c
View file

@ -26,7 +26,7 @@ void show_frame()
while(strlen(top) != (size_t)COLS - 1) { while(strlen(top) != (size_t)COLS - 1) {
snprintf(top + strlen(top), COLS - strlen(top), " "); snprintf(top + strlen(top), COLS - strlen(top), " ");
} }
mvprintw(0, 0, top); mvprintw(0, 0, "%s", top);
for(i = 0; i < LINES; i++) { for(i = 0; i < LINES; i++) {
mvprintw(i, 0, " "); mvprintw(i, 0, " ");
mvprintw(i, COLS - 1, " "); mvprintw(i, COLS - 1, " ");
@ -42,7 +42,7 @@ void show_footer()
snprintf(footer + strlen(footer), COLS - strlen(footer), " "); snprintf(footer + strlen(footer), COLS - strlen(footer), " ");
} }
attrset(COLOR_PAIR(4)); 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) 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"; char assigned_to[128] = "\0";
for_each_int(irq->assigned_to, copy_assigned_obj, assigned_to); for_each_int(irq->assigned_to, copy_assigned_obj, assigned_to);
assigned_to[strlen(assigned_to) - 2] = '\0'; 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) void print_irq_line(irq_t *irq, void *data)
@ -566,7 +566,7 @@ void settings()
uint8_t sleep_input_offset = strlen(info) + 3; uint8_t sleep_input_offset = strlen(info) + 3;
snprintf(info + strlen(info), 128 - strlen(info), "%" PRIu64 "\n", setup.sleep); snprintf(info + strlen(info), 128 - strlen(info), "%" PRIu64 "\n", setup.sleep);
attrset(COLOR_PAIR(1)); attrset(COLOR_PAIR(1));
mvprintw(2, 3, info); mvprintw(2, 3, "%s", info);
print_all_cpus(); print_all_cpus();
int user_input = 1; int user_input = 1;
@ -664,7 +664,7 @@ void display_tree_node_irqs(irq_t *irq, void *data)
char indent[32] = " \0"; char indent[32] = " \0";
snprintf(indent + strlen(indent), 32 - strlen(indent), "%s", (char *)data); snprintf(indent + strlen(indent), 32 - strlen(indent), "%s", (char *)data);
attrset(COLOR_PAIR(3)); 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); indent, irq->vector, irq->diff);
} }
@ -711,7 +711,7 @@ void display_tree_node(cpu_node_t *node, void *data)
default: default:
break; break;
} }
printw(copy_to); printw("%s", copy_to);
if(g_list_length(node->irqs) > 0) { if(g_list_length(node->irqs) > 0) {
for_each_irq(node->irqs, display_tree_node_irqs, indent); for_each_irq(node->irqs, display_tree_node_irqs, indent);
} }