diff options
| author | Phil Sutter <phil@nwl.cc> | 2022-08-31 13:00:52 +0200 |
|---|---|---|
| committer | Phil Sutter <phil@nwl.cc> | 2022-08-31 17:05:01 +0200 |
| commit | 96980c548d3a1aeb07ab6aaef45389efb058a69a (patch) | |
| tree | 81647e9ef472af9d1ace0942d659ff0b5aed6bc5 /include | |
| parent | c23f117ed4414aa848f273a6e77850471ce21e0e (diff) | |
local: Avoid sockaddr_un::sun_path buffer overflow
The array's size in struct sockaddr_un is only UNIX_PATH_MAX and
according to unix(7), it should hold a null-terminated string. So adjust
config reader to reject paths of length UNIX_PATH_MAX and above and
adjust the internal arrays to aid the compiler.
Fixes: f196de88cdd97 ("src: fix strncpy -Wstringop-truncation warnings")
Signed-off-by: Phil Sutter <phil@nwl.cc>
Diffstat (limited to 'include')
| -rw-r--r-- | include/local.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/local.h b/include/local.h index 9379446..22859d7 100644 --- a/include/local.h +++ b/include/local.h @@ -7,12 +7,12 @@ struct local_conf { int reuseaddr; - char path[UNIX_PATH_MAX + 1]; + char path[UNIX_PATH_MAX]; }; struct local_server { int fd; - char path[UNIX_PATH_MAX + 1]; + char path[UNIX_PATH_MAX]; }; /* callback return values */ |