diff options
author | Oliver Neukum <oneukum@suse.com> | 2016-04-04 08:30:53 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2016-04-13 14:53:07 -0400 |
commit | 2a14759a03a0a57b2fc99b1e416180788b8ba24c (patch) | |
tree | 0c7ddf79ac0ecf03dfd6ba25f8549dbf64fac276 | |
parent | bf16200689118d19de1b8d2a3c314fc21f5dc7bb (diff) |
cdc-acm: fix crash if flushed with nothing buffered
Under some circumstances acm_tty_flush_chars() is called
with no buffer to flush. We simply need to do nothing.
Signed-off-by: Oliver Neukum <ONeukum@suse.com>
Reported-by: Torsten Hilbrich <torsten.hilbrich@secunet.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | drivers/usb/class/cdc-acm.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/usb/class/cdc-acm.c b/drivers/usb/class/cdc-acm.c index 83fd30b0577c..a6c4a1b895bd 100644 --- a/drivers/usb/class/cdc-acm.c +++ b/drivers/usb/class/cdc-acm.c | |||
@@ -744,11 +744,15 @@ static void acm_tty_flush_chars(struct tty_struct *tty) | |||
744 | int err; | 744 | int err; |
745 | unsigned long flags; | 745 | unsigned long flags; |
746 | 746 | ||
747 | if (!cur) /* nothing to do */ | ||
748 | return; | ||
749 | |||
747 | acm->putbuffer = NULL; | 750 | acm->putbuffer = NULL; |
748 | err = usb_autopm_get_interface_async(acm->control); | 751 | err = usb_autopm_get_interface_async(acm->control); |
749 | spin_lock_irqsave(&acm->write_lock, flags); | 752 | spin_lock_irqsave(&acm->write_lock, flags); |
750 | if (err < 0) { | 753 | if (err < 0) { |
751 | cur->use = 0; | 754 | cur->use = 0; |
755 | acm->putbuffer = cur; | ||
752 | goto out; | 756 | goto out; |
753 | } | 757 | } |
754 | 758 | ||