Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp2395851imu; Tue, 6 Nov 2018 13:55:26 -0800 (PST) X-Google-Smtp-Source: AJdET5eMy2Osc8x7KIUjwUfnWdzeSEsk2cFHAsBA/f/IELgi/RNtIK0MjeJfjSO0ZuUy3q35dfR/ X-Received: by 2002:a63:b16:: with SMTP id 22mr25136904pgl.306.1541541325993; Tue, 06 Nov 2018 13:55:25 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1541541325; cv=none; d=google.com; s=arc-20160816; b=aAY2aAsnpsT3kcJ5eiIRuST925imKxZy6IVzB8uLNYt3BRq2CvC8WDUwXqr4fpQFmi BAvOHITHsPkjiCdVzGUZQ6pE0udPec/HY5F4q4W0tQObF23vyJ5zArYZmZ688cJcC9Wq BgLGG55sx4gbemDEwx6I1tTfCzRTypA6Na4CU1LSyATW447Y6YeXALeFc3Y6qTseL2n1 XVqvFaEPHtLQEzEZJ3qYm1zv/xWPDP4zRyihn5xfcGv2DqIuoraxKxTV0x1R+cr1MSc9 a9tc23Gv07XNPJIWgqTMe1xNDR+U7CKkvxygXvjTR5tJ0EAtles/EVHFo5zZg9tEKCZW BKTQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:mail-followup-to :message-id:subject:cc:to:from:date:dkim-signature; bh=iXElp8w1zzYKfimvI2VqAXLUv+s8NYA+OE6lJJAihF4=; b=aT9LJlmkz8ButEE1Vpzyt5gmRQgemIGt6b4YSnoQfRLn1cGK2emmCcHnyznjdiExYf nFWtcHCH7MfAWGGzCRMrbhx8FJfCSfavklkXFhpaqFcr5VnNEaS6++rG6Efj44XVIZOP MteI5rsJaqIwYLpnOU/1k3C9YTPkvarrSQ0pFkH8gReQtVwr+czq+qryE+V+e4pCTmt2 Z7mDgj/WXHyG+yEpjfunCHf3vLeC+fCNgofvJb5lqDUWwcKuCQhA8pUr1BpCbP+IhC8k 2o+f7MxavVxw09cwfz6UO8twujH8k/aWwFT8ZM09gpyZUasSdL3QocX1dUVcBc0t8cWB TcQw== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@thunk.org header.s=ef5046eb header.b=SG0gd6vz; 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 h5si13598244pgc.237.2018.11.06.13.55.11; Tue, 06 Nov 2018 13:55:25 -0800 (PST) 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=fail header.i=@thunk.org header.s=ef5046eb header.b=SG0gd6vz; 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 S1730806AbeKGHUa (ORCPT + 99 others); Wed, 7 Nov 2018 02:20:30 -0500 Received: from imap.thunk.org ([74.207.234.97]:46048 "EHLO imap.thunk.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726069AbeKGHU3 (ORCPT ); Wed, 7 Nov 2018 02:20:29 -0500 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=thunk.org; s=ef5046eb; h=In-Reply-To:Content-Type:MIME-Version:References:Message-ID: Subject:Cc:To:From:Date:Sender:Reply-To:Content-Transfer-Encoding:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=iXElp8w1zzYKfimvI2VqAXLUv+s8NYA+OE6lJJAihF4=; b=SG0gd6vznAW8aPfLq/LtEl3oQ2 6BMI2NaEXrrtsNloF7rDPSB5PS8UxEJ1WZHidvgprCLeAYFUROaNmXLZ/r/K1fKi27qnqu3vzon4R iFIKtfvikfP98RTVLbpWr2bB4Rz4T/3JeB+BNl45yKW+6MaZ9/L/W37RRYTtleEuT/J4=; Received: from root (helo=callcc.thunk.org) by imap.thunk.org with local-esmtp (Exim 4.89) (envelope-from ) id 1gK9Ha-0005nF-Ku; Tue, 06 Nov 2018 21:53:10 +0000 Received: by callcc.thunk.org (Postfix, from userid 15806) id 7B6B87A7C2F; Tue, 6 Nov 2018 16:53:09 -0500 (EST) Date: Tue, 6 Nov 2018 16:53:09 -0500 From: "Theodore Y. Ts'o" To: Vasily Averin Cc: linux-ext4@vger.kernel.org, Andreas Dilger , linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 07/11] ext4: lost put_bh in ext4_mark_iloc_dirty() Message-ID: <20181106215309.GF2570@thunk.org> Mail-Followup-To: "Theodore Y. Ts'o" , Vasily Averin , linux-ext4@vger.kernel.org, Andreas Dilger , linux-kernel@vger.kernel.org References: <61d28461-8fdd-d614-8bde-844703936ac1@virtuozzo.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <61d28461-8fdd-d614-8bde-844703936ac1@virtuozzo.com> User-Agent: Mutt/1.10.1 (2018-07-13) X-SA-Exim-Connect-IP: X-SA-Exim-Mail-From: tytso@thunk.org X-SA-Exim-Scanned: No (on imap.thunk.org); SAEximRunCond expanded to false Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Oct 31, 2018 at 12:58:24AM +0300, Vasily Averin wrote: > ext4_mark_iloc_dirty() callers expect that it releases iloc->bh > even if it returns an error. > > Fixes 0db1ff222d40 ("ext4: add shutdown bit and check for it") # 4.11 > > Signed-off-by: Vasily Averin Thanks, applied. I fixed up the commit description and Fixes/Cc trailers. I used the one-line description: ext4: avoid buffer leak on shutdown in ext4_mark_iloc_dirty() - Ted