diff options
| author | Kees Cook <keescook@chromium.org> | 2023-09-22 10:54:02 -0700 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2023-09-22 19:01:38 +0100 |
| commit | 6e6891ccf0c7e70f13bc1d5ce5a7edee2b5de045 (patch) | |
| tree | 6eb3c33e64ed26df773012278136be1a88d8b239 | |
| parent | dd35a4debcf917f069e83f60b6ac84b5cad6e5e3 (diff) | |
| download | lwn-6e6891ccf0c7e70f13bc1d5ce5a7edee2b5de045.tar.gz lwn-6e6891ccf0c7e70f13bc1d5ce5a7edee2b5de045.zip | |
regulator: mc13xxx: Annotate struct mc13xxx_regulator_priv with __counted_by
Prepare for the coming implementation by GCC and Clang of the __counted_by
attribute. Flexible array members annotated with __counted_by can have
their accesses bounds-checked at run-time checking via CONFIG_UBSAN_BOUNDS
(for array indexing) and CONFIG_FORTIFY_SOURCE (for strcpy/memcpy-family
functions).
As found with Coccinelle[1], add __counted_by for struct mc13xxx_regulator_priv.
[1] https://github.com/kees/kernel-tools/blob/trunk/coccinelle/examples/counted_by.cocci
Cc: Liam Girdwood <lgirdwood@gmail.com>
Cc: Mark Brown <broonie@kernel.org>
Signed-off-by: Kees Cook <keescook@chromium.org>
Link: https://lore.kernel.org/r/20230922175402.work.819-kees@kernel.org
Signed-off-by: Mark Brown <broonie@kernel.org>
| -rw-r--r-- | drivers/regulator/mc13xxx.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/regulator/mc13xxx.h b/drivers/regulator/mc13xxx.h index e03279dc43f4..057aaef6f086 100644 --- a/drivers/regulator/mc13xxx.h +++ b/drivers/regulator/mc13xxx.h @@ -24,7 +24,7 @@ struct mc13xxx_regulator_priv { u32 powermisc_pwgt_state; struct mc13xxx_regulator *mc13xxx_regulators; int num_regulators; - struct regulator_dev *regulators[]; + struct regulator_dev *regulators[] __counted_by(num_regulators); }; extern int mc13xxx_fixed_regulator_set_voltage(struct regulator_dev *rdev, |