Page MenuHomeFreeBSD

libpfctl: Don't pass stack garbage to free.
ClosedPublic

Authored by jhb on Mar 22 2023, 2:57 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, May 24, 4:46 PM
Unknown Object (File)
Apr 28 2024, 3:15 PM
Unknown Object (File)
Apr 27 2024, 6:13 PM
Unknown Object (File)
Mar 12 2024, 10:18 PM
Unknown Object (File)
Mar 12 2024, 10:18 PM
Unknown Object (File)
Mar 8 2024, 8:19 AM
Unknown Object (File)
Jan 12 2024, 6:52 PM
Unknown Object (File)
Jan 4 2024, 6:44 AM
Subscribers

Details

Summary

GCC 9 on stable/12 reports a -Wmaybe-uninitialized error for the call
to free in _pfctl_clear_states.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jhb requested review of this revision.Mar 22 2023, 2:57 AM
mjg added inline comments.
lib/libpfctl/libpfctl.c
1191

but the var is unused to begin with? modulo the bogus free call

mjg added inline comments.
lib/libpfctl/libpfctl.c
1191

huh i somehow did not connect that you removed it altogether, apologies :)

This revision is now accepted and ready to land.Mar 22 2023, 4:57 AM
This revision was automatically updated to reflect the committed changes.