From: kernel test robot <lkp@intel.com>
To: kbuild@lists.01.org
Subject: Re: [PATCH 1/3] kunit: add kunit.filter_glob cmdline option to filter suites
Date: Thu, 04 Feb 2021 09:48:24 +0800 [thread overview]
Message-ID: <202102040936.AzT5TU0m-lkp@intel.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 2818 bytes --]
CC: kbuild-all(a)lists.01.org
In-Reply-To: <20210203234116.839819-2-dlatypov@google.com>
References: <20210203234116.839819-2-dlatypov@google.com>
TO: Daniel Latypov <dlatypov@google.com>
TO: brendanhiggins(a)google.com
TO: davidgow(a)google.com
CC: linux-kernel(a)vger.kernel.org
CC: linux-kselftest(a)vger.kernel.org
CC: skhan(a)linuxfoundation.org
CC: Daniel Latypov <dlatypov@google.com>
Hi Daniel,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on 88bb507a74ea7d75fa49edd421eaa710a7d80598]
url: https://github.com/0day-ci/linux/commits/Daniel-Latypov/kunit-support-running-subsets-of-test-suites-from/20210204-074405
base: 88bb507a74ea7d75fa49edd421eaa710a7d80598
:::::: branch date: 2 hours ago
:::::: commit date: 2 hours ago
config: x86_64-randconfig-m001-20210202 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
smatch warnings:
lib/kunit/executor.c:110 kunit_run_all_tests() error: double free of 'suite_set.start'
vim +110 lib/kunit/executor.c
45dcbb6f5ef78b Brendan Higgins 2020-08-04 95
8c0d884986ba22 Brendan Higgins 2020-08-04 96 int kunit_run_all_tests(void)
aac35468ca20a3 Alan Maguire 2020-08-04 97 {
aac35468ca20a3 Alan Maguire 2020-08-04 98 struct kunit_suite * const * const *suites;
aac35468ca20a3 Alan Maguire 2020-08-04 99
d5554dd78a454b Daniel Latypov 2021-02-03 100 struct suite_set suite_set = kunit_filter_suites();
45dcbb6f5ef78b Brendan Higgins 2020-08-04 101
d5554dd78a454b Daniel Latypov 2021-02-03 102 kunit_print_tap_header(&suite_set);
d5554dd78a454b Daniel Latypov 2021-02-03 103
d5554dd78a454b Daniel Latypov 2021-02-03 104 for (suites = suite_set.start; suites < suite_set.end; suites++)
aac35468ca20a3 Alan Maguire 2020-08-04 105 __kunit_test_suites_init(*suites);
aac35468ca20a3 Alan Maguire 2020-08-04 106
d5554dd78a454b Daniel Latypov 2021-02-03 107 if (filter_glob) { /* a copy was made of each array */
d5554dd78a454b Daniel Latypov 2021-02-03 108 for (suites = suite_set.start; suites < suite_set.end; suites++)
d5554dd78a454b Daniel Latypov 2021-02-03 109 kfree(suites);
d5554dd78a454b Daniel Latypov 2021-02-03 @110 kfree(suite_set.start);
d5554dd78a454b Daniel Latypov 2021-02-03 111 }
d5554dd78a454b Daniel Latypov 2021-02-03 112
aac35468ca20a3 Alan Maguire 2020-08-04 113 return 0;
aac35468ca20a3 Alan Maguire 2020-08-04 114 }
aac35468ca20a3 Alan Maguire 2020-08-04 115
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 33996 bytes --]
next reply other threads:[~2021-02-04 1:48 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-02-04 1:48 kernel test robot [this message]
-- strict thread matches above, loose matches on Subject: below --
2021-02-03 23:41 [PATCH 0/3] kunit: support running subsets of test suites from Daniel Latypov
2021-02-03 23:41 ` [PATCH 1/3] kunit: add kunit.filter_glob cmdline option to filter suites Daniel Latypov
2021-02-04 7:11 ` Dan Carpenter
2021-02-04 7:11 ` Dan Carpenter
2021-02-04 7:11 ` Dan Carpenter
2021-02-04 17:30 ` Daniel Latypov
2021-02-04 17:30 ` Daniel Latypov
2021-02-08 11:12 ` Dan Carpenter
2021-02-08 11:12 ` Dan Carpenter
2021-02-08 11:12 ` Dan Carpenter
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202102040936.AzT5TU0m-lkp@intel.com \
--to=lkp@intel.com \
--cc=kbuild@lists.01.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.