From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj1-f45.google.com (mail-pj1-f45.google.com [209.85.216.45]) (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 E1FB85C613 for ; Mon, 17 Aug 2026 01:54:45 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.45 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786931687; cv=none; b=l28EMWl0wsLUO38f8dLLYn726npdiJ9F6P3actElhpAVvaeC8pXjpHNDAxE7ectxshthdwVBtdxWhA86C0cmOgiP1Lusi0XIV5tBpynvvmxsN2drd/AwdH2AwbQOxojXApJ/ftE70CCJldClyBIN1erAE8yKr9ixbI3bxBpOl8E= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786931687; c=relaxed/simple; bh=36JFFlxfBXpL6mrHKCopsIrLCMknMDR/uGqKUa4I68U=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=oV6kUnrafGeD/11x4JOKrg17oQvPOizXf9a+yj4sd7Oo2mZt+Ljfr2pYftUVK0gOL+eVM6Ke96CfvUrTNhfETSZR+GZEzOBnQwoKRK/8OuaoOhCikayqwk94SttWhZ4uKWaVZ/oIWXLtcGOk4sgScylDbAkav/d7SaFtmlXWA1M= 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=IJJe68pV; arc=none smtp.client-ip=209.85.216.45 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="IJJe68pV" Received: by mail-pj1-f45.google.com with SMTP id 98e67ed59e1d1-382ef647e20so2630329a91.1 for ; Sun, 16 Aug 2026 18:54:45 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1786931685; x=1787536485; 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:content-type; bh=DE8tCuQ7LoO1kue2S/5nN10COmBlfB5FK5rIj/7OLZ0=; b=IJJe68pVQU7B4gYnwcTzm4wn1f8gnO5lW9dnR03vRPezS1Rh9dYQbmlV42tmFy91kS uDqjXhmn6nHAdMhtc5f73F65/ATW3S94aLRM3V8nP4LPHrvSzOi2ABGeXNRY7nHisue6 FG9Cv0+Syge5kWRpQ3zTSrYYyZ9rfpXwpMf5VRJ1vI0RNWGqbmhjLwyKXiXv5Ez3d2SD Nd2FWV7cTL39GnDrDxnX8kPikjqN/UnU1I+9fFYmAJNFz7GKYXjNeRQz246u5elO3YU6 nMVzTwbu6If0RfW1UEUxettF89Uvkz3GutFUEraTXmfd7LbogALWIWm23gfacx38WrKo vjiQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1786931685; x=1787536485; 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:content-type; bh=DE8tCuQ7LoO1kue2S/5nN10COmBlfB5FK5rIj/7OLZ0=; b=E/mX3GMWhe8ggvnBVvyGWzYWe6QvNorSg6iN8s3NkGYI3Z7mjYf2XVR4qe4AqEZIES Y1KRV8Wc74fFEfhivnmHje8WTUohYdiUYeZb+PzA80hUZlymE24Tw1t+VrodG3R9Yhh9 DNnv47iEa731vthvACIIkaTfJACPUdOzpD66m/iYRtPA+r+BREgqavP1yyIG6FAPLaVa GixYic42FDqfE6uOvoPXmHHbvBlFn7l/V89JVMqci0DWQAjGP8YZZ9oswYAjIMS08hpK qEmPW7y5CacCItJxS4n4YY9arS0bvKlH9eku4hZc3cLYIsKOjrZkJ5VObLMbdpmHGnks OAaQ== X-Forwarded-Encrypted: i=1; AHgh+Rp8Pf/fh9O0DODT07vBCyV8fvG2oTHxJkL0DSPXge8Fkkrqppzoe9Iqy4CzIcW8ltImBMC9U1Cj/qgBoWM=@vger.kernel.org X-Gm-Message-State: AOJu0YzOgFDAPbL0+1p/ck2QupFNxmlLAe/g+50v9rBu/GmDxP+4Lgmo 9xH85TLLOIgygrbIwRp7SAlVjxNXHPgQK6j36sFSzcbTMxt45GN4/6e4 X-Gm-Gg: AR+sD111Opbw1V4G/AzBG+lQxWJCiyflPLC2BqckfaiHayrYynm8o6pelI51lcASHqV Xa3Nf9G2Bdtr1e9hm8Ux9BW6HFmx/sYo28pClGHn7LF//9jmI1j7ESIqbm0Df/3fQ4DhCGfET3S agEKGG4hoTjTOZOLIR3un7NIgUhMKy44zjdgO3vRaf3Jnhnk7gdjztIzdOAb2JF0rjK1RXADe/o sgEJHMzOF6r7Wn52EpAUjoseCk/H0X+WjAo0n/qepzQOK5aw3ujwdm8It6IhumjBIK0D2VsnrX0 zAGfuwjOIdJhORKLLujleY50PKCGRKIQk1ktgS283DjSKG0wl/eu9NFSL7bje+69b7tRvZOKgO0 iExoPwnXLxW0+08tHQMQio8PqM2CpW3TX52XTf8mmZlPdvLYnE9R8QN7CdqgJAlwCiQIylFxv6S cn7VD075waj1Hpk6HW5wQVQbGLIKhIAOIW+rRbWlK2DM0qDq/c0TeHU3fgJ91WlA7/sIwdD3VFe nSQ84xRpbwtfAuGGiabmcQXvgLQMox5XO/7cK+7X6Fb8atrVRHCw0ZPuvfMs+RruCBcJjhhXFsU yhZG3HYN7IrV2jPzLcH8BC9wAJ/h+DJfhgBDCb/s4c6R0G6j X-Received: by 2002:a17:90b:2e8f:b0:38f:5869:387b with SMTP id 98e67ed59e1d1-3933cb718fcmr23449061a91.9.1786931685205; Sun, 16 Aug 2026 18:54:45 -0700 (PDT) Received: from ryzen.lan ([2601:644:8000:7a86::e35]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-39339cec8c8sm4405106a91.1.2026.08.16.18.54.43 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 16 Aug 2026 18:54:44 -0700 (PDT) From: Rosen Penev To: dmaengine@vger.kernel.org Cc: Vinod Koul , Frank Li , Paul Mackerras , Grant Likely , linux-kernel@vger.kernel.org (open list) Subject: [PATCH] dmaengine: bestcomm: set bcom_eng to NULL on probe failure Date: Sun, 16 Aug 2026 18:54:42 -0700 Message-ID: <20260817015442.136038-1-rosenp@gmail.com> X-Mailer: git-send-email 2.55.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 Functions like bcom_task_alloc() use if (!bcom_eng) as a safety readiness check. If the probe fails and hits this error path, subsequent or concurrent API calls will bypass the check and dereference the freed memory. Fixes: 9560aea4e9d1 ("[POWERPC] mpc5200: Fix null dereference if bestcomm fails to initialize") Signed-off-by: Rosen Penev --- drivers/dma/bestcomm/bestcomm.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/dma/bestcomm/bestcomm.c b/drivers/dma/bestcomm/bestcomm.c index 432b43520ddc..ec58d2c6573f 100644 --- a/drivers/dma/bestcomm/bestcomm.c +++ b/drivers/dma/bestcomm/bestcomm.c @@ -445,6 +445,7 @@ static int mpc52xx_bcom_probe(struct platform_device *op) release_mem_region(res_bcom.start, sizeof(struct mpc52xx_sdma)); error_sramclean: kfree(bcom_eng); + bcom_eng = NULL; bcom_sram_cleanup(); error_ofput: of_node_put(op->dev.of_node); -- 2.55.0