summaryrefslogtreecommitdiffstats
path: root/glusterfsd
diff options
context:
space:
mode:
authorZhang Huan <zhanghuan@open-fs.com>2019-01-08 16:17:06 +0800
committerAmar Tumballi <amarts@redhat.com>2019-01-28 04:52:37 +0000
commitf747d55a7fd364e2b9a74fe40360ab3cb7b11537 (patch)
tree70ccd73edaa2b2e00ba2f3c8006d8624c9875e8d /glusterfsd
parent72922c1fd69191b220f79905a23395c3a87f86ce (diff)
socket: fix issue on concurrent handle of a socket
Found an issue on concurrent invoke of event handler to the same socket fd, causing memory corruption. This issue arises after applying commit "socket: Remove redundant in_lock in incoming message handling" that removes priv->in_lock to serialize socket read. The following call sequence describes how concurrent socket event handle happens. thread 1 thread 2 thread 3 epoll_wait() return (slot->in_handler is 0) call select_on_epoll() and epoll_ctl() on fd epoll_wait() return slot->in_handler++ (slot->in_handler is 1) slot->in_handler++ (slot->in_handler is 2) call handler() call handler() Fix this issue by skip invoke of handler if there is already a handler inprogress. Change-Id: I437126ac772debcadb00993a948919c931cd607b updates: bz#1467614 Signed-off-by: Zhang Huan <zhanghuan@open-fs.com>
Diffstat (limited to 'glusterfsd')
0 files changed, 0 insertions, 0 deletions