Commit cac90852 authored by unknown's avatar unknown
Browse files

Manual merge of fix for bug #6081 "Call to deprecated mysql_create_db() 

function crashes server" with main tree.
parents 2f8b0ab3 1b142600

sql/sql_parse.cc

+3 −1
Original line number Diff line number Diff line
@@ -1562,6 +1562,7 @@ bool dispatch_command(enum enum_server_command command, THD *thd, 
  case COM_CREATE_DB:				// QQ: To be removed

    {

      char *db=thd->strdup(packet), *alias;

      HA_CREATE_INFO create_info;



      statistic_increment(com_stat[SQLCOM_CREATE_DB],&LOCK_status);

      // null test to handle EOM
@@ -1573,8 +1574,9 @@ bool dispatch_command(enum enum_server_command command, THD *thd, 
      if (check_access(thd,CREATE_ACL,db,0,1,0))

	break;

      mysql_log.write(thd,command,packet);

      bzero(&create_info, sizeof(create_info));

      if (mysql_create_db(thd, (lower_case_table_names == 2 ? alias : db),

                          0, 0) < 0)

                          &create_info, 0) < 0)

        send_error(thd, thd->killed ? ER_SERVER_SHUTDOWN : 0);

      break;

    }

tests/client_test.c

+26 −0
Original line number Diff line number Diff line
@@ -10688,6 +10688,31 @@ static void test_bug6046() 
}





/*

  Altough mysql_create_db(), mysql_rm_db() are deprecated since 4.0 they

  should not crash server and should not hang in case of errors.



  Since those functions can't be seen in modern API (unless client library

  was compiled with USE_OLD_FUNCTIONS define) we use simple_command() macro.

*/

static void test_bug6081()

{

  int rc;

  myheader("test_bug6081");



  rc= simple_command(mysql, COM_DROP_DB, current_db,

                     (ulong)strlen(current_db), 0);

  myquery(rc);

  rc= simple_command(mysql, COM_DROP_DB, current_db,

                     (ulong)strlen(current_db), 0);

  myquery_r(rc);

  rc= simple_command(mysql, COM_CREATE_DB, current_db,

                     (ulong)strlen(current_db), 0);

  myquery(rc);

  rc= simple_command(mysql, COM_CREATE_DB, current_db,

                     (ulong)strlen(current_db), 0);

  myquery_r(rc);

}





/*
@@ -11004,6 +11029,7 @@ int main(int argc, char **argv) 
    test_bug6058();         /* check support for 0000-00-00 dates */

    test_bug6059();         /* correct metadata for SELECT ... INTO OUTFILE */

    test_bug6046();         /* NATURAL JOIN transformation works in PS */

    test_bug6081();         /* test of mysql_create_db()/mysql_rm_db() */

    /*

      XXX: PLEASE RUN THIS PROGRAM UNDER VALGRIND AND VERIFY THAT YOUR TEST

      DOESN'T CONTAIN WARNINGS/ERRORS BEFORE YOU PUSH.