Commit f9785593 authored by Dave Jones's avatar Dave Jones Committed by Linus Torvalds

[PATCH] Remove pointless code from ALSA emu10k1 midi driver.

Spotted with the source checker from Coverity.com.
Signed-off-by: default avatarDave Jones <davej@redhat.com>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 06e0d8ad
...@@ -320,7 +320,6 @@ static ssize_t emu10k1_midi_write(struct file *file, const char __user *buffer, ...@@ -320,7 +320,6 @@ static ssize_t emu10k1_midi_write(struct file *file, const char __user *buffer,
{ {
struct emu10k1_mididevice *midi_dev = (struct emu10k1_mididevice *) file->private_data; struct emu10k1_mididevice *midi_dev = (struct emu10k1_mididevice *) file->private_data;
struct midi_hdr *midihdr; struct midi_hdr *midihdr;
ssize_t ret = 0;
unsigned long flags; unsigned long flags;
DPD(4, "emu10k1_midi_write(), count=%#x\n", (u32) count); DPD(4, "emu10k1_midi_write(), count=%#x\n", (u32) count);
...@@ -344,7 +343,7 @@ static ssize_t emu10k1_midi_write(struct file *file, const char __user *buffer, ...@@ -344,7 +343,7 @@ static ssize_t emu10k1_midi_write(struct file *file, const char __user *buffer,
if (copy_from_user(midihdr->data, buffer, count)) { if (copy_from_user(midihdr->data, buffer, count)) {
kfree(midihdr->data); kfree(midihdr->data);
kfree(midihdr); kfree(midihdr);
return ret ? ret : -EFAULT; return -EFAULT;
} }
spin_lock_irqsave(&midi_spinlock, flags); spin_lock_irqsave(&midi_spinlock, flags);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment