diff mbox series

[next] ACPICA: iASL: Replace zero-length array with flexible-array member

Message ID 20220216203240.GA918517@embeddedor
State Accepted
Commit aa29b2083e11722a0ceedfa2a87de74cc7cfa676
Headers show
Series [next] ACPICA: iASL: Replace zero-length array with flexible-array member | expand

Commit Message

Gustavo A. R. Silva Feb. 16, 2022, 8:32 p.m. UTC
There is a regular need in the kernel to provide a way to declare
having a dynamically sized set of trailing elements in a structure.
Kernel code should always use “flexible array members”[1] for these
cases. The older style of one-element or zero-length arrays should
no longer be used[2].

[1] https://en.wikipedia.org/wiki/Flexible_array_member
[2] https://www.kernel.org/doc/html/v5.16/process/deprecated.html#zero-length-and-one-element-arrays

Link: https://github.com/KSPP/linux/issues/78
Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org>
---
 include/acpi/actbl2.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Gustavo A. R. Silva Feb. 18, 2022, 9:45 p.m. UTC | #1
On Fri, Feb 18, 2022 at 08:47:15PM +0100, Rafael J. Wysocki wrote:
> On Wed, Feb 16, 2022 at 9:36 PM Kees Cook <keescook@chromium.org> wrote:
> 
> I've created an upstream ACPICA pull request for this change, thanks!

Great.

Thanks, Rafael.
--
Gustavo
diff mbox series

Patch

diff --git a/include/acpi/actbl2.h b/include/acpi/actbl2.h
index 16847c8d9d5f..9ee4d1b39125 100644
--- a/include/acpi/actbl2.h
+++ b/include/acpi/actbl2.h
@@ -2319,7 +2319,7 @@  struct acpi_table_rgrt {
 	u16 version;
 	u8 image_type;
 	u8 reserved;
-	u8 image[0];
+	u8 image[];
 };
 
 /* image_type values */