Skip to content

Changed the test expected result of `pdo_mysql/bug76815 to %d - #13808

Merged
kamil-tekiela merged 1 commit into
php:masterfrom
grooverdan:bug76815_mariadb
Oct 9, 2026
Merged

kamil-tekiela merged 1 commit into
php:masterfrom
grooverdan:bug76815_mariadb

Conversation

@grooverdan

Copy link
Copy Markdown
Contributor

MariaDB was reporting row 0 instead of 1.

Example test run: https://buildbot.mariadb.org/#/builders/237/builds/6717/steps/7/logs/stdio

…o %d

MariaDB was reporting row 0 instead of 1.
grooverdan added a commit to MariaDB/buildbot that referenced this pull request Mar 26, 2024
@SakiTakamachi

Copy link
Copy Markdown
Member

Thanks for working on this, but there's already a PR.

#13500

If the SSL issue is resolved, I can merge it...

@grooverdan

Copy link
Copy Markdown
Contributor Author

Ack, are you waiting for assistance on anything explicitly that I can help with?

@SakiTakamachi

Copy link
Copy Markdown
Member

@grooverdan

If you don't mind, I would be very grateful if you could check that out.

Using 11.4-rc was not enough.

I was planning to try and figure out how to set up SSL over the weekend, but it would be very helpful if someone knowledgeable in MariaDB could check it for me.

@ndossche
ndossche requested a review from kamil-tekiela as a code owner March 14, 2026 14:04

@kamil-tekiela kamil-tekiela left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had some time to look into this now and I found out that this is an intentional change in MariaDB 10.7. https://jira.mariadb.org/browse/MDEV-26635 and https://jira.mariadb.org/browse/MDEV-10075

So apparently, we cannot check that the row matches anymore, which in this case makes sense as the 1/0 value is just a placeholder due to the error message being a generic error.

@kamil-tekiela
kamil-tekiela merged commit a5e3dee into php:master Oct 9, 2026
arnaud-lb added a commit to frodeborli/php-src that referenced this pull request Oct 9, 2026
* up/master: (180 commits)
  Changed the test expected result of `pdo_mysql/bug76815_pdo_mysql_f to %d (php#13808)
  ext/standard: name the real parameter in the unpack() offset error (php#24215)
  ext/readline: Refactor CLI readline completion generators
  Fix phpGH-24081: User opcode DISPATCH runs on a stale frame in the TAILCALL VM
  ext/standard: Validate the bcrypt cost before reading it
  JIT: Avoid object type check if the object is known to be a type (php#24086)
  zend_alloc: move a small block shrunk to the size of the bin below
  Fix phpGH-23979: Nullsafe operator must not flush delayed oplines of an enclosing function
  ext/zip: Reject ZipArchive mutators during close() (php#24025)
  Fix OSS-Fuzz #568005340: FETCH_DIM_FUNC_ARG partial conversion
  Fix too wide type inference for ASSIGN_DIM_OP
  Fix type inference of ADD_ARRAY_UNPACK with integer keys
  Evaluate ZEND_SPACESHIP in SCCP
  Add range inference for SPACESHIP
  JIT: Optimize array checks in comparisons (php#24084)
  Fix leak when the added previous exception is already in the chain (php#24177)
  date: Add `php_date_time_duration_create()` in a new `time_duration.h` (php#24072)
  ext/zip: Fix use-after-free in the archive destructor path (php#23779)
  ext/standard: Optimize array_chunk() by filling packed chunks directly
  Fix phpGH-17626: JIT corrupts opline handler when blacklisting root trace
  ...
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.

3 participants