lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-Id: <20220227071253.28193-2-tangmeng@uniontech.com> Date: Sun, 27 Feb 2022 15:12:53 +0800 From: Meng Tang <tangmeng@...ontech.com> To: perex@...ex.cz, tiwai@...e.com Cc: alsa-devel@...a-project.org, linux-kernel@...r.kernel.org, Meng Tang <tangmeng@...ontech.com> Subject: [PATCH 2/2] ALSA: core: Remove redundant 'err' variable Return the result from file->f_op->open() directly instead of taking this in another redundant variable. Signed-off-by: Meng Tang <tangmeng@...ontech.com> --- sound/sound_core.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/sound/sound_core.c b/sound/sound_core.c index 2ee54070b2b5..e7fa820b08be 100644 --- a/sound/sound_core.c +++ b/sound/sound_core.c @@ -586,13 +586,12 @@ static int soundcore_open(struct inode *inode, struct file *file) * We rely upon the fact that we can't be unloaded while the * subdriver is there. */ - int err = 0; replace_fops(file, new_fops); if (file->f_op->open) - err = file->f_op->open(inode,file); + return file->f_op->open(inode, file); - return err; + return 0; } return -ENODEV; } -- 2.20.1
Powered by blists - more mailing lists