From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f170.google.com (mail-pl1-f170.google.com [209.85.214.170]) (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 C11CD3749F6 for ; Thu, 2 Jul 2026 06:32:08 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.170 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782973930; cv=none; b=npvB6NMkTtGKh16UAlj5WOFC5Wpk/IGDu5bkKLbgQUtkfhafMXJgUB2hJ+HTpOAb7U9AtYVPNej/yNZsOiXAsc7mYfDPKwPQavDCPRHzOhP2ZH6qOhX02NcWkpqpCZNUukg537UbgM5H9YdRFtIJGyso5mAyJ/28gNeRHFoRc4k= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782973930; c=relaxed/simple; bh=ezSTqTTyBGFhigxAlLrLusnQsmbWlTjeDJA634A9ywM=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=D7/alq3HWh4hZjg2TCEwIlZtcy7bck01NgS6K172CUsavqba1rxfnyH9MGiqiw5ndXG5Jxk10ucudsG9OAOJV3MGEnqr21GXV8wPShSpXbPbjASw7eybJmMFDmNO7U4d6F+u4+Jh/ec55A2I/YxNgu8nf1x7TDr/TMqMFyYzfs0= 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=TszyZBfc; arc=none smtp.client-ip=209.85.214.170 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="TszyZBfc" Received: by mail-pl1-f170.google.com with SMTP id d9443c01a7336-2ca7aaa4b85so10092245ad.3 for ; Wed, 01 Jul 2026 23:32:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1782973928; x=1783578728; 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=oyuadsrLS81NHqHm0Hn0mdT4fJKqMPl6qVcybVPUQso=; b=TszyZBfcRuBW7xOFTDN1+Y8R0K8Boi95oOnhT014ISXFobsAhsM1WLuXMcEmivc5BZ YldRbvUq4i0bWn7/GnSSB7o+Uybh9ydo52lCzdlwmCVJcYDOsIUxn3W9/RiJVOXFji99 aC/gZ6rAKdyYqtHfeMKD2Kya6Wz/CTAN2pDXIGmb5elMJbsKxcUVd6lnnN4q5Gwhypzu lDfTJ1eaYI12F105M7cbGRbyStQew27OF1d6QTSkZWhyYTU1Dv8BH4LFaGNyRif8hiaR A8RQ4biwPci3xpswfxAK9iSh7swz8DfuZREe+ncjKeia7Qt0EA0DwhEFw0P9vSLyEgZn N3/Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1782973928; x=1783578728; 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=oyuadsrLS81NHqHm0Hn0mdT4fJKqMPl6qVcybVPUQso=; b=PZ+eb613e8/35eisF2RiHbpXXme2Ug+p4magyMNjw1+IktvSE2P4D8+HiqyUiarkII tVYYhxieH1Ii1IQptb5IVa0zvtqBM5C+T15DRv7/hdmIDN90uPUFjySMIVjkH65eU9OQ yDsxLeQ1ec2gO/2i9H+yD/vK06MFtPgt8UjbqKlS7N3P804CsBPMZsqwC3PxhZCwYqTV 65MRQVNEKpys8SVrAcqeu8JH501cc+07m6jxGaG615Bn41cBUEzqLmT3cLgXxZd/nQTr x0lynoKXLpyWIOJ82NwBAA0h0vAInA8BvUYnPiC6zwRrSgE4BgbmIlBQ9N3ziwOCyIWa 6DuQ== X-Forwarded-Encrypted: i=1; AHgh+Rqfk1SVFKCEI9EQu7f7NQJqwuFvYW9IL3OJI7p0PPt6ABda8s8LMB/xicfyYmw8HwhMmaZH4IphPWuL8mE=@vger.kernel.org X-Gm-Message-State: AOJu0YzVbkOV8/NIxlav11lPWoB/8nVty60N3KJ36Nthn98KMLY/RmKf y1OgsJv3EOTSshoaZxM3mdUQCjTqoe9T44wpyrLP01JOfhXSeCzvS8cJf2qYhg== X-Gm-Gg: AfdE7clw+StPOCmuIiBFQREKqQmbRVH2cKx/A0ZlouUC3wtc2hUb6rcR1vThlI5/NJ2 gn0gwdSBq3qurbRx04+Wmgdzi+CSA8sSL/t0dvZLDq1ZzTanoAGERTFHLfFAWzXpBPuBBdFI00/ 3xlLD7BqOAgz9fYHGRwOVhbz4OST49Rw+lr3FaS2k2m4QRfyIuynqZiVLSSTnJb+QIM4Cnq5nEy jLskCN3uHi2WA0XkR5vL71kIDU1FmFM6kZ49G8LSIDc94AEatcW08j4FyFzn/Z8U9xIdaVc3P45 Eg6h9a8mJ/vRUNU8VoUXox0GzqdnGDo+4bsZVqAx3ztoBAhOCLFq/ZRJwD2vp8fvBaot+QRscQ1 stJm8hfHjpu6IM3J50G8xo+Jte1ZNTAXklrAcEhi7/mh8xPAeqR5dMyQVHBo3FBY/JibizXwCSX PmGPShZSO0ZbXYNGblo+rSrS8Xj3sInEoKLl+bSZT0zRWRPCw= X-Received: by 2002:a17:902:f712:b0:2c9:e2c1:4fb5 with SMTP id d9443c01a7336-2ca911d7bdbmr33936035ad.22.1782973927785; Wed, 01 Jul 2026 23:32:07 -0700 (PDT) Received: from phuc-desktop.. ([183.91.15.56]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2ca9a8dec40sm8676435ad.5.2026.07.01.23.32.04 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 01 Jul 2026 23:32:07 -0700 (PDT) From: phucduc.bui@gmail.com To: Jaroslav Kysela , Takashi Iwai Cc: cassiogabrielcontato@gmail.com, Kees Cook , HariKrishna Sagala , Cen Zhang , Len Bao , HanQuan , Kyle Zeng , Dewei Meng , Maoyi Xie , linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org, bui duc phuc Subject: [PATCH] ALSA: core: Drop redundant stream_open() return check Date: Thu, 2 Jul 2026 13:31:53 +0700 Message-ID: <20260702063153.291425-1-phucduc.bui@gmail.com> X-Mailer: git-send-email 2.43.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: bui duc phuc The previous conversion from nonseekable_open() to stream_open() retained the existing error check. Since stream_open() always returns 0, remove the dead error handling path. Signed-off-by: bui duc phuc --- sound/core/rawmidi.c | 4 +--- sound/core/seq/seq_clientmgr.c | 5 +---- sound/core/timer.c | 5 +---- 3 files changed, 3 insertions(+), 11 deletions(-) diff --git a/sound/core/rawmidi.c b/sound/core/rawmidi.c index 4dfd9d53e6d3..789254a88e53 100644 --- a/sound/core/rawmidi.c +++ b/sound/core/rawmidi.c @@ -441,9 +441,7 @@ static int snd_rawmidi_open(struct inode *inode, struct file *file) if ((file->f_flags & O_APPEND) && !(file->f_flags & O_NONBLOCK)) return -EINVAL; /* invalid combination */ - err = stream_open(inode, file); - if (err < 0) - return err; + stream_open(inode, file); if (maj == snd_major) { rmidi = snd_lookup_minor_data(iminor(inode), diff --git a/sound/core/seq/seq_clientmgr.c b/sound/core/seq/seq_clientmgr.c index 28782e1776fa..8fe872367568 100644 --- a/sound/core/seq/seq_clientmgr.c +++ b/sound/core/seq/seq_clientmgr.c @@ -296,11 +296,8 @@ static int snd_seq_open(struct inode *inode, struct file *file) int c, mode; /* client id */ struct snd_seq_client *client; struct snd_seq_user_client *user; - int err; - err = stream_open(inode, file); - if (err < 0) - return err; + stream_open(inode, file); scoped_guard(mutex, ®ister_mutex) { client = seq_create_client1(-1, SNDRV_SEQ_DEFAULT_EVENTS); diff --git a/sound/core/timer.c b/sound/core/timer.c index 51c6ac4df9f4..937d7996f7ab 100644 --- a/sound/core/timer.c +++ b/sound/core/timer.c @@ -1531,11 +1531,8 @@ static int realloc_user_queue(struct snd_timer_user *tu, int size) static int snd_timer_user_open(struct inode *inode, struct file *file) { struct snd_timer_user *tu; - int err; - err = stream_open(inode, file); - if (err < 0) - return err; + stream_open(inode, file); tu = kzalloc_obj(*tu); if (tu == NULL) -- 2.43.0