1
0
Fork 0

Fix crash on quick `/reconnect now` usage (double free)

Check connection state before reconnection.

Fix #1894 https://github.com/profanity-im/profanity/issues/1894
This commit is contained in:
John Hernandez 2023-10-15 23:23:01 +02:00
parent 634cb586d9
commit be42ffcee9
2 changed files with 8 additions and 2 deletions

View File

@ -6467,7 +6467,6 @@ cmd_reconnect(ProfWin* window, const char* const command, gchar** args)
int intval = 0;
auto_char char* err_msg = NULL;
if (g_strcmp0(value, "now") == 0) {
cons_show("Reconnecting now.");
cl_ev_reconnect();
} else if (strtoi_range(value, &intval, 0, INT_MAX, &err_msg)) {
prefs_set_reconnect(intval);

View File

@ -98,7 +98,14 @@ cl_ev_disconnect(void)
void
cl_ev_reconnect(void)
{
if (connection_get_status() != JABBER_DISCONNECTED) {
jabber_conn_status_t conn_status = connection_get_status();
if (conn_status == JABBER_CONNECTING) {
cons_show_error("Reconnection aborted: Connection attempt is already in progress");
return;
}
cons_show("Reconnecting now.");
if (conn_status != JABBER_DISCONNECTED && conn_status != JABBER_DISCONNECTING) {
connection_disconnect();
ev_disconnect_cleanup();
// on intentional disconnect reset the counter