summaryrefslogtreecommitdiff
path: root/src/test/test-fileio.c
diff options
context:
space:
mode:
authorLennart Poettering <lennart@poettering.net>2021-09-16 11:31:52 +0200
committerLennart Poettering <lennart@poettering.net>2021-09-16 12:23:32 +0200
commitf3b751220bb842ce22a4f607d817f481f0961d40 (patch)
tree6019b26abad176cb84765d1acdff5bed5d310e00 /src/test/test-fileio.c
parent00bd9a4a82ed57bc0c7f158da4564fc1eab808b4 (diff)
downloadsystemd-f3b751220bb842ce22a4f607d817f481f0961d40.tar.gz
test-fileio: test read_virtual_file() with more files from /proc
i.e. let's pick some files we know are too large, or where struct stat's .st_size is zero even though non-empty, and test read_virtual_file() with that, to ensure things are handled sensibly. Goal is to ensure all three major codepaths in read_virtual_file() are tested. Prompted-by: #20743
Diffstat (limited to 'src/test/test-fileio.c')
-rw-r--r--src/test/test-fileio.c12
1 files changed, 10 insertions, 2 deletions
diff --git a/src/test/test-fileio.c b/src/test/test-fileio.c
index 2b6ef495aa..fa8a66df04 100644
--- a/src/test/test-fileio.c
+++ b/src/test/test-fileio.c
@@ -1047,7 +1047,11 @@ static void test_read_virtual_file(size_t max_size) {
FOREACH_STRING(filename,
"/proc/1/cmdline",
"/etc/nsswitch.conf",
- "/sys/kernel/uevent_seqnum") {
+ "/sys/kernel/uevent_seqnum",
+ "/proc/kcore",
+ "/proc/kallsyms",
+ "/proc/self/exe",
+ "/proc/self/pagemap") {
_cleanup_free_ char *buf = NULL;
size_t size = 0;
@@ -1055,7 +1059,11 @@ static void test_read_virtual_file(size_t max_size) {
r = read_virtual_file(filename, max_size, &buf, &size);
if (r < 0) {
log_info_errno(r, "read_virtual_file(\"%s\", %zu): %m", filename, max_size);
- assert_se(ERRNO_IS_PRIVILEGE(r) || r == -ENOENT);
+ assert_se(ERRNO_IS_PRIVILEGE(r) || /* /proc/kcore is not accessible to unpriv */
+ IN_SET(r,
+ -ENOENT, /* Some of the files might be absent */
+ -EINVAL, /* too small reads from /proc/self/pagemap trigger EINVAL */
+ -EFBIG)); /* /proc/kcore and /proc/self/pagemap should be too large */
} else
log_info("read_virtual_file(\"%s\", %zu): %s (%zu bytes)", filename, max_size, r ? "non-truncated" : "truncated", size);
}