babelfish-for-postgresql / babelfish_extensions / 18590213208

77%

Build Type

push

github

Committed by web-flow

Commit Message

Fixing fluctuating errors and crash during self assignment of parameters. (#4190)

Problem: Previously, during variable self-assignment (e.g., SELECT @var = @var), Babelfish reused the same Datum reference for both source and target without creating a copy. This caused memory corruption, use-after-free errors, and crashes due to unsafe sharing across memory contexts.

Root Cause: The source and target pointed to the same memory location, which could be invalidated by context switches or garbage collection.

Solution: Introduced safe duplication using datumCopy() to ensure each variable has its own memory allocation in the correct context (estate->datum_context). This preserves data integrity and prevents invalid memory access.

Issues Resolved
BABEL-6119, BABEL-5947

Signed-off-by: Tanya Gupta <tanyagp@amazon.com>
Co-authored-by: Tanya Gupta <tanyagp@amazon.com>

Run Details

3 of 3 new or added lines in 1 file covered. (100.0%)

2 existing lines in 1 file now uncovered.

51732 of 67501 relevant lines covered (76.64%)

379302.59 hits per line