logger: fix off-by-one sscanf width in filter_parse_component_name

filter_parse_component_name() builds the sscanf format string with
field width UUID_NAME_MAX_LEN, but a %N[...] conversion writes up to
N characters plus a NUL terminator. comp_name is only
UUID_NAME_MAX_LEN bytes, so a component name of exactly that length
overflows the stack buffer by one byte.

Cap the scan width at UUID_NAME_MAX_LEN - 1 so the terminator always
fits in comp_name.

Signed-off-by: jmestwa-coder <jmestwa@gmail.com>

Author: jmestwa-coder

tools/logger/filter.c

@@ -104,7 +104,7 @@ static char *filter_parse_component_name(char *input_str, struct filter_element
 	 */
 	if (strlen(scan_format_string) == 0) {
 		ret = snprintf(scan_format_string, sizeof(scan_format_string),
-				"%%%d[^0-9* ]s", UUID_NAME_MAX_LEN);
+				"%%%d[^0-9* ]s", UUID_NAME_MAX_LEN - 1);
 		if (ret <= 0)
 			return NULL;
 	}