Received: by 2002:a05:6359:6284:b0:131:369:b2a3 with SMTP id se4csp1034643rwb; Sat, 5 Aug 2023 06:10:59 -0700 (PDT) X-Google-Smtp-Source: AGHT+IEEDDUJKyB7m2FQtFjMC9hQ+f+F+JWamWLJLmHIcqYfH1tPev0JfHCUwF36GpyUeCZYakSK X-Received: by 2002:a17:902:e5d1:b0:1b2:676d:1143 with SMTP id u17-20020a170902e5d100b001b2676d1143mr3196030plf.15.1691241059258; Sat, 05 Aug 2023 06:10:59 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1691241059; cv=none; d=google.com; s=arc-20160816; b=wN8H+KrK/M+G2N2AB8TnHH38lD6TtU0zMzxGWFKaIAWQPG/f/Ak9VfF26edCtFrFCt lQbg8xn3EJELOfMhDLsRmRKwh8ltc8LueKTqH7Jfk0AVFNpDfulb3YY244WaaDM+neWE PC2IMrnImdzanNl2h49w8g5dPG8UmbZjFKpD2wBY9YdQpS+Ei74cGFN3cYJRroCRMuoD dExyWfSi/BcEfZesZR2pQ5kazk9hwswv3gNejEXSbrNAtovY3MRaUS7HDJIJYbxaUKCe 4rzuq4e+2BLhN2XU4K7jqOeIsKZ7ZJQt9Ws8Vc/7HOOEVtjkMpL/woIubRDr+c0VhfH5 84VQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=eYb2O0IuXmf6IAFBpyM9vHqkTVB4ZgGCAGWOPW1z0h0=; fh=VNu+q/OmxP47EcyJz/3EN4i7T2d5iVBskPNxUJNEk1Y=; b=ou6guzh0zCTLvAn02d7Fld0KV7ako3MEsmdcL3JPqK19GN8HmhR1aSiyRkU1S8+A3S te7fnwq1av4YN8caaCWxeHcQDWxUHR9TUjO2V1cU9aUl1kWGWFNN6VItcn2XhdqccBEU DrZvmnj2boIk+5akNJG4zs+0YAb+2SBcjpcNq0/YwufXjGqBDYC6gAPeHUbOd/KFJQKe QzPcejsA11e/FlQxKmKJ3f3s2vDdAYH5eXVBPmOzGPDOPRiUOaL4Bj1dCRqpi0qrGGbL ymvMx8CF/A65bnV2rtj9S5lBoSvI7SYvnzr5or6Xgnv3YYkwU36r5tFf9Zd1/hPoXiV1 iqOA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@mit.edu header.s=outgoing header.b=ZY2yisKM; spf=pass (google.com: domain of linux-ext4-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-ext4-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=mit.edu Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id c16-20020a170902d49000b001b9e1d5f8e6si3276868plg.447.2023.08.05.06.10.45; Sat, 05 Aug 2023 06:10:59 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-ext4-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@mit.edu header.s=outgoing header.b=ZY2yisKM; spf=pass (google.com: domain of linux-ext4-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-ext4-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=mit.edu Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229767AbjHEMUq (ORCPT + 99 others); Sat, 5 Aug 2023 08:20:46 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59630 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229506AbjHEMUp (ORCPT ); Sat, 5 Aug 2023 08:20:45 -0400 Received: from outgoing.mit.edu (outgoing-auth-1.mit.edu [18.9.28.11]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id BBFB544BD for ; Sat, 5 Aug 2023 05:20:44 -0700 (PDT) Received: from cwcc.thunk.org (pool-173-48-112-100.bstnma.fios.verizon.net [173.48.112.100]) (authenticated bits=0) (User authenticated as tytso@ATHENA.MIT.EDU) by outgoing.mit.edu (8.14.7/8.12.4) with ESMTP id 375CKVrX027553 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Sat, 5 Aug 2023 08:20:32 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=mit.edu; s=outgoing; t=1691238033; bh=eYb2O0IuXmf6IAFBpyM9vHqkTVB4ZgGCAGWOPW1z0h0=; h=From:Subject:Date:Message-Id:MIME-Version:Content-Type; b=ZY2yisKMU0yHb1/kCsKiDCDsQsEn660in3xvdlb2WQk3zPH76vea1fx51ChilGeQL rTTOCLXGq6aSfrjIS7XwhePnbC3IwWovsS/ILiFSAsk85/U3tcI88fzk+IMIPI+j6B 74u3/fEBO2iDNCzSKH5m6BOm+zGidr8br7FO8noipuOHLTJck5AE8JwVyyw11PC9cN EFcgem/DqZfeb0a7P8NceygvA7Yh3jJYDofhSZZ8PK2EhMYwBSiaBc/hZdGaq5Uhp9 5kkix/Woi/tkIPt8s6PFG14pluz5bB71glHJT92XOgCoOEl1D8X2lUz9rvH0PJiPDb QRSXS+QbgUTAw== Received: by cwcc.thunk.org (Postfix, from userid 15806) id 525B815C04F1; Sat, 5 Aug 2023 08:20:31 -0400 (EDT) From: "Theodore Ts'o" To: Andreas Dilger , Daniel Rosenberg , Eric Biggers , =?UTF-8?q?Lu=C3=ADs=20Henriques?= Cc: "Theodore Ts'o" , linux-ext4@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v3] ext4: fix memory leaks in ext4_fname_{setup_filename,prepare_lookup} Date: Sat, 5 Aug 2023 08:20:24 -0400 Message-Id: <169123801881.1434487.6868481309254151521.b4-ty@mit.edu> X-Mailer: git-send-email 2.31.0 In-Reply-To: <20230803091713.13239-1-lhenriques@suse.de> References: <20230803091713.13239-1-lhenriques@suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,SPF_HELO_NONE, SPF_NONE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-ext4@vger.kernel.org On Thu, 03 Aug 2023 10:17:13 +0100, Luís Henriques wrote: > If the filename casefolding fails, we'll be leaking memory from the > fscrypt_name struct, namely from the 'crypto_buf.name' member. > > Make sure we free it in the error path on both ext4_fname_setup_filename() > and ext4_fname_prepare_lookup() functions. > > > [...] Applied, thanks! [1/1] ext4: fix memory leaks in ext4_fname_{setup_filename,prepare_lookup} commit: 7ca4b085f430f3774c3838b3da569ceccd6a0177 Best regards, -- Theodore Ts'o