Skip to content

Commit

Permalink
session: flag instead of state for filtering ntf
Browse files Browse the repository at this point in the history
Avoid situations when notifications are delayed for long enough for
transports to start closing/cleaning up.

Type: fix

Signed-off-by: Florin Coras <fcoras@cisco.com>
Change-Id: Id35b0099adb5242108154a5e19d5ee15e6ca0058
  • Loading branch information
florincoras committed Jan 23, 2024
1 parent 0aa0d6f commit 77680ae
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/vnet/session/session_input.c
Original file line number Diff line number Diff line change
Expand Up @@ -194,12 +194,12 @@ app_worker_flush_events_inline (app_worker_t *app_wrk, u32 thread_index,
break;
case SESSION_CTRL_EVT_DISCONNECTED:
s = session_get (evt->session_index, thread_index);
if (s->session_state <= SESSION_STATE_TRANSPORT_CLOSING)
if (!(s->flags & SESSION_F_APP_CLOSED))
app->cb_fns.session_disconnect_callback (s);
break;
case SESSION_CTRL_EVT_RESET:
s = session_get (evt->session_index, thread_index);
if (s->session_state <= SESSION_STATE_TRANSPORT_CLOSING)
if (!(s->flags & SESSION_F_APP_CLOSED))
app->cb_fns.session_reset_callback (s);
break;
case SESSION_CTRL_EVT_UNLISTEN_REPLY:
Expand Down

0 comments on commit 77680ae

Please sign in to comment.