Bug #54401 assert in Diagnostics_area::set_eof_status , HANDLER

This assert checks that the server does not try to send EOF to the
client if there has been some error during processing. This to make
sure that the error is in fact sent to the client.

The problem was that any errors during processing of WHERE conditions
in HANDLER ... READ statements where not detected by the handler code.
The handler code therefore still tried to send EOF to the client,
triggering the assert. The bug was only noticeable in debug builds.

This patch fixes the problem by making sure that the handler code
checks for errors during condition processing and acts accordingly.

1 files changed, 4 insertions, 0 deletions

diff --git a/sql/sql_handler.cc b/sql/sql_handler.cc
index d6f2a472e05..e49da2b0423 100644
--- a/sql/sql_handler.cc
+++ b/sql/sql_handler.cc
@@ -747,7 +747,11 @@ retry:
       goto ok;
     }
     if (cond && !cond->val_int())
+    {
+      if (thd->is_error())
+        goto err;
       continue;
+    }
     if (num_rows >= offset_limit_cnt)
     {
       protocol->prepare_for_resend();