Commit c0897656 authored by Gustavo A. R. Silva's avatar Gustavo A. R. Silva Committed by Greg Kroah-Hartman
Browse files

staging: r8188eu: Replace zero-length array with flexible-array member 

One-element and zero-length arrays are deprecated and should be
replaced with flexible-array members[1].

Replace zero-length array with flexible-array member and make use
of the struct_size() helper in kmalloc().

[1] https://www.kernel.org/doc/html/v5.10/process/deprecated.html#zero-length-and-one-element-arrays

Link: https://github.com/KSPP/linux/issues/78
Link: https://github.com/KSPP/linux/issues/160


Signed-off-by: default avatarGustavo A. R. Silva <gustavoars@kernel.org>
Link: https://lore.kernel.org/r/20210929193658.GA339070@embeddedor


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 403aa62d

drivers/staging/r8188eu/include/osdep_service.h

+1 −1
Original line number Diff line number Diff line
@@ -289,7 +289,7 @@ struct rtw_cbuf { 
	u32 write;

	u32 read;

	u32 size;

	void *bufs[0];

	void *bufs[];

};



bool rtw_cbuf_empty(struct rtw_cbuf *cbuf);

drivers/staging/r8188eu/os_dep/osdep_service.c

+1 −1
Original line number Diff line number Diff line
@@ -262,7 +262,7 @@ struct rtw_cbuf *rtw_cbuf_alloc(u32 size) 
{

	struct rtw_cbuf *cbuf;



	cbuf = kmalloc(sizeof(*cbuf) + sizeof(void *) * size, GFP_KERNEL);

	cbuf = kmalloc(struct_size(cbuf, bufs, size), GFP_KERNEL);



	if (cbuf) {

		cbuf->write = 0;