Received: by 2002:ac0:bc90:0:0:0:0:0 with SMTP id a16csp1880535img; Sat, 23 Mar 2019 15:14:13 -0700 (PDT) X-Google-Smtp-Source: APXvYqzhZcPfQT3KG1ran/RlmE4KIt5J+Fm4J92dQNIlo4I/8EkuSdLeXDAQ93JTqINR+juhOzmu X-Received: by 2002:a63:f541:: with SMTP id e1mr15722032pgk.388.1553379253322; Sat, 23 Mar 2019 15:14:13 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1553379253; cv=none; d=google.com; s=arc-20160816; b=z2Kn9S296YDixfl4RAiLYTfhyH4zfYtLtiAK5yErj/iICsf/mDQlr6i54rgmABx4fB ejU3UPwlbwKLYkVKOavc+OiGZC05vyN7tC8WoQEV5412Ew6uDxdIUD+vAwVeYdvzHvad bIh4+h/W6H2/xUDg1lE5B2oa3aB/5oBIpGmlMj/z/qT+zGDaduEqOYTzV9+vV2WUirVM RXIUi2sybUfYFd6EcWGmp8K0z9zv8NWob4M61UeB1Fqe/XV9+8Y5Tzv6hHXqhXnzVP+B ndWo654MVHvaU81phfdmcJfKVyhne9RgPTGAG1ktjLraaSgkAeUieCoQ2b1vujbuaHdm sdqg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:message-id:date:subject:cc:to:from :dkim-signature; bh=4N/n9Cj2LolUHnf3+FkKh4bEuQdQ9jfWnJb1diucgu4=; b=vtMtmEKu3d/vct+NUbVoWIQHAMi8eigNqvTAC6sEv99z+wA8zxp8RYFyJmkzgpLOcU zlP1vVrL0wEap/LrMWh9Cc2fYvoAfv3rCKpHvckROFk6kxiEZPN+cZjTQoNwSg8RAeSC 6jvcg12QNgFKABJ9/KtDmtD1sNSFW1xLVduwh/UTMBulCua+DeOXNKGnZA6YF5DKnnDK +r5UzAD5ONddvgHta4SW214cUvNJP8Isy7NSvTwo/D++81qUmdseelRct7lINPobxZj8 2WTcgmyEueJA9ZcyKaF+QWJ7bDnIoJg4s/MutYBXDeFTj1ZJt9dONoJB+AOWNRIjGK7+ 73Hw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@umn.edu header.s=google header.b=KN2ObHhr; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=umn.edu Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id 31si10969210plc.190.2019.03.23.15.13.58; Sat, 23 Mar 2019 15:14:13 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@umn.edu header.s=google header.b=KN2ObHhr; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=umn.edu Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727767AbfCWWKZ (ORCPT + 99 others); Sat, 23 Mar 2019 18:10:25 -0400 Received: from mta-p8.oit.umn.edu ([134.84.196.208]:53564 "EHLO mta-p8.oit.umn.edu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727114AbfCWWKZ (ORCPT ); Sat, 23 Mar 2019 18:10:25 -0400 Received: from localhost (unknown [127.0.0.1]) by mta-p8.oit.umn.edu (Postfix) with ESMTP id 1F845BA6 for ; Sat, 23 Mar 2019 22:10:24 +0000 (UTC) X-Virus-Scanned: amavisd-new at umn.edu Received: from mta-p8.oit.umn.edu ([127.0.0.1]) by localhost (mta-p8.oit.umn.edu [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id OnD5wULZoaee for ; Sat, 23 Mar 2019 17:10:23 -0500 (CDT) Received: from mail-io1-f69.google.com (mail-io1-f69.google.com [209.85.166.69]) (using TLSv1.2 with cipher AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mta-p8.oit.umn.edu (Postfix) with ESMTPS id E62D89D2 for ; Sat, 23 Mar 2019 17:10:23 -0500 (CDT) Received: by mail-io1-f69.google.com with SMTP id e126so4617737ioa.8 for ; Sat, 23 Mar 2019 15:10:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=umn.edu; s=google; h=from:to:cc:subject:date:message-id; bh=4N/n9Cj2LolUHnf3+FkKh4bEuQdQ9jfWnJb1diucgu4=; b=KN2ObHhrt29Y0liChsaT3HzGeSW3TxGbIM79QEiVTHn4ouD3nfeFixfk6xVZJ7vfFQ MVEOqOS6x1JuH918+o49aTklcnHMe1QD49Q1MpJhQQwswJC4CkeeeoJJA54wQ5r2ETjN YVp809in6YRY6OrraaQIjwwhOIIfRpSKxY0QygcucjdN4HiM/tI5D5WlcFzQ8yF6oLCX RQc7PE+Vd07Ze/c17R0tHpuQ7AP8WtGiNMlpz0zx+VqSe1CBZ234sLVU0BsXf+SP6sdR nA0rdIKygnawfgEhsHIBWJKiKYXTC/1xbDhqlrsdtWJ8fxvKulw1dIzCRHO7UH+FKTAE /tBg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=4N/n9Cj2LolUHnf3+FkKh4bEuQdQ9jfWnJb1diucgu4=; b=AIkh2lsqM1xJzpo2MFPKRJeEWmIGjNG5dnxfdi5M64LMlYV6fM2ilgqkfXs8LLeSDv TAU06q2k16WWPZ5MnyK6F1rdk2gjPJ/C+I+2LM41dFkJBO92EgI/kpijDx0ACKLPX6ZI /Wxv/61aKhNbiAoYCCvLqHETJeqm6qcVCh2HhSNX7KK9FIRwx/gx10BuUdz/L/0JVl2P 6C2Kb6d0NocZUcO8u1ejlS0JIevrK6BxyuioXAXIWQS63lW0INBPQaIh5HKUmMvozmgg LPiE+yaqhJR+0CasaRTOWPdU7AcnkBlsJNFFgiFLCaPxVaiRNrIrRagCCXjNU2ZeEfEt 7nQA== X-Gm-Message-State: APjAAAVP0U4XEpmvAgP2l86LQCmqv0jlJe4Yb5LX+z+ATXtv303Hs+Ju odoHFvX9Lm+6xffvYHBw6ArDoNd5EHlj0B0uGC8m0H/xa+1OzplyvrEhnbTgoREhWkqhaFR3cEJ wUac0g5hXWulGjTtAxEY8QDthGdH+ X-Received: by 2002:a5d:8708:: with SMTP id u8mr11267913iom.299.1553379023584; Sat, 23 Mar 2019 15:10:23 -0700 (PDT) X-Received: by 2002:a5d:8708:: with SMTP id u8mr11267905iom.299.1553379023376; Sat, 23 Mar 2019 15:10:23 -0700 (PDT) Received: from bee.dtc.umn.edu (cs-bee-u.cs.umn.edu. [128.101.106.63]) by smtp.gmail.com with ESMTPSA id z13sm4730012ioj.21.2019.03.23.15.10.22 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sat, 23 Mar 2019 15:10:22 -0700 (PDT) From: Kangjie Lu To: kjlu@umn.edu Cc: pakki001@umn.edu, Jiri Kosina , Johannes Thumshirn , linux-kernel@vger.kernel.org Subject: [PATCH] firmware: edd: fix a NULL pointer dereference Date: Sat, 23 Mar 2019 17:10:16 -0500 Message-Id: <20190323221018.13026-1-kjlu@umn.edu> X-Mailer: git-send-email 2.17.1 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org As other functions in this module do, edev should be checked to ensure that it is not NULL. The fix inserts such as check to avoid potential NULL pointer dereference. Signed-off-by: Kangjie Lu --- drivers/firmware/edd.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/firmware/edd.c b/drivers/firmware/edd.c index 1b82c89a49df..7ebfaca53721 100644 --- a/drivers/firmware/edd.c +++ b/drivers/firmware/edd.c @@ -279,6 +279,8 @@ static ssize_t edd_show_mbr_signature(struct edd_device *edev, char *buf) { char *p = buf; + if (!edev) + return -EINVAL; p += scnprintf(p, left, "0x%08x\n", edev->mbr_signature); return (p - buf); } -- 2.17.1