• Michael Widenius's avatar
    Fixed bug that MAX_USER_CONNECTIONS was not working properly in all situations... · c463f30c
    Michael Widenius authored
    Fixed bug that MAX_USER_CONNECTIONS was not working properly in all situations (which could cause aborted connects)
    thd->user_connect is now handled in thd->clenup() which will ensure that it works in all context (including slaves).
    I added also some DBUG_ASSERT() to ensure that things are working correctly.
    
    
    sql/sql_acl.cc:
      Reset thd->user_connect on failed check_for_max_user_connections() to ensure we don't decrement value twice.
      Removed not needed call to decrease_user_connections() as thd->cleanup() will now do it.
    sql/sql_class.cc:
      Call decrease_user_connections() in thd->cleanup()
    sql/sql_connect.cc:
      Ensure we don't allocate thd->user_connect twice.
      Simplify check_for_max_user_connections().
    sql/sql_parse.cc:
      Ensure that thd->user_connect is handled properly in for 'change_user' command.
    c463f30c
sql_acl.cc 249 KB