[TS-4125] Wrong non-active conditions check

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 6.0.0
• Fix Version/s: 6.1.0
• Component/s: Core
• Assignee: Bryan Call
• Reporter: Chao Xu
• Created: 12/Jan/16 12:31
• Updated: 04/Feb/16 18:48
• Resolved: 12/Jan/16 17:31

Description

In the NetHandler::manage_active_queue() function:

we should close inactivity timeout vc that means "vc->next_activity_timeout_at <= now".

but "vc->next_activity_timeout_at > now" here.

It's call _close_vc() for an activity vc and close vc early.

  for (; vc != NULL; vc = vc_next) {
    if ((vc->next_inactivity_timeout_at > now) || (vc->next_activity_timeout_at > now)) {
      _close_vc(vc, now, handle_event, closed, total_idle_time, total_idle_count);
    }
    if (max_connections_active_per_thread_in > active_queue_size) {
      return true;
    }
  }

Issue Links

• is duplicated by: TS-3930 Activity timeout doesn't close the connection