Activity log
- Created & dismissed (max. allowed matches exceeded) suggestion
tty: serial: samsung: Remove redundant port lock acquisition in rx helpers
In the Linux kernel, the following vulnerability has been resolved: tty: serial: samsung: Remove redundant port lock acquisition in rx helpers Sashiko identified a deadlock when the console flow is engaged [1]. When console flow control is enabled (UPF_CONS_FLOW), s3c24xx_serial_stop_tx() calls s3c24xx_serial_rx_enable() and s3c24xx_serial_start_tx() calls s3c24xx_serial_rx_disable(). The serial core framework invokes the .stop_tx() and .start_tx() callbacks with the port->lock spinlock already held. Furthermore, all internal driver paths that invoke stop_tx (such as the DMA TX completion handler s3c24xx_serial_tx_dma_complete() or the PIO TX IRQ handler s3c24xx_serial_tx_irq()) also acquire port->lock prior to calling it. (Note that s3c24xx_serial_start_tx() is only invoked by the serial core). However, s3c24xx_serial_rx_enable() and s3c24xx_serial_rx_disable() unconditionally attempt to acquire port->lock again using uart_port_lock_irqsave(). Since spinlocks are not recursive, this causes a deadlock on the same CPU when console flow control is engaged. Remove the redundant lock acquisition from both rx helper functions.
References
Affected products
- <14143ec10d69f42806b5d7b046f0fd1b835831ae
- =<6.18.*
- <ee9eb72be95490602c493db050c73d925c3a4d74
- <a3bb136bff5e6a5e48cdd813246c9c4686feaaa9
- =<7.0.*
- <9c92b42207978559e32903c3098aaf5c5b5788b2
- =<6.12.*
- <2.6.27
- <f4c3e63fa8639aedf96fb200d9939945a9eed51e
- ==2.6.27
- =<6.1.*
- =<*
- =<6.6.*
- <9fd48937046efc9abb89379d63ee9cc5c661d711
- =<5.15.*
- <a9c22e0f93ba18322a6623ecdda2f0cd858ca350
- <10014eb7eee351f7b587f8ac85830f0c9343cb9a
- =<5.10.*