diff options
| author | Tavian Barnes <tavianator@tavianator.com> | 2024-01-03 17:17:35 -0500 |
|---|---|---|
| committer | Tavian Barnes <tavianator@tavianator.com> | 2024-01-04 14:42:36 -0500 |
| commit | 45fb1d952c3b262278a3b22e9c7d60cca19a5407 (patch) | |
| tree | 500f3e4d2288c427669c9fe62dafac64d6e53533 /tests/xtouch.c | |
| parent | d874481987c381fd572a6f3c43840c50111a9554 (diff) | |
| download | bfs-45fb1d952c3b262278a3b22e9c7d60cca19a5407.tar.xz | |
Work around DragonFly BSD kernel bug
DragonFly's x86_64 assembly implementation of copyinstr() checks the
wrong pointer when deciding whether to return EFAULT or ENAMETOOLONG,
causing it to always return EFAULT for overlong paths. Work around it
by treating EFAULT the same as ENAMETOOLONG on DragonFly.
Link: https://twitter.com/tavianator/status/1742991411203485713
Diffstat (limited to 'tests/xtouch.c')
| -rw-r--r-- | tests/xtouch.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/tests/xtouch.c b/tests/xtouch.c index 260a3a3..ed8bbee 100644 --- a/tests/xtouch.c +++ b/tests/xtouch.c @@ -70,13 +70,19 @@ static int open_parent(const struct args *args, const char **path) { switch (errno) { case ENAMETOOLONG: +#if __DragonFly__ + // https://twitter.com/tavianator/status/1742991411203485713 + case EFAULT: +#endif break; + case ENOENT: if (args->flags & CREATE_PARENTS) { break; } else { goto err; } + default: goto err; } |