From 0262d3ff9fadbc6300e4146101539b4f07fcf4b8 Mon Sep 17 00:00:00 2001 From: Nico Golde Date: Fri, 21 Sep 2012 17:44:58 +0200 Subject: logging.c: Do not crash on empty category name log_parse_category_mask(), skip log category name right away if name is NULL to prevent passing a NULL ptr to strlen. --- src/logging.c | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/src/logging.c b/src/logging.c index 0816570e..f58265f7 100644 --- a/src/logging.c +++ b/src/logging.c @@ -175,17 +175,19 @@ void log_parse_category_mask(struct log_target* target, const char *_mask) category_token = strtok(mask, ":"); do { for (i = 0; i < osmo_log_info->num_cat; ++i) { + size_t length, cat_length; char* colon = strstr(category_token, ","); - int length = strlen(category_token); - int cat_length = strlen(osmo_log_info->cat[i].name); + + if (!osmo_log_info->cat[i].name) + continue; + + length = strlen(category_token); + cat_length = strlen(osmo_log_info->cat[i].name); /* Use longest length not to match subocurrences. */ if (cat_length > length) length = cat_length; - if (!osmo_log_info->cat[i].name) - continue; - if (colon) length = colon - category_token; -- cgit v1.2.3