diff options
author | Will McVicker <willmcvicker@google.com> | 2022-05-03 10:56:23 -0700 |
---|---|---|
committer | Will McVicker <willmcvicker@google.com> | 2022-05-03 11:08:42 -0700 |
commit | f744666b6e2e7b3602da2d0e262a10e8b183d552 (patch) | |
tree | 7bc57c0e723876041da592c7d851f1827e629478 | |
parent | 407b433e06df2afe3eda9541a202095f7b7e78af (diff) | |
download | uwb-f744666b6e2e7b3602da2d0e262a10e8b183d552.tar.gz |
dw3000: fix memset warningandroid-t-beta-4_r0.6android-t-beta-4_r0.5android-t-beta-4_r0.4android-t-beta-3_r0.4android-t-beta-3.3_r0.4android-t-beta-3.2_r0.4android-13.0.0_r0.5android-13.0.0_r0.45android-13.0.0_r0.44android-13.0.0_r0.43android-13.0.0_r0.4android-13.0.0_r0.32android-13.0.0_r0.31android-13.0.0_r0.20android-13.0.0_r0.19android-13.0.0_r0.15android-13.0.0_r0.14android-gs-raviole-5.10-t-beta-4android-gs-raviole-5.10-t-beta-3android-gs-raviole-5.10-android13android-gs-pantah-5.10-android13-d1android-gs-bluejay-5.10-t-beta-4android-gs-bluejay-5.10-android13
The below warning happens when we build with LTO=none. Fix this by
dropping the memset and init the local variable on declaration.
common/include/linux/fortify-string.h:242:4: error: call to
__write_overflow_field declared with 'warning' attribute: detected write
beyond size of field (1st parameter); maybe use struct_group()?
[-Werror,-Wattribute-warning]
__write_overflow_field(p_size_field, size);
^
1 error generated.
Bug: 231318697
Signed-off-by: Will McVicker <willmcvicker@google.com>
Change-Id: I1b118229904c1a9327a0b4b8744a039317514512
-rw-r--r-- | kernel/drivers/net/ieee802154/dw3000_core.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/kernel/drivers/net/ieee802154/dw3000_core.c b/kernel/drivers/net/ieee802154/dw3000_core.c index fb4afef..6905c3c 100644 --- a/kernel/drivers/net/ieee802154/dw3000_core.c +++ b/kernel/drivers/net/ieee802154/dw3000_core.c @@ -894,10 +894,9 @@ int dw3000_xfer(struct dw3000 *dw, u32 reg_fileid, u16 reg_offset, u16 length, struct spi_transfer header; struct spi_transfer data; u8 header_buf[2]; - } xfer; + } xfer = {}; /* Init transfers first because spi_message_init_with_transfer don't! */ - memset(&xfer.header, 0, sizeof(xfer.header) * 2); xfer.header.tx_buf = xfer.header_buf; xfer.header.len = sizeof(xfer.header_buf); |