Skip to content

Conversation

@justforlxz
Copy link
Member

No description provided.

remove jump vt and use logind activate user session

Log:
release 0.1.7

Log:
@deepin-ci-robot
Copy link

deepin pr auto review

代码审查意见:

  1. Display.cpp文件中,switchToUser函数中新增了对dde用户的特殊处理,这可能是为了解决黑屏问题。但是,如果dde用户不是唯一需要特殊处理的用户,建议将这部分逻辑提取出来,避免代码重复。

  2. Display.cpp文件中,switchToUser函数中新增了对Logind服务的检查和会话激活逻辑。这部分代码应该有相应的错误处理机制,以防止在Logind服务不可用或会话激活失败时程序崩溃。

  3. HelperApp.cpp文件中,新增了qDebug输出,这是一个好的做法,可以帮助调试和记录关键信息。但是,在生产环境中,建议使用更高级别的日志记录,如qWarningqCritical,以避免泄露敏感信息。

  4. UserSession.cpp文件中,takeControl函数中新增了对m_helperApp->isGreeter()的检查。这可能是为了确保只有在Greeter模式下才执行ioctl操作。但是,如果m_helperApp->isGreeter()返回false,那么ioctl操作将不会执行。这可能不是预期的行为,应该确认这是否符合设计意图。

  5. UserSession.cpp文件中,takeControl函数中使用了qCritical来记录错误信息。这是一个好的做法,可以帮助调试和记录关键信息。但是,如果错误信息中包含敏感信息,应该考虑使用更高级别的日志记录,如qWarningqCritical,以避免泄露敏感信息。

  6. UserSession.cpp文件中,takeControl函数中使用了errno来获取错误信息。这是一个好的做法,可以帮助调试和记录错误信息。但是,如果errnoioctl调用之前被修改,那么获取的错误信息可能不准确。建议在ioctl调用之前保存errno的值,并在记录错误信息时使用保存的值。

  7. UserSession.cpp文件中,takeControl函数中使用了qCritical来记录错误信息。这是一个好的做法,可以帮助调试和记录关键信息。但是,如果错误信息中包含敏感信息,应该考虑使用更高级别的日志记录,如qWarningqCritical,以避免泄露敏感信息。

  8. UserSession.cpp文件中,takeControl函数中使用了errno来获取错误信息。这是一个好的做法,可以帮助调试和记录错误信息。但是,如果errnoioctl调用之前被修改,那么获取的错误信息可能不准确。建议在ioctl调用之前保存errno的值,并在记录错误信息时使用保存的值。

总体来说,代码的改动看起来是为了解决特定问题而进行的,但是需要确保这些改动不会引入新的问题,并且符合项目的整体设计和架构。

@github-actions
Copy link

github-actions bot commented Dec 6, 2024

TAG Bot

TAG: 0.1.7
EXISTED: no
DISTRIBUTION: unstable

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

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

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@Groveer Groveer merged commit b69889e into linuxdeepin:master Dec 6, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants