Page Menu
Home
FreeBSD
Search
Configure Global Search
Log In
Files
F149253588
D46051.1789768323.diff
No One
Temporary
Actions
View File
Edit File
Delete File
View Transforms
Subscribe
Flag For Later
Award Token
Size
2 KB
Referenced Files
None
Subscribers
None
D46051.1789768323.diff
View Options
diff --git a/lib/libc/tests/string/memccpy_test.c b/lib/libc/tests/string/memccpy_test.c
--- a/lib/libc/tests/string/memccpy_test.c
+++ b/lib/libc/tests/string/memccpy_test.c
@@ -1,6 +1,6 @@
/*-
* Copyright (c) 2009 David Schultz <das@FreeBSD.org>
- * Copyright (c) 2023 The FreeBSD Foundation
+ * Copyright (c) 2023, 2024 The FreeBSD Foundation
* All rights reserved.
*
* Portions of this software were developed by Robert Clausecker
@@ -54,34 +54,57 @@
buf = mmap(NULL, alloc_size, PROT_READ | PROT_WRITE, MAP_ANON, -1, 0);
assert(buf);
if (guard_at_end) {
- assert(munmap(buf + alloc_size - page_size, page_size) == 0);
+ assert(mprotect(buf + alloc_size - page_size, page_size, PROT_NONE) == 0);
return (buf + alloc_size - page_size - len);
} else {
- assert(munmap(buf, page_size) == 0);
+ assert(mprotect(buf, page_size, PROT_NONE) == 0);
return (buf + page_size);
}
}
static void
-test_memccpy(const char *s)
+freebuf(char * buf, size_t len, int guard_at_end)
+{
+ size_t alloc_size, page_size;
+
+ page_size = getpagesize();
+ alloc_size = roundup2(len, page_size) + page_size;
+
+ if (guard_at_end)
+ munmap(buf + len + page_size - alloc_size, alloc_size);
+ else
+ munmap(buf - page_size, alloc_size);
+}
+
+static void
+test_memccpy(const char *s, size_t size)
{
char *src, *dst, *expected;
- size_t size, bufsize, x;
+ size_t bufsize, x;
int i, j;
- size = strlen(s) + 1;
for (i = 0; i <= 1; i++) {
for (j = 0; j <= 1; j++) {
- for (bufsize = 0; bufsize <= size + 10; bufsize++) {
- src = makebuf(size, i);
- memcpy(src, s, size);
+ for (bufsize = 0; bufsize <= size + 32; bufsize++) {
dst = makebuf(bufsize, j);
+ if (bufsize < size) {
+ src = makebuf(bufsize, i);
+ memcpy(src, s, bufsize);
+ expected = NULL;
+ } else {
+ src = makebuf(size, i);
+ memcpy(src, s, size);
+ expected = dst + size;
+ }
+
memset(dst, 'X', bufsize);
- expected = bufsize >= size ? dst + size : NULL;
- assert(memccpy_fn(dst, src, src[size-1], bufsize) == expected);
- assert(bufsize == 0 || strncmp(src, dst, bufsize - 1) == 0);
+ assert(memccpy_fn(dst, src, s[size-1], bufsize) == expected);
+ assert(memcmp(src, dst, MIN(bufsize, size)) == 0);
for (x = size; x < bufsize; x++)
assert(dst[x] == 'X');
+
+ freebuf(dst, bufsize, j);
+ freebuf(src, bufsize < size ? bufsize : size, i);
}
}
}
@@ -168,7 +191,8 @@
for (i = 0; i < sizeof(buf) - 1; i++) {
buf[i] = ' ' + i;
- test_memccpy(buf);
+ buf[i+1] = '\0';
+ test_memccpy(buf, i + 1);
}
}
File Metadata
Details
Attached
Mime Type
text/plain
Expires
Fri, Sep 18, 9:52 PM (58 m, 15 s)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
29762871
Default Alt Text
D46051.1789768323.diff (2 KB)
Attached To
Mode
D46051: lib/libc/tests/string: improve memccpy "bounds" unit test
Attached
Detach File
Event Timeline
Log In to Comment