From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1754052AbdKFQhD (ORCPT <rfc822;w@1wt.eu>);
        Mon, 6 Nov 2017 11:37:03 -0500
Received: from mx2.suse.de ([195.135.220.15]:38113 "EHLO mx2.suse.de"
        rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
        id S1750922AbdKFQhB (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
        Mon, 6 Nov 2017 11:37:01 -0500
Message-ID: <1509985977.2024.41.camel@suse.com>
Subject: Re: [PATCH] net: usb: asix: fill null-ptr-deref in asix_suspend
From: Oliver Neukum <oneukum@suse.com>
To: Andrey Konovalov <andreyknvl@google.com>
Cc: allan <allan@asix.com.tw>, Colin Ian King <colin.king@canonical.com>,
        "David S . Miller" <davem@davemloft.net>,
        Philippe Reynes <tremyfr@gmail.com>,
        Dmitry Vyukov <dvyukov@google.com>, Kostya Serebryany <kcc@google.com>,
        Greg Ungerer <gerg@linux-m68k.org>,
        Dean Jenkins <Dean_Jenkins@mentor.com>,
        Peter Chen <peter.chen@nxp.com>, LKML <linux-kernel@vger.kernel.org>,
        USB list <linux-usb@vger.kernel.org>, netdev <netdev@vger.kernel.org>
Date: Mon, 06 Nov 2017 17:32:57 +0100
In-Reply-To: <CAAeHK+y9GTPPjL2xTasun_e9M7jdE98DCGHWFtGF=JTPzLYOyw@mail.gmail.com>
References: <9e768a837391af216cf1d2c8ac08af41f28cb147.1509654127.git.andreyknvl@google.com>
         <1509961757.2024.21.camel@suse.com>
         <CAAeHK+z9xWusLR6n_ooBdLeCQwzmLeMrbT+7cFm1PL1vuG4JMQ@mail.gmail.com>
         <1509981658.2024.37.camel@suse.com>
         <CAAeHK+y9GTPPjL2xTasun_e9M7jdE98DCGHWFtGF=JTPzLYOyw@mail.gmail.com>
Content-Type: text/plain; charset="UTF-8"
X-Mailer: Evolution 3.20.5 
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@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 <oneukum@suse.com> 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