diff options
| author | Johan Hovold <johan@kernel.org> | 2019-11-28 18:22:01 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2020-02-05 14:18:09 +0000 |
| commit | c662ea4fab81c530590c2be3e86e26313d047e3b (patch) | |
| tree | cde1c7cf6b509e01bcef98f52fcf3dfb469b5fff /security/min_addr.c | |
| parent | b4cdf5066ce23d1cc23c1dd4c71438e762c82581 (diff) | |
rsi: fix use-after-free on probe errors
commit 92aafe77123ab478e5f5095878856ab0424910da upstream.
The driver would fail to stop the command timer in most error paths,
something which specifically could lead to the timer being freed while
still active on I/O errors during probe.
Fix this by making sure that each function starting the timer also stops
it in all relevant error paths.
Reported-by: syzbot+1d1597a5aa3679c65b9f@syzkaller.appspotmail.com
Fixes: b78e91bcfb33 ("rsi: Add new firmware loading method")
Cc: stable <stable@vger.kernel.org> # 4.12
Cc: Prameela Rani Garnepudi <prameela.j04cs@gmail.com>
Cc: Amitkumar Karwar <amit.karwar@redpinesignals.com>
Signed-off-by: Johan Hovold <johan@kernel.org>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'security/min_addr.c')
0 files changed, 0 insertions, 0 deletions