lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Date:   Wed, 29 Mar 2017 18:21:05 +0200
From:   Johan Hovold <>
To:     Lauro Ramos Venancio <>,
        Aloisio Almeida Jr <>,
        Samuel Ortiz <>
Cc:,,, Johan Hovold <>
Subject: [PATCH 0/7] NFC: fix device allocation and nfcmrvl crashes

This started out with the observation that the nfcmrvl_uart driver
unconditionally dereferenced the tty class device despite the fact that
not every tty has an associated struct device (Unix98 ptys). Some
further changes were needed in the common nfcmrvl code to fully address
this, some of which also incidentally fixed a few related bugs (e.g.
resource leaks in error paths).

While fixing this I stumbled over a regression in NFC core that lead to
broken registration error paths and misnamed workqueues.

Note that this has only been tested by configuring the n_hci line
discipline for different ttys without any actual NFC hardware connected.


Johan Hovold (7):
  NFC: fix broken device allocation
  NFC: nfcmrvl_uart: add missing tty-device sanity check
  NFC: nfcmrvl: do not use device-managed resources
  NFC: nfcmrvl: use nfc-device for firmware download
  NFC: nfcmrvl: fix firmware-management initialisation
  NFC: nfcmrvl_uart: fix device-node leak during probe
  NFC: nfcmrvl_usb: use interface as phy device

 drivers/nfc/nfcmrvl/fw_dnld.c |  7 +++++--
 drivers/nfc/nfcmrvl/main.c    | 25 +++++++++++++------------
 drivers/nfc/nfcmrvl/uart.c    | 11 +++++++----
 drivers/nfc/nfcmrvl/usb.c     |  4 +---
 net/nfc/core.c                | 31 ++++++++++++++++++-------------
 net/nfc/nci/core.c            |  3 +--
 6 files changed, 45 insertions(+), 36 deletions(-)


Powered by blists - more mailing lists