Commit 0b5dfa35 authored by Kees Cook's avatar Kees Cook Committed by Jakub Kicinski

ipv6: ioam: Replace 0-length array with flexible array

Zero-length arrays are deprecated[1]. Replace struct ioam6_trace_hdr's
"data" 0-length array with a flexible array. Detected with GCC 13,
using -fstrict-flex-arrays=3:

net/ipv6/ioam6_iptunnel.c: In function 'ioam6_build_state':
net/ipv6/ioam6_iptunnel.c:194:37: warning: array subscript <unknown> is outside array bounds of '__u8[0]' {aka 'unsigned char[]'} [-Warray-bounds=]
  194 |                 tuninfo->traceh.data[trace->remlen * 4] = IPV6_TLV_PADN;
      |                 ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~
In file included from include/linux/ioam6.h:11,
                 from net/ipv6/ioam6_iptunnel.c:13:
include/uapi/linux/ioam6.h:130:17: note: while referencing 'data'
  130 |         __u8    data[0];
      |                 ^~~~

[1] https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arraysSigned-off-by: default avatarKees Cook <keescook@chromium.org>
Reviewed-by: default avatarGustavo A. R. Silva <gustavoars@kernel.org>
Reviewed-by: default avatarJustin Iurman <justin.iurman@uliege.be>
Tested-by: default avatarJustin Iurman <justin.iurman@uliege.be>
Link: https://lore.kernel.org/r/20230105222115.never.661-kees@kernel.orgSigned-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 6bd4755c
...@@ -127,7 +127,7 @@ struct ioam6_trace_hdr { ...@@ -127,7 +127,7 @@ struct ioam6_trace_hdr {
#endif #endif
#define IOAM6_TRACE_DATA_SIZE_MAX 244 #define IOAM6_TRACE_DATA_SIZE_MAX 244
__u8 data[0]; __u8 data[];
} __attribute__((packed)); } __attribute__((packed));
#endif /* _UAPI_LINUX_IOAM6_H */ #endif /* _UAPI_LINUX_IOAM6_H */
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment