Skip to content

ext/pdo_pgsql: factorize clearing of running_stmt - #23917

Open
outtersg wants to merge 1 commit into
php:masterfrom
outtersg:pdopg_clear_runningstmt
Open

outtersg wants to merge 1 commit into
php:masterfrom
outtersg:pdopg_clear_runningstmt

Conversation

@outtersg

Copy link
Copy Markdown
Contributor

#23065 fixed running_stmt being only conditionally cleared in pgsql_stmt_finish(), by clearing it from the different callers of pgsql_stmt_finish(); put the clearing back into pgsql_stmt_finish(), albeit this time with the right condition to have all callers satisfied.

(this is part 1. of #23892 (comment))

phpGH-23065 fixed running_stmt being only conditionally cleared in pgsql_stmt_finish(), by clearing it from the different callers of pgsql_stmt_finish();
put the clearing back into pgsql_stmt_finish(), albeit this time with the right condition to have all callers satisfied.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant