From 9e1f208795b62ad4825aaafd5b740c979bb67048 Mon Sep 17 00:00:00 2001 From: Rafael Antognolli Date: Tue, 12 Jun 2018 12:18:19 -0700 Subject: [PATCH] intel/aubinator: Use int to store getopt_long flags. getopt_long flag parameter is an int pointer, so if we use bool to store those values, when getopt_long writes to one of them, it might end up overwriting the next one. Reviewed-by: Ian Romanick --- src/intel/tools/aubinator.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/intel/tools/aubinator.c b/src/intel/tools/aubinator.c index 3120e82b22e..c8d79ae377d 100644 --- a/src/intel/tools/aubinator.c +++ b/src/intel/tools/aubinator.c @@ -56,8 +56,8 @@ /* options */ -static bool option_full_decode = true; -static bool option_print_offsets = true; +static int option_full_decode = true; +static int option_print_offsets = true; static int max_vbo_lines = -1; static enum { COLOR_AUTO, COLOR_ALWAYS, COLOR_NEVER } option_color;