From 1f6f42e63ae50237f22963733d71a208f9e36599 Mon Sep 17 00:00:00 2001 From: Mathieu Desnoyers Date: Sat, 11 Mar 2017 08:43:09 -0500 Subject: [PATCH] Use SIZE_MAX instead of -1ULL for size_t parameter strutils_star_glob_match() receives a size_t. Passing -1ULL truncates the value implicitly on systems where size_t is 32-bit. It is cleaner to use SIZE_T. Signed-off-by: Mathieu Desnoyers --- liblttng-ust/lttng-events.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/liblttng-ust/lttng-events.c b/liblttng-ust/lttng-events.c index fa54a025..bfdae4ff 100644 --- a/liblttng-ust/lttng-events.c +++ b/liblttng-ust/lttng-events.c @@ -635,8 +635,8 @@ int lttng_desc_match_star_glob_enabler(const struct lttng_event_desc *desc, unsigned int has_loglevel = 0; assert(enabler->type == LTTNG_ENABLER_STAR_GLOB); - if (!strutils_star_glob_match(enabler->event_param.name, -1ULL, - desc->name, -1ULL)) + if (!strutils_star_glob_match(enabler->event_param.name, SIZE_MAX, + desc->name, SIZE_MAX)) return 0; if (desc->loglevel) { loglevel = *(*desc->loglevel); @@ -699,7 +699,7 @@ int lttng_desc_match_enabler(const struct lttng_event_desc *desc, excluder_name = (char *) (excluder->excluder.names) + count * LTTNG_UST_SYM_NAME_LEN; len = strnlen(excluder_name, LTTNG_UST_SYM_NAME_LEN); - if (len > 0 && strutils_star_glob_match(excluder_name, len, desc->name, -1ULL)) + if (len > 0 && strutils_star_glob_match(excluder_name, len, desc->name, SIZE_MAX)) return 0; } } -- 2.34.1