diff options
| author |   Qiu-ji Chen <chenqiuji666@gmail.com> | 2024-11-07 19:33:37 +0800 |
|---|---|---|
| committer |   Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2024-11-10 08:02:37 +0100 |
| commit | fe0ebeafc3b723b2f8edf27ecec6d353b08397df (patch) | |
| tree | 18623bffecd99d1ff7aef65eb642f1271f26e20a /scripts/basic/git: | |
| parent | staging: rtl8723bs: Remove unused function Efuse_GetCurrentSize (diff) | |
staging: greybus: uart: Fix atomicity violation in get_serial_info()
Our static checker found a bug where set_serial_info() uses a mutex, but
get_serial_info() does not. Fortunately, the impact of this is relatively
minor. It doesn't cause a crash or any other serious issues. However, if a
race condition occurs between set_serial_info() and get_serial_info(),
there is a chance that the data returned by get_serial_info() will be
meaningless.
Signed-off-by: Qiu-ji Chen <chenqiuji666@gmail.com>
Fixes: 0aad5ad563c8 ("greybus/uart: switch to ->[sg]et_serial()")
Reviewed-by: Johan Hovold <johan+linaro@kernel.org>
Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org>
Reviewed-by: Alex Elder <elder@riscstar.com>
Link: https://lore.kernel.org/r/20241107113337.402042-1-chenqiuji666@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'scripts/basic/git:')
0 files changed, 0 insertions, 0 deletions