• Jon Olav Hauglid's avatar
    Bug #51355 handler stmt cause assertion in · 3eead1f0
    Jon Olav Hauglid authored
               bool MDL_context::try_acquire_lock(MDL_request*)
    
    This assert was triggered in the following way:
    1) HANDLER OPEN t1 from connection 1
    2) DROP TABLE t1 from connection 2. This will block due to the metadata lock
    held by the open handler in connection 1.
    3) DML statement (e.g. INSERT) from connection 1. This will close the table
    opened by the HANDLER in 1) and release its metadata lock. This is done due
    to the pending exclusive metadata lock from 2). 
    4) DROP TABLE t1 from connection 2 now completes and removes table t1.
    5) HANDLER READ from connection 1. Since the handler table was closed in 3),
    the handler code will try to reopen the table. First a new metadata lock on
    t1 will be granted before the command fails since the table was removed in 4).
    6) HANDLER READ from connection 1. This caused the assert.
    
    The reason for the assert was that the MDL_request's pointer to the lock
    ticket was not reset when the statement failed. HANDLER READ then tried to
    acquire a lock using the same MDL_request object, triggering the assert.
    This bug was only noticeable on debug builds and did not cause any problems
    on release builds.
    
    This patch fixes the problem by assuring that the pointer to the metadata 
    lock ticket is reset when reopening of handler tables fails.
    
    Test case added to handler.inc
    3eead1f0
sql_handler.cc 24.7 KB