diff options
| author | Al Viro <viro@zeniv.linux.org.uk> | 2020-07-27 19:22:20 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2020-07-27 13:25:39 -0700 |
| commit | 181964e619b76ae2e71bcdc6001cf977bec4cf6e (patch) | |
| tree | 84fc012351abcb4d314b7711b0bc2a69e4a046b7 /net/lapb | |
| parent | 11d268107ad4c7d694148269e3b23ef43dd0e8ba (diff) | |
fix a braino in cmsghdr_from_user_compat_to_kern()
commit 547ce4cfb34c ("switch cmsghdr_from_user_compat_to_kern() to
copy_from_user()") missed one of the places where ucmlen should've been
replaced with cmsg.cmsg_len, now that we are fetching the entire struct
rather than doing it field-by-field.
As the result, compat sendmsg() with several different-sized cmsg
attached started to fail with EINVAL. Trivial to fix, fortunately.
Fixes: 547ce4cfb34c ("switch cmsghdr_from_user_compat_to_kern() to copy_from_user()")
Reported-by: Nick Bowler <nbowler@draconx.ca>
Tested-by: Nick Bowler <nbowler@draconx.ca>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/lapb')
0 files changed, 0 insertions, 0 deletions