From: Ilya Kosarev <i.kosarev@tarantool.org> To: tarantool-patches@freelists.org Cc: georgy@tarantool.org, i.kosarev@corp.mail.ru, Ilya Kosarev <i.kosarev@tarantool.org> Subject: [tarantool-patches] [PATCH] Close socket explicitly before wal_dir at exit Date: Fri, 22 Feb 2019 15:05:00 +0300 [thread overview] Message-ID: <20190222120500.21180-1-i.kosarev@tarantool.org> (raw) tarantool instance didn't close socket explicitly which could cause hot standby instance to fail to bind in case it tries to bind before socket is closed by OS. Now it is fixed by closing socket explicitly before wal_dir. Closes #3967 --- Branch: https://github.com/tarantool/tarantool/tree/i.kosarev/gh-3967-close-socket-explicitly Issue: https://github.com/tarantool/tarantool/issues/3967 src/box/box.cc | 1 + src/box/iproto.cc | 7 +++++++ src/box/iproto.h | 3 +++ 3 files changed, 11 insertions(+) diff --git a/src/box/box.cc b/src/box/box.cc index 9a8ca1585..38bdebeba 100644 --- a/src/box/box.cc +++ b/src/box/box.cc @@ -1666,6 +1666,7 @@ box_free(void) tuple_free(); port_free(); #endif + iproto_socket_free(); replication_free(); sequence_free(); gc_free(); diff --git a/src/box/iproto.cc b/src/box/iproto.cc index 19a945d2a..1d2fb3bdd 100644 --- a/src/box/iproto.cc +++ b/src/box/iproto.cc @@ -2075,3 +2075,10 @@ iproto_set_msg_max(int new_iproto_msg_max) iproto_do_cfg(&cfg_msg); cpipe_set_max_input(&net_pipe, new_iproto_msg_max / 2); } + +void +iproto_socket_free() +{ + tt_pthread_cancel(net_cord.id); + close(binary.ev.fd); +} diff --git a/src/box/iproto.h b/src/box/iproto.h index b9a6cf8f7..6a9627705 100644 --- a/src/box/iproto.h +++ b/src/box/iproto.h @@ -80,6 +80,9 @@ iproto_listen(const char *uri); void iproto_set_msg_max(int iproto_msg_max); +void +iproto_socket_free(); + #endif /* defined(__cplusplus) */ #endif -- 2.17.1
next reply other threads:[~2019-02-22 12:05 UTC|newest] Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-02-22 12:05 Ilya Kosarev [this message] 2019-02-25 11:28 ` Vladimir Davydov
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20190222120500.21180-1-i.kosarev@tarantool.org \ --to=i.kosarev@tarantool.org \ --cc=georgy@tarantool.org \ --cc=i.kosarev@corp.mail.ru \ --cc=tarantool-patches@freelists.org \ --subject='Re: [tarantool-patches] [PATCH] Close socket explicitly before wal_dir at exit' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox