Changeset a0d4afe in mainline for uspace/srv/hid/display/seat.c

Timestamp:

2023-01-18T16:51:44Z (2 years ago)

Author:

Jiri Svoboda <jiri@…>

Branches:

master, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

3e7e226

Parents:

b0ae23f

Message:

Make sure input device configuration is destroyed together with seat

When a seat is destroyed without unassigning devices first, this causes
a dangling seat pointer that would cause the display server to crash
if the corresponding device generates an event.

File:

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

uspace/srv/hid/display/seat.c

@@ -43 +43 @@
 #include "cursor.h"
 #include "display.h"
+#include "idevcfg.h"
 #include "seat.h"
 #include "window.h"
@@ -79 +80 @@
         }
 
+        list_initialize(&seat->idevcfgs);
+
         ds_display_add_seat(display, seat);
         seat->pntpos.x = 0;
@@ -96 +99 @@
 void ds_seat_destroy(ds_seat_t *seat)
 {
+        ds_idevcfg_t *idevcfg;
+
+        /* Remove all input device configuration entries pointing to this seat */
+        idevcfg = ds_seat_first_idevcfg(seat);
+        while (idevcfg != NULL) {
+                ds_idevcfg_destroy(idevcfg);
+                idevcfg = ds_seat_first_idevcfg(seat);
+        }
+
         ds_display_remove_seat(seat);
+
         free(seat->name);
         free(seat);
@@ -549 +562 @@
 }
 
+/** Add input device configuration entry to seat.
+ *
+ * @param seat Seat
+ * @param idevcfg Input device configuration
+ */
+void ds_seat_add_idevcfg(ds_seat_t *seat, ds_idevcfg_t *idevcfg)
+{
+        assert(idevcfg->seat == NULL);
+        assert(!link_used(&idevcfg->lseatidcfgs));
+
+        idevcfg->seat = seat;
+        list_append(&idevcfg->lseatidcfgs, &seat->idevcfgs);
+}
+
+/** Remove input device configuration entry from seat.
+ *
+ * @param idevcfg Input device configuration entry
+ */
+void ds_seat_remove_idevcfg(ds_idevcfg_t *idevcfg)
+{
+        list_remove(&idevcfg->lseatidcfgs);
+        idevcfg->seat = NULL;
+}
+
+/** Get first input device configuration entry in seat.
+ *
+ * @param disp Display
+ * @return First input device configuration entry or @c NULL if there is none
+ */
+ds_idevcfg_t *ds_seat_first_idevcfg(ds_seat_t *seat)
+{
+        link_t *link = list_first(&seat->idevcfgs);
+
+        if (link == NULL)
+                return NULL;
+
+        return list_get_instance(link, ds_idevcfg_t, lseatidcfgs);
+}
+
+/** Get next input device configuration entry in seat.
+ *
+ * @param idevcfg Current input device configuration entry
+ * @return Next input device configuration entry or @c NULL if there is none
+ */
+ds_idevcfg_t *ds_seat_next_idevcfg(ds_idevcfg_t *idevcfg)
+{
+        link_t *link = list_next(&idevcfg->lseatidcfgs, &idevcfg->seat->idevcfgs);
+
+        if (link == NULL)
+                return NULL;
+
+        return list_get_instance(link, ds_idevcfg_t, lseatidcfgs);
+}
+
 /** @}
  */