]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
mon: set session_timeout when adding to session_map 37494/head
authorIlya Dryomov <idryomov@gmail.com>
Tue, 29 Sep 2020 13:29:16 +0000 (15:29 +0200)
committerIlya Dryomov <idryomov@gmail.com>
Wed, 30 Sep 2020 11:30:25 +0000 (13:30 +0200)
commit76fbe8b4aaad7f39b37baf2f2d0128e9fe5c29a6
tree37086c3a43ce237634feda0a2bf358838a1d9275
parentfd159956a728c39d4110a9993208eeccbf54eb93
mon: set session_timeout when adding to session_map

With msgr2, the session is added in Monitor::ms_handle_accept()
which is queued by ProtocolV2 at the end of handling CLIENT_IDENT
frame, before responding with SERVER_IDENT frame.  session_timeout
is 0 and gets set only in Monitor::ms_dispatch(), so if the session
trimming code in Monitor::tick() gets to the session before the peer
receives our SERVER_IDENT, handles it, sends the first message and
we receive it, the session is wrongly closed.

This doesn't happen with msgr1, because there the session is added in
Monitor::ms_dispatch(), upon receive of the first message (MSG_AUTH).

Fixes: https://tracker.ceph.com/issues/47697
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
src/mon/Session.h