Hi, and thanks for the patch.

Firstly, this needs to be fixed in devel (stonking) before an SRU can
happen. Please provide a patch for stonking.

Beyond that, a couple of comments on the diff below:

> diff -Nru libkysdk-system-3.0.1.2/debian/changelog 
> libkysdk-system-3.0.1.2/debian/changelog
> --- libkysdk-system-3.0.1.2/debian/changelog  2026-04-20 18:15:24.000000000 
> +0800
> +++ libkysdk-system-3.0.1.2/debian/changelog  2026-05-06 10:30:00.000000000 
> +0800
> @@ -1,3 +1,9 @@
> +libkysdk-system (3.0.1.2-0ubuntu1.1) resolute-proposed; urgency=medium

The series should just be "resolute", not "resolute-proposed". The
upload will go to -proposed automatically when accepted by the SRU team.

> +
> +  * Fix /usr/bin/bluetoothService crash during startup (LP: #2151287)
> +
> + -- zhangpan <[email protected]>  Tue, 06 May 2026 10:30:00 +0800
> +
>  libkysdk-system (3.0.1.2-0ubuntu1) resolute; urgency=medium
>  
>    * Bugfix only:
> diff -Nru 
> libkysdk-system-3.0.1.2/debian/patches/fix-bluetooth-coredump-lp2151287.patch 
> libkysdk-system-3.0.1.2/debian/patches/fix-bluetooth-coredump-lp2151287.patch
> --- 
> libkysdk-system-3.0.1.2/debian/patches/fix-bluetooth-coredump-lp2151287.patch 
>     1970-01-01 08:00:00.000000000 +0800
> +++ 
> libkysdk-system-3.0.1.2/debian/patches/fix-bluetooth-coredump-lp2151287.patch 
>     2026-05-06 10:30:00.000000000 +0800
> @@ -0,0 +1,300 @@
> +From: zhangpan <[email protected]>
> +Date: Wed, 06 May 2026 16:31:38 +0800
> +Subject: [PATCH] Fix /usr/bin/bluetoothService crash by refactoring 
> kdk_system_get_eUser
> +
> +The original implementation of `kdk_system_get_eUser()` in libkysysinfo
> +relied on D-Bus queries to org.freedesktop.login1, which caused a
> +segmentation fault (dbus_message_iter_recurse) during the early
> +initialization of `/usr/bin/bluetoothService`.
> +
> +This patch addresses the crash by completely refactoring the user retrieval
> +logic. Instead of D-Bus, it now securely reads the active tty from
> +`/sys/class/tty/tty0/active` and parses the output of the `who` command.
> +
> +Additional robustness fixes included:
> +- Added missing <syslog.h> header for error logging.
> +- Added missing "ActiveSession" property validation in D-Bus signal handler.
> +- Added null pointer checks to prevent crashes during user switching and 
> when freeing the current user string.
> +
> +Bug-Ubuntu: https://bugs.launchpad.net/bugs/2151287
> +Origin: upstream

Can you provide the upstream link/commit hash? It's not clear where to
find this otherwise.

> +Last-Update: 2026-05-06

Lastly, it is a bit unfortunate that the SRU patch involves a complete
refactor. Ideally, we would have a small, targeted fix for this in the
stable release. I'm not sure if we can do anything about that.


** Changed in: libkysdk-system (Ubuntu Resolute)
       Status: New => Incomplete

** Changed in: libkysdk-system (Ubuntu Stonking)
       Status: New => Incomplete

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/2151287

Title:
  [SRU] /usr/bin/bluetoothService crashes in kdk_system_get_eUser during
  startup

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntukylin/+bug/2151287/+subscriptions


-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to