From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pg1-f169.google.com (mail-pg1-f169.google.com [209.85.215.169]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 171193D75CF for ; Tue, 14 Apr 2026 23:46:38 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.215.169 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1776210401; cv=none; b=a6gT4YrAz3VmMHnLevd7Qr1olvVqUkdALjoI7gokmXXwLKQFuKfz1vanJNivHCFTYT6iAGNy8niEF7NcYYtuOhlxKEAyAAvbY5xgHnqbqLKtBp7jkHD23oggontn7rjpWkUfBoues6Xw3qVCkaGR4+uKU9pOAndGqgE7EuQH2aw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1776210401; c=relaxed/simple; bh=/2hIp5BZLooUW37+BmEgbWSArMAppLPs2okM3gNV8jo=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=L7fYFygkJo8nNbdwu/kFl/D674nIQfrvms8y9P/5iWn7PRJ3OKyRog5AxFlfOp74XR7s/tMVQKYnPiJOk/nJHBjrJswU8GeJ0danPZ65C7FfYDFF6OYWKbQE7Wd2NlANjY/niDKiYI2EakyuVP2ViBw0fLdV0+zxQ2RT+8LO+bc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=ejhHwtrB; arc=none smtp.client-ip=209.85.215.169 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="ejhHwtrB" Received: by mail-pg1-f169.google.com with SMTP id 41be03b00d2f7-c79467f128cso799502a12.3 for ; Tue, 14 Apr 2026 16:46:38 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1776210398; x=1776815198; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=e+aagt8mewkTvyv3R6Dqrd15T0YDKTqKrH1kiymIT8g=; b=ejhHwtrB9EqxqKBxEBH2Cyon+JDPgc9th03SRehxMCuHi/tXl8bxG+sEz+WVEe+sL2 auQiqktdFMjf5HApILzs9foRZJAeKxvJYXN9fW1dqUjKNDsfEMhckn1kbJYHn5UKaGDS QWNdc6h+OF2wWw+OzZmO5liCnFwCneVC06sRksBprsSQ+R0kdZoe2NaKvOgA/E6/woiV 3vFuVZyzPZbFFfsU1Guuoavj6w+vwI8xoXn+gMj832bsywq7xJC9uXUYsEFeCEAZq1zU wilfqTa8EnYVzD9Av6Uz4zq0GL3uRlWOqC9GgV3MsV5TJykuJ/HH3Jo9B/hBAqexy84G gWDQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1776210398; x=1776815198; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=e+aagt8mewkTvyv3R6Dqrd15T0YDKTqKrH1kiymIT8g=; b=ksh+tMcDY5k8x+iuBDrA3sfIKm6PILPBqOMGRk0sgptz0QAwZ2N3uCynguN2CkBO3p A6tMyp5I+GOqacvJWhNfbjOpxRna6LXUMbd7x/2oxo/hdK1X03etbcLC44IcAz2hNmKZ rEKvfrDwXY38irGU/OpsgG4yjxs8CZUQvVumB+2YtbpQMmfdYIBZEgAVAsinyh0ggOpc Y9Ot8Ym/s6da7SXh+Os86PdgREYHR4GHvyedp0hNOmxm3IGrhawxslnJZfgZyH7/mt3g hA4jHtZ82R8ehQ5O/prtr8E/9XpC0GsaU/Cl13xUaE3RCRmjbEbPIB1uTGgtXnet38jh aHJQ== X-Forwarded-Encrypted: i=1; AFNElJ+SHD3PCjz5BqyKcWgKdR9xXqmRjmK38BlkEMo+8AyFqyvzyg0AkmUPogieI3q21ljNWifFtgiOUEX4sog=@vger.kernel.org X-Gm-Message-State: AOJu0YwSnxlEvg/vTI5VVE7r8huKLXkAJcWA+X55Z4bSiefll1E6FRYh Ab0xa9afxmPBHPNWUBNNv/XyhFObFLy9h6MGVl/nHM/qvVSyrgvWNQHJ X-Gm-Gg: AeBDiesm32/DjXBfrQzlJT/IxswClfLNolQqYVc+4oQkCkfo33ee/gmA1CwjcXDuodb uMPfzDyfyx/rvI79NRvL9VH7rTchcmmjAalzytTom8R/HiLkvRxJFZGERlT5Wm/9mWBZm77X9Jk CfTErzBpTPDwxk3iBd2gnlwoCZqaUz9FFFQcslAwu5a0h2dfe9TTbOo7iJHOJVBN5bb7oWs5z22 2mw2c8cAo/elatUX0cbZ7998lQXv4C4c2Nd4HSEcKChMDVtHEyja3eBqkC1uShrquXRygfm1DCv E37PKMi3K/1vW5IQE6i2kG9x/RL6V2mBITIdL8EMiFo4PKILbtV17gUQJQIVcoJWgi9yEGcyOiE CSCTTDLYDDFghik8envk1PfPD7q+lKfozeZAIQ/Mvs+vebgUXwJtrs/niyYWhf/h1Z91+HoSR2F R4lDIftzkThZQ2JWsjBFK2lRQV7ptbInMiL7c11gM= X-Received: by 2002:a05:6a00:886:b0:82f:592f:2ed7 with SMTP id d2e1a72fcca58-82f592f35f5mr3850866b3a.45.1776210398171; Tue, 14 Apr 2026 16:46:38 -0700 (PDT) Received: from zenbook ([203.30.245.12]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-82f6702f900sm153027b3a.2.2026.04.14.16.46.33 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 14 Apr 2026 16:46:37 -0700 (PDT) From: Wilfred Mallawa To: linux-xfs@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Damien Le Moal , Alistair Francis , Carlos Maiolino , Hans Holmberg , "Darrick J . Wong" , Christoph Hellwig , Wilfred Mallawa , stable@vger.kernel.org, Carlos Maiolino Subject: [PATCH v2] xfs: fix memory leak on error in xfs_alloc_zone_info() Date: Wed, 15 Apr 2026 09:45:14 +1000 Message-ID: <20260414234513.1457961-2-wilfred.opensource@gmail.com> X-Mailer: git-send-email 2.53.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit From: Wilfred Mallawa Currently, the 0th index of the zi_used_bucket_bitmap array is not freed on error due to the pre-decrement then evaluate semantic of the while loop used in xfs_alloc_zone_info(). Fix it by allowing for the i == 0 case to be covered. Fixes: 080d01c41d44 ("xfs: implement zoned garbage collection") Cc: stable@vger.kernel.org Reviewed-by: Damien Le Moal Reviewed-by: Carlos Maiolino Signed-off-by: Wilfred Mallawa --- fs/xfs/xfs_zone_alloc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/xfs/xfs_zone_alloc.c b/fs/xfs/xfs_zone_alloc.c index a851b98143c0..c64f9ab743a6 100644 --- a/fs/xfs/xfs_zone_alloc.c +++ b/fs/xfs/xfs_zone_alloc.c @@ -1217,7 +1217,7 @@ xfs_alloc_zone_info( return zi; out_free_bitmaps: - while (--i > 0) + while (--i >= 0) kvfree(zi->zi_used_bucket_bitmap[i]); kfree(zi); return NULL; -- 2.53.0