gregfelice commented on PR #2410: URL: https://github.com/apache/age/pull/2410#issuecomment-4860104587
Thanks for catching and reporting this, @sfc-gh-dachristensen — the stack overflow is real and query-reachable, exactly as you identified. We've opened #2451 to land the fix for the 1.8.0 release. It takes a slightly different end state: since the element is already a `float8`, it drops the `sprintf`/reparse entirely and assigns the value directly. That resolves the overflow **and** a latent precision-loss bug (`"%f"` truncates to 6 fractional digits), and removes the need for a fixed buffer at all. It also adds regression coverage, which was the main thing blocking this from merging as-is. Your original report and fix are credited via `Co-authored-by` on that PR. Closing this one as superseded by #2451 — thank you again for the find. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
