summaryrefslogtreecommitdiffstats
path: root/lib
diff options
context:
space:
mode:
authorI Hsin Cheng <richard120310@gmail.com>2024-09-10 12:35:31 +0800
committerAndrew Morton <akpm@linux-foundation.org>2024-09-17 01:11:20 -0700
commite620799c414a035dea1208bcb51c869744931dbb (patch)
tree3d3b736f0ce61f080c13c977afdd3e02c37e3b11 /lib
parent13309764720624caf2c5afba99b198f3f9fcd9f0 (diff)
downloadlinux-stable-e620799c414a035dea1208bcb51c869744931dbb.tar.gz
linux-stable-e620799c414a035dea1208bcb51c869744931dbb.tar.bz2
linux-stable-e620799c414a035dea1208bcb51c869744931dbb.zip
list: test: fix tests for list_cut_position()
Fix test for list_cut_position*() for the missing check of integer "i" after the second loop. The variable should be checked for second time to make sure both lists after the cut operation are formed as expected. Link: https://lkml.kernel.org/r/20240910043531.71343-1-richard120310@gmail.com Signed-off-by: I Hsin Cheng <richard120310@gmail.com> Cc: David Gow <davidgow@google.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Diffstat (limited to 'lib')
-rw-r--r--lib/list-test.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/lib/list-test.c b/lib/list-test.c
index 37cbc33e9fdb..8d1d47a9fe9e 100644
--- a/lib/list-test.c
+++ b/lib/list-test.c
@@ -404,10 +404,13 @@ static void list_test_list_cut_position(struct kunit *test)
KUNIT_EXPECT_EQ(test, i, 2);
+ i = 0;
list_for_each(cur, &list1) {
KUNIT_EXPECT_PTR_EQ(test, cur, &entries[i]);
i++;
}
+
+ KUNIT_EXPECT_EQ(test, i, 1);
}
static void list_test_list_cut_before(struct kunit *test)
@@ -432,10 +435,13 @@ static void list_test_list_cut_before(struct kunit *test)
KUNIT_EXPECT_EQ(test, i, 1);
+ i = 0;
list_for_each(cur, &list1) {
KUNIT_EXPECT_PTR_EQ(test, cur, &entries[i]);
i++;
}
+
+ KUNIT_EXPECT_EQ(test, i, 2);
}
static void list_test_list_splice(struct kunit *test)