From 712a59d4612567952a00035ed90ea98344fdd723 Mon Sep 17 00:00:00 2001 From: Dan Carpenter Date: Wed, 16 Dec 2020 16:22:39 +0300 Subject: [PATCH] param_set: don't record so much data in the DB This was printing a lot of duplicated information. If "foo->bar" is modified then we know "foo->bar->baz" is modified. No need to record that. Anyway on this new code path the check was missed. Signed-off-by: Dan Carpenter --- smatch_param_set.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/smatch_param_set.c b/smatch_param_set.c index efab01f5..ebcf0381 100644 --- a/smatch_param_set.c +++ b/smatch_param_set.c @@ -257,7 +257,9 @@ static void print_return_value_param_helper(int return_id, char *return_ranges, if (new) { if (strcmp(new, two_dot) == 0) continue; + strncpy(two_dot, new, sizeof(two_dot)); + insert_string(&set_list, (char *)sm->name); sql_insert_return_states(return_id, return_ranges, PARAM_SET, param, new, "s64min-s64max"); continue; -- 2.11.4.GIT