Lukas Wunner 8b4846ac1a serial: 8250: Avoid error message on reprobe ... commit e0a851fe6b upstream. If the call to uart_add_one_port() in serial8250_register_8250_port() fails, a half-initialized entry in the serial_8250ports[] array is left behind. A subsequent reprobe of the same serial port causes that entry to be reused. Because uart->port.dev is set, uart_remove_one_port() is called for the half-initialized entry and bails out with an error message: bcm2835-aux-uart 3f215040.serial: Removing wrong port: (null) != (ptrval) The same happens on failure of mctrl_gpio_init() since commit 4a96895f74 ("tty/serial/8250: use mctrl_gpio helpers"). Fix by zeroing the uart->port.dev pointer in the probe error path. The bug was introduced in v2.6.10 by historical commit befff6f5bf5f ("[SERIAL] Add new port registration/unregistration functions."): https://git.kernel.org/tglx/history/c/befff6f5bf5f The commit added an unconditional call to uart_remove_one_port() in serial8250_register_port(). In v3.7, commit 835d844d1a ("8250_pnp: do pnp probe before legacy probe") made that call conditional on uart->port.dev which allows me to fix the issue by zeroing that pointer in the error path. Thus, the present commit will fix the problem as far back as v3.7 whereas still older versions need to also cherry-pick 835d844d1a. Fixes: 835d844d1a ("8250_pnp: do pnp probe before legacy probe") Signed-off-by: Lukas Wunner <lukas@wunner.de> Cc: stable@vger.kernel.org # v2.6.10 Cc: stable@vger.kernel.org # v2.6.10: 835d844d1a28: 8250_pnp: do pnp probe before legacy Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Link: https://lore.kernel.org/r/b4a072013ee1a1d13ee06b4325afb19bda57ca1b.1589285873.git.lukas@wunner.de [iwamatsu: Backported to 4.14, 4.19: adjust context] Signed-off-by: Nobuhiro Iwamatsu (CIP) <nobuhiro1.iwamatsu@toshiba.co.jp> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>		2020-09-26 18:01:32 +02:00
..
hvc	Revert "tty: hvc: Fix data abort due to race in hvc_open"	2020-06-30 23:17:19 -04:00
ipwireless	tty: ipwireless: Fix potential NULL pointer dereference	2020-01-27 14:50:16 +01:00
serdev	serdev: ttyport: restore client ops on deregistration	2020-02-28 16:38:49 +01:00
serial	serial: 8250: Avoid error message on reprobe	2020-09-26 18:01:32 +02:00
vt	vt_ioctl: change VT_RESIZEX ioctl to check for error return from vc_resize()	2020-09-03 11:24:27 +02:00
amiserial.c
cyclades.c
ehv_bytechan.c	tty: evh_bytechan: Fix out of bounds accesses	2020-04-23 10:30:23 +02:00
goldfish.c
isicom.c
Kconfig
Makefile
mips_ejtag_fdc.c
moxa.c
moxa.h
mxser.c
mxser.h
n_gsm.c	tty: n_gsm: Fix bogus i++ in gsm_data_kick	2020-06-25 15:32:57 +02:00
n_hdlc.c	tty: Don't block on IO when ldisc change is pending	2019-12-13 08:52:28 +01:00
n_null.c
n_r3964.c	tty: Don't block on IO when ldisc change is pending	2019-12-13 08:52:28 +01:00
n_tracerouter.c
n_tracesink.c
n_tracesink.h
n_tty.c	tty: Don't block on IO when ldisc change is pending	2019-12-13 08:52:28 +01:00
nozomi.c
pty.c
rocket_int.h
rocket.c	tty: rocket, avoid OOB access	2020-04-29 16:31:30 +02:00
rocket.h
synclink_gt.c	tty: synclink_gt: Adjust indentation in several functions	2020-02-24 08:34:47 +01:00
synclink.c
synclinkmp.c	tty: synclinkmp: Adjust indentation in several functions	2020-02-24 08:34:47 +01:00
sysrq.c	sysrq: Remove duplicated sysrq message	2020-03-05 16:42:15 +01:00
tty_audit.c
tty_baudrate.c
tty_buffer.c
tty_io.c
tty_ioctl.c
tty_jobctrl.c
tty_ldisc.c	tty: Don't block on IO when ldisc change is pending	2019-12-13 08:52:28 +01:00
tty_ldsem.c
tty_mutex.c
tty_port.c	serdev: ttyport: restore client ops on deregistration	2020-02-28 16:38:49 +01:00
vcc.c