tombstoned: Fix calls to evconnlistener_new.
The order of arguments is wrong - we're passing flags=static_cast<unsigned>(-1) and backlog=LEV_OPT_CLOSE_ON_FREE (which is 2). On versions of libevent prior to 2.1.8, this ends up accidentally setting OPT_LEAVE_SOCKETS_BLOCKING, OPT_CLOSE_ON_EXEC, OPT_REUSABLE and OPT_THREADSAFE and limiting our backlog to two. These unintentional changes are relatively benign; we never make our sockets block, we never exec, we never reuse sockets and the additional locking overhead should be negligible. The backlog of two might be a problem in theory, but there haven't been any reports of issues caused by it. Things get worse on 2.1.8 - that version introduces several new flags, one of which is OPT_DISABLED. This disables the new listener by default, which means that our event loop returns early because it has no active listeners for any of its events. Bug: 64543673 Test: Manual. Change-Id: I9954bc7fe1af761de1a950d935dd2e6ce7e2c5f5
Loading
Please register or sign in to comment