From patchwork Sun Jul 31 07:58:44 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Heinrich Schuchardt X-Patchwork-Id: 654314 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3s2FQc0GHfz9syB for ; Sun, 31 Jul 2016 18:05:59 +1000 (AEST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751467AbcGaIFm (ORCPT ); Sun, 31 Jul 2016 04:05:42 -0400 Received: from mout.gmx.net ([212.227.17.20]:51079 "EHLO mout.gmx.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751209AbcGaIFg (ORCPT ); Sun, 31 Jul 2016 04:05:36 -0400 Received: from LT002.fritz.box ([109.91.48.91]) by mail.gmx.com (mrgmx101) with ESMTPSA (Nemesis) id 0MB2G8-1bbQd43A8a-009zQQ; Sun, 31 Jul 2016 09:58:58 +0200 From: Heinrich Schuchardt To: Florian Fainelli Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Heinrich Schuchardt Subject: [PATCH 1/1] net: ethernet: ax88796: avoid null pointer dereference Date: Sun, 31 Jul 2016 09:58:44 +0200 Message-Id: <1469951924-20276-1-git-send-email-xypron.glpk@gmx.de> X-Mailer: git-send-email 2.8.1 X-Provags-ID: V03:K0:z/W+OilInrfHdhS7mmCoWdbhDGUhOx0MsbQgH1/tB7nyZQYk9/l 28uuTRibLBI2GoOb9vululhJdYuv28b7H3/83DT6rl8H1zJdy0ANfLFssJdeI2FrKPlBEaa 9b4S4oiiUQuLvWyw9XxB+kw5oX5JQbMQ4SuLqNb0/+uGHozKrmVj+EckInmsOxC2ZSHpuXX kncNlcV0jS/lEqo55pd1Q== X-UI-Out-Filterresults: notjunk:1; V01:K0:/xN2y3SHUeM=:pDroHl/WMO6ZUoMtT6yr50 Z0Zrn0AvZ7Se25Js1Y2M/MhNp5WoUxagrTux5iRsvZ2IJMZ7GpSEMf/vxfMjyRjwMxN5+zTxm uLPvB97Q2IQl16xPnvAjHsm1WjrinkFVd1x1CladBLIMrJ7SEBuxX/YmazLkzE/JCp5I29aUM z9qXnVwxrnutUqLFC4XrlTOlita4qYwyrYeLIXGSav7AnNZLZJ2ZH2l/0ITqXUfcHj8q8VaSp tVnXjwwMhpnbeRXY6Vtqc/aa0a+f9X8GC2Iz7zSZe3lWLBTjO1zcsEzw28NnY+nf/VcU/VJkq dHhTGfEaMeS9NlKSZXzyL8lH/5njzELy5xvIPi3FD4CWVL1yJ0VFA1vUdohtQEYiswD2GJk7h ZMDACozUm8fYzBi1teiO3IEQX6OvSkuRds4b7hHyEcl4zj0MTUPYeOjWiGGPztumMyk8gtleQ +X5flvPBhzxYtZlfo7g4UOzFBmMPTRW9w3WfRiz6oh46HTa3sHkbK5lYZOPL4JC5kPGiwV/J5 D65she5awtpvMmGJu41019hDKr9p9JPJ4d6bXNXG1FnpiYx1WRORCCG8r66KrTj35rENw/+4d 9TRBda4Rtd38s5MYhWY4dCLxywrn336uQyE2qqgsEGJuHNLWWgJpdsCcHf2Qcw4/BY7kJy4j4 7wEBi83nYgbiJUWbXDyFWDH1UpGydusrhRFGX6+BFMGWd1iWRL4UnxavW1rnUMikMfsVs+OVT ioZI1B18deHNbe5jke9z0ai6sxrbtT4gPpP+N2YfitysqtHYLhDi4oukgFs= Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org If platform_get_resource fails, mem2 is null. Do not dereference null. Signed-off-by: Heinrich Schuchardt --- drivers/net/ethernet/8390/ax88796.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/net/ethernet/8390/ax88796.c b/drivers/net/ethernet/8390/ax88796.c index 5698f53..39ca935 100644 --- a/drivers/net/ethernet/8390/ax88796.c +++ b/drivers/net/ethernet/8390/ax88796.c @@ -910,7 +910,8 @@ static int ax_probe(struct platform_device *pdev) iounmap(ax->map2); exit_mem2: - release_mem_region(mem2->start, mem2_size); + if (mem2) + release_mem_region(mem2->start, mem2_size); exit_mem1: iounmap(ei_local->mem);