On Fri, Sep 22, 2023 at 10:52:03AM -0700, Kees Cook wrote: > 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 linereq. > Additionally, since the element count member must be set before accessing > the annotated flexible array member, move its initialization earlier. Ah, hadn't taken into account in my patch, so, Bart, feel free to ignore mine and apply this one. Reviewed-by: Andy Shevchenko <andy@xxxxxxxxxx> -- With Best Regards, Andy Shevchenko
Re: [PATCH] gpiolib: cdev: Annotate struct linereq with __counted_by
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [PATCH] gpiolib: cdev: Annotate struct linereq with __counted_by
- From: Andy Shevchenko <andy@xxxxxxxxxx>
- Date: Mon, 25 Sep 2023 09:53:59 +0300
- In-reply-to: <20230922175203.work.760-kees@kernel.org>
- Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo
- References: <20230922175203.work.760-kees@kernel.org>
- References:
- [PATCH] gpiolib: cdev: Annotate struct linereq with __counted_by
- From: Kees Cook
- [PATCH] gpiolib: cdev: Annotate struct linereq with __counted_by
- Prev by Date: Re: [PATCH v2 2/2] gpio: sim: initialize a managed pointer when declaring it
- Next by Date: Re: [PATCH v1 1/1] gpiolib: cdev: Annotate struct linereq with __counted_by()
- Previous by thread: Re: [PATCH] gpiolib: cdev: Annotate struct linereq with __counted_by
- Next by thread: Re: [PATCH] gpiolib: cdev: Annotate struct linereq with __counted_by
- Index(es):
 |