diff options
author | ngie <ngie@FreeBSD.org> | 2016-06-10 18:02:51 +0000 |
---|---|---|
committer | ngie <ngie@FreeBSD.org> | 2016-06-10 18:02:51 +0000 |
commit | fea5a69f20857ffb5d29a2124b16560e58384078 (patch) | |
tree | c41800b7a8ebafbf4f266d7c0bb731d71ca72683 /contrib/diff/src/cmp.c | |
parent | ea6cfdfcf90bbd6d13e10ff2f6e33eb24b865a01 (diff) | |
download | FreeBSD-src-fea5a69f20857ffb5d29a2124b16560e58384078.zip FreeBSD-src-fea5a69f20857ffb5d29a2124b16560e58384078.tar.gz |
MFC r299507:
r299507 (by cem):
rtadvd(8): Fix a typo in full msg receive logic
Check against the size of the struct, not the pointer. Previously, a message
with a cm_len between 9 and 23 (inclusive) could cause int msglen to underflow
and read(2) to be invoked with msglen size (implicitly cast to signed),
overrunning the caller-provided buffer.
All users of cm_recv() supply a stack buffer.
On the other hand, the rtadvd control socket appears to only be writable by the
owner, who is probably root.
While here, correct some types to be size_t or ssize_t.
CID: 1008477
Security: unix socket remotes may overflow stack in rtadvd
Diffstat (limited to 'contrib/diff/src/cmp.c')
0 files changed, 0 insertions, 0 deletions