From mboxrd@z Thu Jan 1 00:00:00 1970 From: Oliver Neukum Subject: Re: [PATCH] net: usb: asix: fill null-ptr-deref in asix_suspend Date: Mon, 06 Nov 2017 17:32:57 +0100 Message-ID: <1509985977.2024.41.camel@suse.com> References: <9e768a837391af216cf1d2c8ac08af41f28cb147.1509654127.git.andreyknvl@google.com> <1509961757.2024.21.camel@suse.com> <1509981658.2024.37.camel@suse.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: allan , Colin Ian King , "David S . Miller" , Philippe Reynes , Dmitry Vyukov , Kostya Serebryany , Greg Ungerer , Dean Jenkins , Peter Chen , LKML , USB list , netdev To: Andrey Konovalov Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org Am Montag, den 06.11.2017, 17:05 +0100 schrieb Andrey Konovalov: > On Mon, Nov 6, 2017 at 4:20 PM, Oliver Neukum wrote: > > > I do have a way to reproduce this. > > As far as I understand, for this particular device ax88172_bind() is > called, which doesn't assign anything to dev->driver_priv, so that's > why it is NULL in suspend() and resume(). Thanks and ouch. That means it never worked for those devices. That makes this a rather serious bug and your fix is right. Regards Oliver