diff options
author | Kyle Evans <kevans@FreeBSD.org> | 2020-10-01 19:56:38 +0000 |
---|---|---|
committer | Kyle Evans <kevans@FreeBSD.org> | 2020-10-01 19:56:38 +0000 |
commit | 320ef1bef081b626c2e230740d85e72313fbefca (patch) | |
tree | 5235659e2c80cc8ab79f31dfb1bd267fc00db112 /sys/kern/vfs_bio.c | |
parent | 2398cd110371c495ae71d9a6ece6ed761ccd38a5 (diff) | |
download | src-320ef1bef081b626c2e230740d85e72313fbefca.tar.gz src-320ef1bef081b626c2e230740d85e72313fbefca.zip |
auxv: partially revert r366207, cast buflen to unsigned int as needed
The warning generated pre-r366207 is actually a sign comparison warning:
error: comparison of integers of different signs: 'unsigned long' and 'int'
if (strlcpy(buf, execpath, buflen) >= buflen)
Revert parts that affected other lines and just cast this to unsigned int.
The buflen < 0 -> EINVAL has been kept despite no longer serving any
purposes w.r.t. sign-extension because I do believe it's the right thing to
do: "The provided buffer was not the right size for the requested item."
The original warning is confirmed to still be gone with an:
env WARNS=6 make WITHOUT_TESTS=yes.
Reviewed by: asomers, kib
X-MFC-With: r366207
Differential Revision: https://reviews.freebsd.org/D26631
Notes
Notes:
svn path=/head/; revision=366342
Diffstat (limited to 'sys/kern/vfs_bio.c')
0 files changed, 0 insertions, 0 deletions