Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp5177483imm; Tue, 31 Jul 2018 06:49:51 -0700 (PDT) X-Google-Smtp-Source: AAOMgpedOtMGckrIbFpKnEphjh/aDNU1eAjP/nXa8c52lcZiF1RV27ffLb8KDw+A5t7W4U1btmT4 X-Received: by 2002:a63:d704:: with SMTP id d4-v6mr20597064pgg.312.1533044991824; Tue, 31 Jul 2018 06:49:51 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1533044991; cv=none; d=google.com; s=arc-20160816; b=ZXpjp+puD0h6F+DIxZnfuXo5ZyAqqh0VoD/6vSYsD3hn0CA4467t6twKVTWQzFexKz LWcqato91FXhbmV8iVMwQtnEhtSSrWo1aCLLKap6AfDX/oJkA2JAC30XgC7A75FxBvTz O/rIawM6cTL2sKQRdHyrdiWlxoPwsTyu/Ya0wwtiSRoAAzZAERXP327RtuulRHYCX6zN 9fdpNUah9mfw/lJRteM9KIX84sN8Hofi1Y+2UeV0LIc0y0cW0n1LXMEJdWMgehETp7o/ swy91rvw2kIuTc974097+ZHJvVUAz3jSfg1gFkPTDk9ZzVlg6rHyWFYDPWDqAw93DT// xCBw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:references:in-reply-to:message-id:date :subject:cc:to:from:arc-authentication-results; bh=1l7yKRrBBrZvy9roYaWvJrmy2xXZ4hrXYXRLRM1PUp0=; b=cMAXD7kH1R84GI5cxactb/8ch/++6HY0HqC/9vQg1TSnrHWOZ0Fn4XvNyAP4MFht87 B3y9rqCzBxMqBuJeupXE+UkHcfsG2EoF9GNAN4gLluTmQlzfNLOTyBcfZ88N7O92EgvF 6i8trj4DRE48g3KUZndc+0s/0OOu5bHpjX160WEr6dCZueNMfMoiWkoG9hfldlzo6mb5 rZjgS6I1+zbuyWNL8IC1dQm2LpVTsxJ9Xku1XFi87viu+6NM1MKvD42bLQNcxyYgWHPY Rz9XFTpnB4amGKnwZsoOl1JHS/PEaQnzFT2yUkCBoP01o0ABQdzGc71FO+TX+mkY3Wh5 b7jw== ARC-Authentication-Results: i=1; mx.google.com; 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 Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id j17-v6si14122850pfk.203.2018.07.31.06.49.37; Tue, 31 Jul 2018 06:49:51 -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; 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 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1732432AbeGaP2s (ORCPT + 99 others); Tue, 31 Jul 2018 11:28:48 -0400 Received: from mx2.suse.de ([195.135.220.15]:41916 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1732378AbeGaP2h (ORCPT ); Tue, 31 Jul 2018 11:28:37 -0400 X-Virus-Scanned: by amavisd-new at test-mx.suse.de Received: from relay2.suse.de (unknown [195.135.220.254]) by mx1.suse.de (Postfix) with ESMTP id 25B20AE1B; Tue, 31 Jul 2018 13:48:12 +0000 (UTC) From: Johannes Thumshirn To: "Martin K . Petersen" Cc: Linux Kernel Mailinglist , Linux SCSI Mailinglist , ard , Johannes Thumshirn Subject: [PATCH 1/3] fcoe: fix use-after-free in fcoe_ctlr_els_send Date: Tue, 31 Jul 2018 15:46:01 +0200 Message-Id: <20180731134603.20089-2-jthumshirn@suse.de> X-Mailer: git-send-email 2.16.4 In-Reply-To: <20180731134603.20089-1-jthumshirn@suse.de> References: <20180731134603.20089-1-jthumshirn@suse.de> Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org KASAN reports a use-after-free in fcoe_ctlr_els_send() when we're sending a LOGO and have FIP debugging enabled. This is because we're first freeing the skb and then printing the frame's DID. But the DID is a member of the FC frame header which in turn is the skb's payload. Exchange the debug print and kfree_skb() calls so we're not touching the freed data. Signed-off-by: Johannes Thumshirn --- drivers/scsi/fcoe/fcoe_ctlr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/fcoe/fcoe_ctlr.c b/drivers/scsi/fcoe/fcoe_ctlr.c index ea23c8dffc25..ceb35ebbeb8f 100644 --- a/drivers/scsi/fcoe/fcoe_ctlr.c +++ b/drivers/scsi/fcoe/fcoe_ctlr.c @@ -799,9 +799,9 @@ int fcoe_ctlr_els_send(struct fcoe_ctlr *fip, struct fc_lport *lport, fip->send(fip, skb); return -EINPROGRESS; drop: - kfree_skb(skb); LIBFCOE_FIP_DBG(fip, "drop els_send op %u d_id %x\n", op, ntoh24(fh->fh_d_id)); + kfree_skb(skb); return -EINVAL; } EXPORT_SYMBOL(fcoe_ctlr_els_send); -- 2.16.4