aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorOliver Neukum <oneukum@suse.com>2016-04-04 08:30:53 -0400
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2016-04-13 14:53:07 -0400
commit2a14759a03a0a57b2fc99b1e416180788b8ba24c (patch)
tree0c7ddf79ac0ecf03dfd6ba25f8549dbf64fac276
parentbf16200689118d19de1b8d2a3c314fc21f5dc7bb (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.c4
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