diff options
| author | Sean Young <sean@mess.org> | 2017-05-16 04:56:14 -0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@s-opensource.com> | 2017-06-06 07:19:10 -0300 |
| commit | f8c627fbabbe9ed6ae68dcfefb7519bd153a7ac0 (patch) | |
| tree | 70d497fb0fea384db72f806c7db85a6ecb1d1469 /drivers/media | |
| parent | ae8eb443a17331a07579bc04817accaaaa62b78e (diff) | |
| download | op-kernel-dev-f8c627fbabbe9ed6ae68dcfefb7519bd153a7ac0.zip op-kernel-dev-f8c627fbabbe9ed6ae68dcfefb7519bd153a7ac0.tar.gz | |
[media] sir_ir: infinite loop in interrupt handler
Since this driver does no detection of hardware, it might be used with
a non-sir port. Escape out if we are spinning.
Reported-by: kbuild test robot <fengguang.wu@intel.com>
Signed-off-by: Sean Young <sean@mess.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
Diffstat (limited to 'drivers/media')
| -rw-r--r-- | drivers/media/rc/sir_ir.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/drivers/media/rc/sir_ir.c b/drivers/media/rc/sir_ir.c index e12ec50..90a5f8f 100644 --- a/drivers/media/rc/sir_ir.c +++ b/drivers/media/rc/sir_ir.c @@ -183,9 +183,15 @@ static irqreturn_t sir_interrupt(int irq, void *dev_id) static unsigned long delt; unsigned long deltintr; unsigned long flags; + int counter = 0; int iir, lsr; while ((iir = inb(io + UART_IIR) & UART_IIR_ID)) { + if (++counter > 256) { + dev_err(&sir_ir_dev->dev, "Trapped in interrupt"); + break; + } + switch (iir & UART_IIR_ID) { /* FIXME toto treba preriedit */ case UART_IIR_MSI: (void)inb(io + UART_MSR); |
