[GitHub] [trafficserver] masaori335 commented on issue #6776: Assertion in HttpSM::main_handler

[GitBox <gi...@apache.org>]

masaori335 commented on issue #6776:
URL: https://github.com/apache/trafficserver/issues/6776#issuecomment-634459001


   More details between [ 3  ] and [ 4  ] on randall's backtrace
   ```
   (gdb) bt
   ...
   #4  0x00005630e9de78b3 in cleanup_entry (this=<optimized out>, e=<optimized out>) at HttpSM.cc:232
   #5  HttpVCTable::cleanup_all (this=0x7ffaeb92bfc0) at HttpSM.cc:243
   #6  0x00005630e9deb2f0 in HttpSM::kill_this (this=0x7ffaeb92ac00) at HttpSM.cc:6863
   ...
   (gdb) frame 4
   #4  0x00005630e9de78b3 in cleanup_entry (this=<optimized out>, e=<optimized out>) at HttpSM.cc:232
   232         e->vc->do_io_close();
   ```
   
   It looks like the HttpSM is trying call `do_io_close()` of invalid `vc`. Below is the vc on frame 5.
   ```
   (gdb) p *vc_table[1].vc
   $5 = {Continuation = {force_VFPT_to_top = {_vptr$force_VFPT_to_top = 0x7ffb81d5a070}, handler = NULL, mutex = {m_ptr = 0x0}, link = {SLink<Continuation> = {next = 0x0}, prev = 0x0}, control_flags = {raw_flags = 0}, thread_affinity = 0x0}, lerrno = 0}
   ```
   
   The `HttpSM::cleanup_entry()` has nullptr check of `vc`, so we forget to set it nullptr at somewhere when HttpSM close/remove the vc entry?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org