linux-kernel - Re: [PATCH 17/17] sound: Add missing spin

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <s5hiq694vgr.wl%tiwai@suse.de>
Date:	Thu, 27 May 2010 10:09:24 +0200
From:	Takashi Iwai <tiwai@...e.de>
To:	Julia Lawall <julia@...u.dk>
Cc:	Jaroslav Kysela <perex@...ex.cz>, linux-kernel@...r.kernel.org,
	alsa-devel@...a-project.org, kernel-janitors@...r.kernel.org
Subject: Re: [PATCH 17/17] sound: Add missing spin_unlock

At Wed, 26 May 2010 17:59:27 +0200 (CEST),
Julia Lawall wrote:
> 
> From: Julia Lawall <julia@...u.dk>
> 
> Add a spin_unlock missing on the error path.
> 
> The semantic match that finds this problem is as follows:
> (http://coccinelle.lip6.fr/)
> 
> // <smpl>
> @@
> expression E1;
> @@
> 
> * spin_lock(E1,...);
>   <+... when != E1
>   if (...) {
>     ... when != E1
> *   return ...;
>   }
>   ...+>
> * spin_unlock(E1,...);
> // </smpl>
> 
> Signed-off-by: Julia Lawall <julia@...u.dk>

Thanks, applied now.


Takashi

> ---
>  sound/mips/au1x00.c                 |    1 +
>  sound/oss/dmasound/dmasound_atari.c |    5 +++--
>  2 files changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/sound/mips/au1x00.c b/sound/mips/au1x00.c
> index 3e763d6..446cf97 100644
> --- a/sound/mips/au1x00.c
> +++ b/sound/mips/au1x00.c
> @@ -516,6 +516,7 @@ get the interrupt driven case to work efficiently */
>  			break;
>  	if (i == 0x5000) {
>  		printk(KERN_ERR "au1000 AC97: AC97 command read timeout\n");
> +		spin_unlock(&au1000->ac97_lock);
>  		return 0;
>  	}
>  
> diff --git a/sound/oss/dmasound/dmasound_atari.c b/sound/oss/dmasound/dmasound_atari.c
> index 1f47741..13c2144 100644
> --- a/sound/oss/dmasound/dmasound_atari.c
> +++ b/sound/oss/dmasound/dmasound_atari.c
> @@ -1277,7 +1277,7 @@ static irqreturn_t AtaInterrupt(int irq, void *dummy)
>  		 * (almost) like on the TT.
>  		 */
>  		write_sq_ignore_int = 0;
> -		return IRQ_HANDLED;
> +		goto out;
>  	}
>  
>  	if (!write_sq.active) {
> @@ -1285,7 +1285,7 @@ static irqreturn_t AtaInterrupt(int irq, void *dummy)
>  		 * the sq variables, so better don't do anything here.
>  		 */
>  		WAKE_UP(write_sq.sync_queue);
> -		return IRQ_HANDLED;
> +		goto out;
>  	}
>  
>  	/* Probably ;) one frame is finished. Well, in fact it may be that a
> @@ -1322,6 +1322,7 @@ static irqreturn_t AtaInterrupt(int irq, void *dummy)
>  	/* We are not playing after AtaPlay(), so there
>  	   is nothing to play any more. Wake up a process
>  	   waiting for audio output to drain. */
> +out:
>  	spin_unlock(&dmasound.lock);
>  	return IRQ_HANDLED;
>  }
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/