* [PATCH] Remove pointless code from ALSA emu10k1 midi driver.
@ 2004-09-01 15:51 Dave Jones
2004-09-01 18:06 ` Takashi Iwai
0 siblings, 1 reply; 2+ messages in thread
From: Dave Jones @ 2004-09-01 15:51 UTC (permalink / raw)
To: linux-kernel
Spotted with the source checker from Coverity.com.
Signed-off-by: Dave Jones <davej@redhat.com>
diff -urpN --exclude-from=/home/davej/.exclude bk-linus/sound/oss/emu10k1/midi.c linux-2.6/sound/oss/emu10k1/midi.c
--- bk-linus/sound/oss/emu10k1/midi.c 2004-08-09 13:12:18.000000000 +0100
+++ linux-2.6/sound/oss/emu10k1/midi.c 2004-08-23 14:08:26.000000000 +0100
@@ -320,7 +320,6 @@ static ssize_t emu10k1_midi_write(struct
{
struct emu10k1_mididevice *midi_dev = (struct emu10k1_mididevice *) file->private_data;
struct midi_hdr *midihdr;
- ssize_t ret = 0;
unsigned long flags;
DPD(4, "emu10k1_midi_write(), count=%#x\n", (u32) count);
@@ -344,7 +343,7 @@ static ssize_t emu10k1_midi_write(struct
if (copy_from_user(midihdr->data, buffer, count)) {
kfree(midihdr->data);
kfree(midihdr);
- return ret ? ret : -EFAULT;
+ return -EFAULT;
}
spin_lock_irqsave(&midi_spinlock, flags);
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] Remove pointless code from ALSA emu10k1 midi driver.
2004-09-01 15:51 [PATCH] Remove pointless code from ALSA emu10k1 midi driver Dave Jones
@ 2004-09-01 18:06 ` Takashi Iwai
0 siblings, 0 replies; 2+ messages in thread
From: Takashi Iwai @ 2004-09-01 18:06 UTC (permalink / raw)
To: Dave Jones; +Cc: linux-kernel
Well it's not ALSA emu10k1 but OSS ;)
Takashi
At Wed, 1 Sep 2004 16:51:22 +0100,
Dave Jones wrote:
>
> Spotted with the source checker from Coverity.com.
>
> Signed-off-by: Dave Jones <davej@redhat.com>
>
>
> diff -urpN --exclude-from=/home/davej/.exclude bk-linus/sound/oss/emu10k1/midi.c linux-2.6/sound/oss/emu10k1/midi.c
> --- bk-linus/sound/oss/emu10k1/midi.c 2004-08-09 13:12:18.000000000 +0100
> +++ linux-2.6/sound/oss/emu10k1/midi.c 2004-08-23 14:08:26.000000000 +0100
> @@ -320,7 +320,6 @@ static ssize_t emu10k1_midi_write(struct
> {
> struct emu10k1_mididevice *midi_dev = (struct emu10k1_mididevice *) file->private_data;
> struct midi_hdr *midihdr;
> - ssize_t ret = 0;
> unsigned long flags;
>
> DPD(4, "emu10k1_midi_write(), count=%#x\n", (u32) count);
> @@ -344,7 +343,7 @@ static ssize_t emu10k1_midi_write(struct
> if (copy_from_user(midihdr->data, buffer, count)) {
> kfree(midihdr->data);
> kfree(midihdr);
> - return ret ? ret : -EFAULT;
> + return -EFAULT;
> }
>
> spin_lock_irqsave(&midi_spinlock, flags);
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2004-09-01 18:09 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-09-01 15:51 [PATCH] Remove pointless code from ALSA emu10k1 midi driver Dave Jones
2004-09-01 18:06 ` Takashi Iwai
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.