diff options
author | Purva Yeshi <purvayeshi550@gmail.com> | 2025-04-10 16:04:42 +0530 |
---|---|---|
committer | Jarkko Sakkinen <jarkko@kernel.org> | 2025-05-15 04:47:14 +0300 |
commit | 32d495b384a2db7d23c2295e03e6b6edb1c0db8d (patch) | |
tree | 637b459e20b8457cd5c7f9ba193eda95359c8a07 /lib/ucs2_string.c | |
parent | 539fbab37881e32ba6a708a100de6db19e1e7e7d (diff) |
char: tpm: tpm-buf: Add sanity check fallback in read helpers
Fix Smatch-detected issue:
drivers/char/tpm/tpm-buf.c:208 tpm_buf_read_u8() error:
uninitialized symbol 'value'.
drivers/char/tpm/tpm-buf.c:225 tpm_buf_read_u16() error:
uninitialized symbol 'value'.
drivers/char/tpm/tpm-buf.c:242 tpm_buf_read_u32() error:
uninitialized symbol 'value'.
Zero-initialize the return values in tpm_buf_read_u8(), tpm_buf_read_u16(),
and tpm_buf_read_u32() to guard against uninitialized data in case of a
boundary overflow.
Add defensive initialization ensures the return values are always defined,
preventing undefined behavior if the unexpected happens.
Signed-off-by: Purva Yeshi <purvayeshi550@gmail.com>
Reviewed-by: Stefano Garzarella <sgarzare@redhat.com>
Reviewed-by: Jarkko Sakkinen <jarkko@kernel.org>
Signed-off-by: Jarkko Sakkinen <jarkko@kernel.org>
Diffstat (limited to 'lib/ucs2_string.c')
0 files changed, 0 insertions, 0 deletions