babelfish-for-postgresql / babelfish_extensions / 24330195217

77%

push

github

Fix Invalid Read in IsTopTransactionName (#4717)

SetTopTransactionName() stores the transaction name in TopTransactionContext
memory. When the transaction ends, this memory is freed but the name pointer
is not set to NULL, leaving a dangling pointer. A subsequent call to
IsTopTransactionName() then reads freed memory via strcmp().

Introduce ResetTopTransactionName() which NULLs out name. This is called from
pltsql_xact_cb to ensure the pointer is reset on every transaction end.

Engine PR: babelfish-for-postgresql/postgresql_modified_for_babelfish#739

Task: BABEL-5245

Authored-by: Rucha Kulkarni ruchask@amazon.com

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

53061 of 68841 relevant lines covered (77.08%)

637723.34 hits per line