Messages in this thread | | | Date | Thu, 9 Dec 2021 06:50:43 -0800 | Subject | Re: [PATCH] scsi: lpfc: Use struct_group() to initialize struct lpfc_cgn_info | From | James Smart <> |
| |
On 12/8/2021 11:59 AM, Kees Cook wrote: > In preparation for FORTIFY_SOURCE performing compile-time and run-time > field bounds checking for memset(), avoid intentionally writing across > neighboring fields. > > Add struct_group() to mark "stat" region of struct lpfc_cgn_info that > should be initialized to zero, and refactor the "data" region memset() > to wipe everything up to the cgn_stats region. > > Signed-off-by: Kees Cook <keescook@chromium.org> > ---
looks fine.
Reviewed-by: James Smart <jsmart2021@gmail.com>
-- james
| |