From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751871AbdIUHbq (ORCPT ); Thu, 21 Sep 2017 03:31:46 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:54852 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751606AbdIUHbp (ORCPT ); Thu, 21 Sep 2017 03:31:45 -0400 Date: Thu, 21 Sep 2017 09:31:54 +0200 From: Greg Kroah-Hartman To: Andrey Konovalov Cc: Jonathan Corbet , Mauro Carvalho Chehab , Jaejoong Kim , USB list , LKML , Dmitry Vyukov , Kostya Serebryany , syzkaller Subject: Re: usb/core: slab-out-of-bounds read in cdc_parse_cdc_header Message-ID: <20170921073154.GA11294@kroah.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.9.0 (2017-09-02) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Sep 20, 2017 at 04:45:08PM +0200, Andrey Konovalov wrote: > Hi! > > I've got the following crash while fuzzing the kernel with syzkaller. > > On commit ebb2c2437d8008d46796902ff390653822af6cc4 (Sep 18). > > It looks like cdc_parse_cdc_header() doesn't validate buflen before > accessing buffer[1], buffer[2] and so on. The only check present is > while (buflen > 0). Ugh, you are right, let me go work on a patch, thanks for the report...