On Tue, Oct 27, 2020 at 02:07:04PM -0400, Sean Anderson wrote:
> 
> On 10/27/20 1:45 PM, Tom Rini wrote:
> > On Sat, Oct 17, 2020 at 02:07:29PM -0400, Sean Anderson wrote:
> > 
> >> Without every category between LOGC_NONE and LOGC_COUNT present in
> >> log_cat_name, log_get_cat_by_name will dereference NULL pointers if it
> >> doesn't find a name early enough.
> >>
> >> Fixes: c3aed5db59 ("sandbox: spi: Add more logging")
> >> Fixes: a5c13b68e7 ("sandbox: log: Add a category for sandbox")
> >> Fixes: 9f407d4ef0 ("Add core support for a bloblist to convey data from 
> >> SPL")
> >> Fixes: cce61fc428 ("dm: devres: Convert to use logging")
> >> Fixes: 7ca2850cbc ("dm: core: Add basic ACPI support")
> >>
> >> Signed-off-by: Sean Anderson <sean...@gmail.com>
> >> Reviewed-by: Simon Glass <s...@chromium.org>
> > 
> > As Simon has added the missing categories I'm deferring this specific
> > patch but think it should be re-posted and reworded with the build-time
> > sanity check as the only change.
> > 
> 
> I think Heinrich posted a patch for adding a sanity check [1], so this
> patch can probably be dropped if that is merged.

That's in my queue as well right now so, OK, thanks.

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to