diff options
| author |   Kees Cook <keescook@chromium.org> | 2018-02-14 15:45:07 -0800 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2018-02-16 15:16:05 -0500 |
| commit | fe9c842695e26d8116b61b80bfb905356f07834b (patch) | |
| tree | 4290060a267efbab9953477c6e8a0e5751be97e2 /net/nfc/netlink.c | |
| parent | tls: getsockopt return record sequence number (diff) | |
| download | linux-dev-fe9c842695e26d8116b61b80bfb905356f07834b.tar.xz linux-dev-fe9c842695e26d8116b61b80bfb905356f07834b.zip | |
NFC: llcp: Limit size of SDP URI
The tlv_len is u8, so we need to limit the size of the SDP URI. Enforce
this both in the NLA policy and in the code that performs the allocation
and copy, to avoid writing past the end of the allocated buffer.
Fixes: d9b8d8e19b073 ("NFC: llcp: Service Name Lookup netlink interface")
Signed-off-by: Kees Cook <keescook@chromium.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to '')
| -rw-r--r-- | net/nfc/netlink.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/nfc/netlink.c b/net/nfc/netlink.c index c0b83dc9d993..f018eafc2a0d 100644 --- a/net/nfc/netlink.c +++ b/net/nfc/netlink.c @@ -61,7 +61,8 @@ static const struct nla_policy nfc_genl_policy[NFC_ATTR_MAX + 1] = { }; static const struct nla_policy nfc_sdp_genl_policy[NFC_SDP_ATTR_MAX + 1] = { - [NFC_SDP_ATTR_URI] = { .type = NLA_STRING }, + [NFC_SDP_ATTR_URI] = { .type = NLA_STRING, + .len = U8_MAX - 4 }, [NFC_SDP_ATTR_SAP] = { .type = NLA_U8 }, }; |