[LTP] [COMMITTED] [PATCH] tst_test: Print tags only when defined.
Cyril Hrubis
chrubis@suse.cz
Mon Mar 30 16:21:42 CEST 2020
We were printing the Tags heading even when no tags were defined.
Signed-off-by: Cyril Hrubis <chrubis@suse.cz>
---
lib/tst_test.c | 19 ++++++++++---------
1 file changed, 10 insertions(+), 9 deletions(-)
diff --git a/lib/tst_test.c b/lib/tst_test.c
index dae3fa1b5..e502c2c1a 100644
--- a/lib/tst_test.c
+++ b/lib/tst_test.c
@@ -447,18 +447,19 @@ static void print_test_tags(void)
unsigned int i;
const struct tst_tag *tags = tst_test->tags;
+ if (!tags)
+ return;
+
printf("\nTags\n");
printf("----\n");
- if (tags) {
- for (i = 0; tags[i].name; i++) {
- if (!strcmp(tags[i].name, "CVE"))
- printf(CVE_DB_URL "%s\n", tags[i].value);
- else if (!strcmp(tags[i].name, "linux-git"))
- printf(LINUX_GIT_URL "%s\n", tags[i].value);
- else
- printf("%s: %s\n", tags[i].name, tags[i].value);
- }
+ for (i = 0; tags[i].name; i++) {
+ if (!strcmp(tags[i].name, "CVE"))
+ printf(CVE_DB_URL "%s\n", tags[i].value);
+ else if (!strcmp(tags[i].name, "linux-git"))
+ printf(LINUX_GIT_URL "%s\n", tags[i].value);
+ else
+ printf("%s: %s\n", tags[i].name, tags[i].value);
}
printf("\n");
--
2.24.1
More information about the ltp
mailing list