• Suren Baghdasaryan's avatar
    NFC: Fix possible memory corruption when handling SHDLC I-Frame commands · 67084e26
    Suren Baghdasaryan authored
    commit 674d9de02aa7d521ebdf66c3958758bdd9c64e11 upstream.
    When handling SHDLC I-Frame commands "pipe" field used for indexing
    into an array should be checked before usage. If left unchecked it
    might access memory outside of the array of size NFC_HCI_MAX_PIPES(127).
    Malformed NFC HCI frames could be injected by a malicious NFC device
    communicating with the device being attacked (remote attack vector),
    or even by an attacker with physical access to the I2C bus such that
    they could influence the data transfers on that bus (local attack vector).
    skb->data is controlled by the attacker and has only been sanitized in
    the most trivial ways (CRC check), therefore we can consider the
    create_info struct and all of its members to tainted. 'create_info->pipe'
    with max value of 255 (uint8) is used to take an offset of the
    hdev->pipes array of 127 elements which can lead to OOB write.
    Cc: Samuel Ortiz <sameo@linux.intel.com>
    Cc: Allen Pais <allen.pais@oracle.com>
    Cc: "David S. Miller" <davem@davemloft.net>
    Suggested-by: default avatarKevin Deus <kdeus@google.com>
    Signed-off-by: default avatarSuren Baghdasaryan <surenb@google.com>
    Acked-by: default avatarKees Cook <keescook@chromium.org>
    Cc: stable <stable@vger.kernel.org>
    Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
    Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Last commit
Last update
Kconfig Loading commit data...
Makefile Loading commit data...
command.c Loading commit data...
core.c Loading commit data...
hci.h Loading commit data...
hcp.c Loading commit data...
llc.c Loading commit data...
llc.h Loading commit data...
llc_nop.c Loading commit data...
llc_shdlc.c Loading commit data...