Commit 11c85e1d authored by Alexey Botchkov's avatar Alexey Botchkov

MDEV-5273 Prepared statement doesn't return metadata after prepare.

        The SQL command 'PREPARE' was broken - should be take into
        account.
parent 418518c0
...@@ -1814,11 +1814,17 @@ static bool mysql_test_create_table(Prepared_statement *stmt) ...@@ -1814,11 +1814,17 @@ static bool mysql_test_create_table(Prepared_statement *stmt)
} }
static bool send_stmt_metadata(THD *thd, Prepared_statement *stmt, List<Item> *fields) static int send_stmt_metadata(THD *thd, Prepared_statement *stmt, List<Item> *fields)
{ {
return send_prep_stmt(stmt, fields->elements) || if (stmt->is_sql_prepare())
thd->protocol->send_result_set_metadata(fields, Protocol::SEND_EOF) || return 0;
thd->protocol->flush();
if (send_prep_stmt(stmt, fields->elements) ||
thd->protocol->send_result_set_metadata(fields, Protocol::SEND_EOF) ||
thd->protocol->flush())
return 1;
return 2;
} }
...@@ -1834,7 +1840,7 @@ static bool send_stmt_metadata(THD *thd, Prepared_statement *stmt, List<Item> *f ...@@ -1834,7 +1840,7 @@ static bool send_stmt_metadata(THD *thd, Prepared_statement *stmt, List<Item> *f
TRUE error, error message is set in THD TRUE error, error message is set in THD
*/ */
static bool mysql_test_show_create_table(Prepared_statement *stmt, static int mysql_test_show_create_table(Prepared_statement *stmt,
TABLE_LIST *tables) TABLE_LIST *tables)
{ {
DBUG_ENTER("mysql_test_show_create_table"); DBUG_ENTER("mysql_test_show_create_table");
...@@ -1843,8 +1849,10 @@ static bool mysql_test_show_create_table(Prepared_statement *stmt, ...@@ -1843,8 +1849,10 @@ static bool mysql_test_show_create_table(Prepared_statement *stmt,
char buff[2048]; char buff[2048];
String buffer(buff, sizeof(buff), system_charset_info); String buffer(buff, sizeof(buff), system_charset_info);
DBUG_RETURN(mysqld_show_create_get_fields(thd, tables, &fields, &buffer) || if (mysqld_show_create_get_fields(thd, tables, &fields, &buffer))
send_stmt_metadata(thd, stmt, &fields)); DBUG_RETURN(1);
DBUG_RETURN(send_stmt_metadata(thd, stmt, &fields));
} }
...@@ -1859,7 +1867,7 @@ static bool mysql_test_show_create_table(Prepared_statement *stmt, ...@@ -1859,7 +1867,7 @@ static bool mysql_test_show_create_table(Prepared_statement *stmt,
TRUE error, error message is set in THD TRUE error, error message is set in THD
*/ */
static bool mysql_test_show_create_db(Prepared_statement *stmt) static int mysql_test_show_create_db(Prepared_statement *stmt)
{ {
DBUG_ENTER("mysql_test_show_create_db"); DBUG_ENTER("mysql_test_show_create_db");
THD *thd= stmt->thd; THD *thd= stmt->thd;
...@@ -1883,7 +1891,7 @@ static bool mysql_test_show_create_db(Prepared_statement *stmt) ...@@ -1883,7 +1891,7 @@ static bool mysql_test_show_create_db(Prepared_statement *stmt)
TRUE error, error message is set in THD TRUE error, error message is set in THD
*/ */
static bool mysql_test_show_grants(Prepared_statement *stmt) static int mysql_test_show_grants(Prepared_statement *stmt)
{ {
DBUG_ENTER("mysql_test_show_grants"); DBUG_ENTER("mysql_test_show_grants");
THD *thd= stmt->thd; THD *thd= stmt->thd;
...@@ -1908,7 +1916,7 @@ static bool mysql_test_show_grants(Prepared_statement *stmt) ...@@ -1908,7 +1916,7 @@ static bool mysql_test_show_grants(Prepared_statement *stmt)
TRUE error, error message is set in THD TRUE error, error message is set in THD
*/ */
static bool mysql_test_show_slave_status(Prepared_statement *stmt) static int mysql_test_show_slave_status(Prepared_statement *stmt)
{ {
DBUG_ENTER("mysql_test_show_slave_status"); DBUG_ENTER("mysql_test_show_slave_status");
THD *thd= stmt->thd; THD *thd= stmt->thd;
...@@ -1931,7 +1939,7 @@ static bool mysql_test_show_slave_status(Prepared_statement *stmt) ...@@ -1931,7 +1939,7 @@ static bool mysql_test_show_slave_status(Prepared_statement *stmt)
TRUE error, error message is set in THD TRUE error, error message is set in THD
*/ */
static bool mysql_test_show_master_status(Prepared_statement *stmt) static int mysql_test_show_master_status(Prepared_statement *stmt)
{ {
DBUG_ENTER("mysql_test_show_master_status"); DBUG_ENTER("mysql_test_show_master_status");
THD *thd= stmt->thd; THD *thd= stmt->thd;
...@@ -1954,7 +1962,7 @@ static bool mysql_test_show_master_status(Prepared_statement *stmt) ...@@ -1954,7 +1962,7 @@ static bool mysql_test_show_master_status(Prepared_statement *stmt)
TRUE error, error message is set in THD TRUE error, error message is set in THD
*/ */
static bool mysql_test_show_binlogs(Prepared_statement *stmt) static int mysql_test_show_binlogs(Prepared_statement *stmt)
{ {
DBUG_ENTER("mysql_test_show_binlogs"); DBUG_ENTER("mysql_test_show_binlogs");
THD *thd= stmt->thd; THD *thd= stmt->thd;
...@@ -1979,7 +1987,7 @@ static bool mysql_test_show_binlogs(Prepared_statement *stmt) ...@@ -1979,7 +1987,7 @@ static bool mysql_test_show_binlogs(Prepared_statement *stmt)
TRUE error, error message is set in THD TRUE error, error message is set in THD
*/ */
static bool mysql_test_show_create_routine(Prepared_statement *stmt, int type) static int mysql_test_show_create_routine(Prepared_statement *stmt, int type)
{ {
DBUG_ENTER("mysql_test_show_binlogs"); DBUG_ENTER("mysql_test_show_binlogs");
THD *thd= stmt->thd; THD *thd= stmt->thd;
...@@ -2325,14 +2333,14 @@ static bool check_prepared_statement(Prepared_statement *stmt) ...@@ -2325,14 +2333,14 @@ static bool check_prepared_statement(Prepared_statement *stmt)
res= mysql_test_create_table(stmt); res= mysql_test_create_table(stmt);
break; break;
case SQLCOM_SHOW_CREATE: case SQLCOM_SHOW_CREATE:
if (!(res= mysql_test_show_create_table(stmt, tables))) if ((res= mysql_test_show_create_table(stmt, tables)) == 2)
{ {
/* Statement and field info has already been sent */ /* Statement and field info has already been sent */
DBUG_RETURN(FALSE); DBUG_RETURN(FALSE);
} }
break; break;
case SQLCOM_SHOW_CREATE_DB: case SQLCOM_SHOW_CREATE_DB:
if (!(res= mysql_test_show_create_db(stmt))) if ((res= mysql_test_show_create_db(stmt)) == 2)
{ {
/* Statement and field info has already been sent */ /* Statement and field info has already been sent */
DBUG_RETURN(FALSE); DBUG_RETURN(FALSE);
...@@ -2340,7 +2348,7 @@ static bool check_prepared_statement(Prepared_statement *stmt) ...@@ -2340,7 +2348,7 @@ static bool check_prepared_statement(Prepared_statement *stmt)
break; break;
#ifndef NO_EMBEDDED_ACCESS_CHECKS #ifndef NO_EMBEDDED_ACCESS_CHECKS
case SQLCOM_SHOW_GRANTS: case SQLCOM_SHOW_GRANTS:
if (!(res= mysql_test_show_grants(stmt))) if ((res= mysql_test_show_grants(stmt)) == 2)
{ {
/* Statement and field info has already been sent */ /* Statement and field info has already been sent */
DBUG_RETURN(FALSE); DBUG_RETURN(FALSE);
...@@ -2349,21 +2357,21 @@ static bool check_prepared_statement(Prepared_statement *stmt) ...@@ -2349,21 +2357,21 @@ static bool check_prepared_statement(Prepared_statement *stmt)
#endif /* NO_EMBEDDED_ACCESS_CHECKS */ #endif /* NO_EMBEDDED_ACCESS_CHECKS */
#ifndef EMBEDDED_LIBRARY #ifndef EMBEDDED_LIBRARY
case SQLCOM_SHOW_SLAVE_STAT: case SQLCOM_SHOW_SLAVE_STAT:
if (!(res= mysql_test_show_slave_status(stmt))) if ((res= mysql_test_show_slave_status(stmt)) == 2)
{ {
/* Statement and field info has already been sent */ /* Statement and field info has already been sent */
DBUG_RETURN(FALSE); DBUG_RETURN(FALSE);
} }
break; break;
case SQLCOM_SHOW_MASTER_STAT: case SQLCOM_SHOW_MASTER_STAT:
if (!(res= mysql_test_show_master_status(stmt))) if ((res= mysql_test_show_master_status(stmt)) == 2)
{ {
/* Statement and field info has already been sent */ /* Statement and field info has already been sent */
DBUG_RETURN(FALSE); DBUG_RETURN(FALSE);
} }
break; break;
case SQLCOM_SHOW_BINLOGS: case SQLCOM_SHOW_BINLOGS:
if (!(res= mysql_test_show_binlogs(stmt))) if ((res= mysql_test_show_binlogs(stmt)) == 2)
{ {
/* Statement and field info has already been sent */ /* Statement and field info has already been sent */
DBUG_RETURN(FALSE); DBUG_RETURN(FALSE);
...@@ -2371,14 +2379,14 @@ static bool check_prepared_statement(Prepared_statement *stmt) ...@@ -2371,14 +2379,14 @@ static bool check_prepared_statement(Prepared_statement *stmt)
break; break;
#endif /* EMBEDDED_LIBRARY */ #endif /* EMBEDDED_LIBRARY */
case SQLCOM_SHOW_CREATE_PROC: case SQLCOM_SHOW_CREATE_PROC:
if (!(res= mysql_test_show_create_routine(stmt, TYPE_ENUM_PROCEDURE))) if ((res= mysql_test_show_create_routine(stmt, TYPE_ENUM_PROCEDURE)) == 2)
{ {
/* Statement and field info has already been sent */ /* Statement and field info has already been sent */
DBUG_RETURN(FALSE); DBUG_RETURN(FALSE);
} }
break; break;
case SQLCOM_SHOW_CREATE_FUNC: case SQLCOM_SHOW_CREATE_FUNC:
if (!(res= mysql_test_show_create_routine(stmt, TYPE_ENUM_FUNCTION))) if ((res= mysql_test_show_create_routine(stmt, TYPE_ENUM_FUNCTION)) == 2)
{ {
/* Statement and field info has already been sent */ /* Statement and field info has already been sent */
DBUG_RETURN(FALSE); DBUG_RETURN(FALSE);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment