pf_find_state_all() expects the caller to initialize "*more" if it is
non-NULL.
Reported by: KMSAN
Differential D47405
pf: Fix a use of an uninitialized variable markj on Nov 2 2024, 7:58 PM. Authored by Tags None Referenced Files
Details
pf_find_state_all() expects the caller to initialize "*more" if it is Reported by: KMSAN
Diff Detail
Event TimelineComment Actions On reflection, I think we should fix this by having pf_find_staye_all() initialise *more to 0, because if we don’t find anything that’s the correct thing to return. Comment Actions Yeah, I was just about to make that change. I think all callers should initialize *more = 0 to avoid extra diffs with OpenBSD, but pf_find_state_all() should also be careful to initialize *more. |