From 952d1be566f19517c2f3172c7dcfa52d6bfa0543 Mon Sep 17 00:00:00 2001
From: Andrew <1@1.ru>
Date: Tue, 17 Mar 2026 18:32:13 +0300
Subject: [PATCH] =?UTF-8?q?=D0=92=D1=8B=D0=BD=D0=B5=D1=81=D1=82=D0=B8=20?=
 =?UTF-8?q?=D0=B2=D1=8B=D1=87=D0=B8=D1=81=D0=BB=D0=B5=D0=BD=D0=B8=D0=B5=20?=
 =?UTF-8?q?UID=20=D0=B2=D0=B8=D1=80=D1=82=D1=83=D0=B0=D0=BB=D1=8C=D0=BD?=
 =?UTF-8?q?=D0=BE=D0=B9=20=D0=BA=D0=BB=D0=B0=D0=B2=D0=B8=D0=B0=D1=82=D1=83?=
 =?UTF-8?q?=D1=80=D1=8B=20=D0=B2=20=D1=81=D1=82=D0=B0=D1=82=D0=B8=D1=87?=
 =?UTF-8?q?=D0=B5=D1=81=D0=BA=D0=B8=D0=B9=20=D0=BC=D0=B5=D1=82=D0=BE=D0=B4?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

В configure() вызывался vk.get_uid() без предварительного vk.init().
Это работало случайно, так как get_uid() использовал только поля,
инициализированные в объявлении класса. Но если get_uid() когда-либо
обратится к dev_ или uidev_, это приведёт к segfault.

Добавлен статический метод VirtualKeyboard::make_uid(), который
вычисляет UID из переданных параметров и не зависит от состояния
объекта. get_uid() теперь делегирует ему. В configure() используется
make_uid() напрямую.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 src/VirtualKeyboard.cpp | 10 +++++++---
 src/VirtualKeyboard.h   |  2 ++
 src/main.cpp            |  2 +-
 3 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/src/VirtualKeyboard.cpp b/src/VirtualKeyboard.cpp
index 07eafe5..7e3f851 100644
--- a/src/VirtualKeyboard.cpp
+++ b/src/VirtualKeyboard.cpp
@@ -52,17 +52,21 @@ bool VirtualKeyboard::init() {
     return false;
 }
 
-std::string VirtualKeyboard::get_uid() const {
-    std::size_t hash = std::hash<std::string>{}(name);
+std::string VirtualKeyboard::make_uid(const char *dev_name, int bus, int vid, int pid, int ver) {
+    std::size_t hash = std::hash<std::string>{}(dev_name ? dev_name : "");
 
     char buf[64];
     std::snprintf(buf, sizeof(buf),
                   "%04x:%04x:%04x:%04x:%016zx",
-                  bustype, vendor, product, version, hash);
+                  bus, vid, pid, ver, hash);
 
     return buf;
 }
 
+std::string VirtualKeyboard::get_uid() const {
+    return make_uid(name, bustype, vendor, product, version);
+}
+
 void VirtualKeyboard::emit_key(int code, int value) {
     if (!uidev_) return;
 
diff --git a/src/VirtualKeyboard.h b/src/VirtualKeyboard.h
index d6d42ce..f2bd097 100644
--- a/src/VirtualKeyboard.h
+++ b/src/VirtualKeyboard.h
@@ -22,6 +22,8 @@ class VirtualKeyboard {
 
     bool init();
 
+    static std::string make_uid(const char *dev_name, int bus, int vid, int pid, int ver);
+
     std::string get_uid() const;
 
     void emit_key(int code, int value);
diff --git a/src/main.cpp b/src/main.cpp
index 64d4762..1c4c022 100644
--- a/src/main.cpp
+++ b/src/main.cpp
@@ -281,7 +281,7 @@ bool configure() {
         return false;
     }
 
-    std::string uid = vk.get_uid();
+    std::string uid = VirtualKeyboard::make_uid(vk.name, vk.bustype, vk.vendor, vk.product, vk.version);
     reader.add_to_blacklist(uid);
 
     bool connected;