Changeset 5be5396 in mainline for uspace/srv/hid/remcons/remcons.c

Timestamp:

2024-10-11T16:08:55Z (3 months ago)

Author:

Jiri Svoboda <jiri@…>

Branches:

master

Children:

b3b79981

Parents:

fb06afd

Message:

Remcons needs to properly clean up when terminating a user session

File:

• uspace/srv/hid/remcons/remcons.c (modified) (4 diffs)

uspace/srv/hid/remcons/remcons.c

@@ -739 +739 @@
         user->srvs.ops = &con_ops;
         user->srvs.sarg = remcons;
-        user->srvs.abort_timeout = 1000;
+        user->srvs.abort_timeout = 1000000;
 
         telnet_user_add(user);
@@ -765 +765 @@
         while (!user_can_be_destroyed_no_lock(user)) {
                 if (user->task_finished) {
-                        user->conn = NULL;
                         user->socket_closed = true;
                         user->srvs.aborted = true;
-                        continue;
                 } else if (user->socket_closed) {
                         if (user->task_id != 0) {
@@ -774 +772 @@
                         }
                 }
-                fibril_condvar_wait_timeout(&user->refcount_cv, &user->recv_lock, 1000);
+                fibril_condvar_wait_timeout(&user->refcount_cv,
+                    &user->recv_lock, 1000000);
         }
         fibril_mutex_unlock(&user->recv_lock);
@@ -786 +785 @@
 
         telnet_user_log(user, "Destroying...");
+
+        if (remcons->enable_ctl) {
+                /* Disable mouse tracking */
+                vt100_set_button_reporting(remcons->vt, false);
+
+                /* Reset all character attributes and clear screen */
+                vt100_sgr(remcons->vt, 0);
+                vt100_cls(remcons->vt);
+                vt100_set_pos(remcons->vt, 0, 0);
+
+                telnet_user_flush(user);
+        }
+
+        tcp_conn_send_fin(user->conn);
+        user->conn = NULL;
+
         telnet_user_destroy(user);
+        vt100_destroy(remcons->vt);
+        free(remcons);
         return;
 error: