On Sat, Jun 27, 2026 at 01:25:39PM +0800, jian he wrote: > That mistake is harmless now, but if somebody use > LOCAL_FCINFO(fcinfo, 1), then it may have potential problems. > So let's try to use InputFunctionCallSafe instead, see attached.
It's an overallocation, harmless still incorrect. This also exists on REL_18_STABLE; this code has been moved on HEAD to its current location from atttribute_stats.c. > I found this while trying to use InputFunctionCallSafe inside array_in_safe. > Should we consider fully refactoring array_in_safe to use it too? Yep, let's do this cleanup in array_in_safe() as well. I was wondering about doing that only once v20 opens up, but array_in_safe() is new in v19. So let's just make the code right now rather than create an inconsistency before v19 gets released. Thanks for the report. Will do something for both spots. -- Michael
signature.asc
Description: PGP signature
