Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

dbd_st_finish() does not check mysql_db_async_result() failure #315

Open
ppisar opened this issue Jul 25, 2019 · 2 comments
Open

dbd_st_finish() does not check mysql_db_async_result() failure #315

ppisar opened this issue Jul 25, 2019 · 2 comments
Labels
async prepared statements

Comments

@ppisar
Copy link
Contributor

ppisar commented Jul 25, 2019

dbd_st_finish() calls mysql_db_async_result() without checking its return value. mysql_db_async_result() can fail and it that case dbd_st_finish() should also signal an error.
@pali
Copy link
Member

pali commented Oct 14, 2019

Hi @ppisar, is this problem also in DBD::MariaDB?

@jplesnik
Copy link

Hi @pali , yes, it is also in DBD::MariaDB.

mariadb_st_finish() calls mariadb_db_async_result() without checking return value.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
async prepared statements
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@ppisar @dveeden @pali @jplesnik