php-firebird icon indicating copy to clipboard operation
php-firebird copied to clipboard

Fix and re-enable tests/004.phpt

Open Jon4t4n opened this issue 3 years ago • 2 comments

tests/004.phpt was disabled in PR #39. We want to fix this and re-enable the test.

Error:

Warning: main(): invalid statement handle  in .../tests/004.php on line 112

Jon4t4n avatar Dec 28 '22 07:12 Jon4t4n

Checked. ibase_free_result doesn't set pointer correct after freeing resource. Test passes after latest code release. See https://github.com/FirebirdSQL/php-firebird/commit/7a0a740048133abd26debc0ffe5f1652aebfdfbc

Please check it too.

MartinKoeditz avatar Jan 31 '23 16:01 MartinKoeditz

Hi, thanks for taking a look at this.

I haven't had time to dig into this, but the proposed fix makes the test pass with release builds. However, it causes failures with debug builds.

See this CI job (Commit 7a0a740048133abd26debc0ffe5f1652aebfdfbc): https://github.com/FirebirdSQL/php-firebird/actions/runs/4056103722/jobs/6980149414

Jon4t4n avatar Feb 04 '23 07:02 Jon4t4n