From mboxrd@z Thu Jan 1 00:00:00 1970 From: Samuel Ortiz Subject: Re: [PATCH] NFC: fix attrs checks in netlink interface Date: Mon, 4 Jun 2018 06:29:00 +0200 Message-ID: <20180604042900.GB31498@zurbaran.ger.intel.com> References: <75ce3040b4086ffa2d2e088ad7f24f5e4a87be56.1523552145.git.andreyknvl@google.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: "David S . Miller" , linux-wireless@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Dmitry Vyukov , Kostya Serebryany To: Andrey Konovalov Return-path: Content-Disposition: inline In-Reply-To: <75ce3040b4086ffa2d2e088ad7f24f5e4a87be56.1523552145.git.andreyknvl@google.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org Hi Andrey, On Thu, Apr 12, 2018 at 06:56:56PM +0200, Andrey Konovalov wrote: > nfc_genl_deactivate_target() relies on the NFC_ATTR_TARGET_INDEX > attribute being present, but doesn't check whether it is actually > provided by the user. Same goes for nfc_genl_fw_download() and > NFC_ATTR_FIRMWARE_NAME. > > This patch adds appropriate checks. > > Found with syzkaller. > > Signed-off-by: Andrey Konovalov > --- > net/nfc/netlink.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) Thanks, applied to nfc-next. Cheers, Samuel.