From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f177.google.com (mail-pf1-f177.google.com [209.85.210.177]) (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 55C542E0B5C for ; Fri, 31 Jul 2026 16:18:20 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.177 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785514702; cv=none; b=WwW7p6OCpW7LZA3tr9XbQA0ydVT0wiUo4bH1yHdMsLzn+dpkwcKtMigrtinp5bri1Jl92XoUkIJEi4cI08sXMSPBdmo6hXU10nntnhIIdKx2bkIbQb23zKj3rvwrmuGJEFxz38eLHVlF9/kaBI/3U1y8xCK8XKzgQuBdw0Kr+60= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785514702; c=relaxed/simple; bh=9Yzh38GRiodpjie4zcPRooc5jBJTJ/dzig1dvhms+IY=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=MFzWbT+6q/m1pINo5t+G34fcplJlbO/afc4DZufUHomDb6zh9Qximlo2vNaN0hKgGnGJWtld1sabU+psetOieC60bZJV0JArBHTkrQhNxL5DK3J1W4tLxNX4w6anVLwZ8TPNMkqMNm4onz47wyrpAtN+lMHahcVPKhCERLcdaAM= 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=nA3kFAUf; arc=none smtp.client-ip=209.85.210.177 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="nA3kFAUf" Received: by mail-pf1-f177.google.com with SMTP id d2e1a72fcca58-84847482584so737858b3a.0 for ; Fri, 31 Jul 2026 09:18:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1785514700; x=1786119500; 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=TKjEK1gC87c9kIwXqKCw727kvsVZgAnx7reueDbTuj0=; b=nA3kFAUfNnWsWvjABU3qAPTa2LJKAgkKSsu6xI0ZOYvcntzGZ81szLCwl51kWs7uxl YwFO3cUVROsezXWdKL+9Fw30h+e6QgLl+lWb5R70DcWUD3X6QH5INo4AcOVXjkTUQBQT Wwr5kgaa9kWCEunaNoM6DGtwemBeMgQDw6UFVQPGDs9d8N11UIxHkCCzIZnRFoPQGWnh E5UpLt0RobFeAH/6QAxowKliQVUfd5wS7uevI9Aj124OpqDU7Nu0sn9xwYWOSonRbGDz gWP3EsYLKJOCCXYMWKAhvuL4/KbpWZZ3KjYNFyLrKrD/YBJjeRQbPFyX72zP2QLwtfsT 6u5g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1785514700; x=1786119500; 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=TKjEK1gC87c9kIwXqKCw727kvsVZgAnx7reueDbTuj0=; b=PksWag9SQ/nFXJ8dBnCzKoGFeTV6ZJuRXy8DUNkwG/8bcYtS9wnPYNm8j143Yve24x LtbxrQ3239M43eLCx/UYpsiNk09NKWkcXV98n5MxDb544s9qeti93k+vdYhPOAZWywJA Z8bnDjN3yEDNZMfKU+7rlQIizTcopaXLNj1bjNMz8Czadx4p3eCDnhOA0WJGuIgXgNkg CO9Wh7s4YJTAEpmJcFmwpxU/wSLSDmCbBMxtkVFC/uOFxSe4rR4VUthkdlKRjiAcRdGb FANq9l8iJDeO0LIfry/HIchTwOs19lVXJarP8bJxhwDkVq404ZfvmyPfhsyeuXxHl/Bx v+xA== X-Forwarded-Encrypted: i=1; AHgh+RpA5uI9lhCJgGxKICtxiE44sRsGFeidzNOJtLZPDrpdYK4maS7LTj19vQ+BioNkvGlxVJR76cAjJ8+RIyQ=@vger.kernel.org X-Gm-Message-State: AOJu0Yy/kwULWyQeqi94kIY0L5UqOZu77ZIZc5vV+qW4Lyo0V0+NRrhl AG59fGMWkxXVgdjyVcXfG8nC2jYWW7m80iyqcNSflRlmZoM7ZsCaUxQ= X-Gm-Gg: AR+sD12gcqZm6CFPVd3rbHLCc9fR5lku288gt/jmjT2Iu7OHO0krNGisvsqF5cY+v1W j9XwsT09ZexOqihx3hDwUmD/PlHJHjEd5hoMXfkROmVtHU+hnMjisS33wZ21jkH3VnsomFZxesb e2++0eH9yI5ZXBLPuVHBoJ3yjO7nyNko8JZx4wgDz1zotkORtoM0SuWCF4Yb6bVhkWYbb7n2xFm Bu2mP83+bWUZh0hgoJk6tG6v2aMaQtdkazt2rCiI8abwt3sQTQVhLcIDF+3BNJHv+BoG7NKv/tJ JkciSdrh8qFdXQAcEanaVDA/DuK1K/EsHObBwnN91LPOWKE2eANEjDyt8LlONqvlaGROGh0PmnE bNfDfkMZ6MEoLRRV7DOdlmPDveh9o6Ls82fpuwEQb+qXz99LSHXdvoWZMIJwF8AJLGhxegQaVxS vhfHZBmw6moOE6qxB62I+f9I3TXffx/QL0b0zm5XXWgYAE9LYmPryoEp+70bP6aYnoCmHY8BVka X/wg0v0xwFwIRFYaOUaXDloBvlEwQxSUO+efoOPS19vYNbxxQntz5CfOwu3JZxO8ianx5Z4tXkp TlUO5TKTIMnBlQKRfzHzbfQ= X-Received: by 2002:a05:6a00:4b02:b0:84e:cd14:aeba with SMTP id d2e1a72fcca58-84ee48123a3mr269712b3a.6.1785514699546; Fri, 31 Jul 2026 09:18:19 -0700 (PDT) Received: from localhost.localdomain ([1.226.165.54]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-84edc29e6f0sm647072b3a.39.2026.07.31.09.18.17 (version=TLS1_3 cipher=TLS_CHACHA20_POLY1305_SHA256 bits=256/256); Fri, 31 Jul 2026 09:18:18 -0700 (PDT) From: Myeonghun Pak To: Greg Kroah-Hartman , Jiri Slaby Cc: linux-serial@vger.kernel.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org, Myeonghun Pak , Ijae Kim Subject: [PATCH 1/2] tty: moxa: unwind tty driver if PCI registration fails Date: Sat, 1 Aug 2026 01:18:08 +0900 Message-ID: <20260731161809.45066-1-mhun512@gmail.com> X-Mailer: git-send-email 2.50.1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit moxa_init() registers the tty driver before registering the PCI driver. If pci_register_driver() fails, module initialization returns without unregistering the tty driver or dropping its reference. moxa_exit() is not called after a failed module initialization. Unregister the tty driver and drop its reference before returning the PCI registration error. This issue was identified during our ongoing static-analysis research while reviewing kernel code. Fixes: a784bf7c195f ("[PATCH] Char: moxa, pci probing") Cc: stable@vger.kernel.org Co-developed-by: Ijae Kim Signed-off-by: Ijae Kim Signed-off-by: Myeonghun Pak --- drivers/tty/moxa.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/tty/moxa.c b/drivers/tty/moxa.c index 1bb2376af85c..680b2dd65462 100644 --- a/drivers/tty/moxa.c +++ b/drivers/tty/moxa.c @@ -1172,8 +1172,11 @@ static int __init moxa_init(void) } retval = pci_register_driver(&moxa_pci_driver); - if (retval) + if (retval) { printk(KERN_ERR "Can't register MOXA pci driver!\n"); + tty_unregister_driver(moxaDriver); + tty_driver_kref_put(moxaDriver); + } return retval; } -- 2.47.1