fix: set uid by self when not use ddm

linuxdeepin / treeland

20 stars 15 forks source link

fix: set uid by self when not use ddm #323

Closed wineee closed 5 months ago

wineee commented 5 months ago

log: fix wallpaper not work in lightdm

deepin-ci-robot commented 5 months ago

deepin pr auto review

关键摘要:

在新增代码中，直接使用 getgid() 可能会导致未处理的初始化错误，如果 getgid() 返回一个无效的GID，可能会导致进程无法正常运行。
环境变量 XDG_SESSION_DESKTOP 的使用可能不是跨平台兼容的，应该考虑使用更通用的平台检测方法。
新增代码没有处理可能出现的异常情况，例如 setUserId 函数可能会因为各种原因失败。

是否建议立即修改:

deepin-ci-robot commented 5 months ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Groveer, wineee

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files: - **[debian/deepin/OWNERS](https://github.com/linuxdeepin/treeland/blob/master/debian/deepin/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment