Skip to content

Conversation

@justforlxz
Copy link
Member

No description provided.

allow session take tty control, but skip SIGINT.

Log:
release 0.1.6

Log:
@deepin-ci-robot
Copy link

deepin pr auto review

代码审查意见:

  1. Changelog 更新

    • debian/changelog 文件中,新增的日志条目应该包含对更改的详细描述,而不仅仅是 "allow user take tty control"。建议添加更多细节,比如描述了哪些用户可以获取TTY控制,以及这样做的原因。
  2. Display.cpp 文件

    • Display.cpp 文件的修改中,VirtualTerminal::jumpToVt(auth->tty(), false); 可能会影响TTY控制的行为。如果这个更改是有意为之,应该确保相关的文档和测试用例都已更新,以反映这一变化。
  3. UserSession.cpp 文件

    • UserSession.cpp 文件的修改中,移除了对 m_helperApp->isSingleMode() 的检查,这可能会改变程序的行为。如果这是有意为之,应该添加相应的注释说明为什么这样做,以及它对程序逻辑的影响。
    • 新增的代码段 tcsetattr(STDIN_FILENO, TCSANOW, &newt);tcsetattr(STDOUT_FILENO, TCSANOW, &tios); 用于修改终端属性,这可能会影响终端的行为。建议添加注释说明这些更改的目的和影响。
    • tcgetattrtcsetattr 函数用于获取和设置终端属性,这些操作可能会影响终端的输入输出行为。建议确保这些更改不会对用户体验产生负面影响。
  4. 代码风格和一致性

    • 代码中存在一些不一致的地方,比如在 UserSession.cpp 文件中,注释的格式和位置不一致。建议统一代码风格,以提高代码的可读性。
  5. 安全性

    • UserSession.cpp 文件中,修改了终端的属性,这可能会影响终端的安全性。建议确保这些更改不会引入安全漏洞,比如允许用户执行未授权的操作。
  6. 错误处理

    • 代码中没有明显的错误处理机制。建议添加适当的错误处理代码,以处理可能出现的异常情况,比如 dup2tcgetattr 函数调用失败的情况。

总体来说,代码的修改看起来是有意为之,但需要更多的注释和文档来解释这些更改的目的和影响。同时,建议进行全面的测试,以确保这些更改不会引入新的问题。

@github-actions
Copy link

github-actions bot commented Dec 5, 2024

TAG Bot

TAG: 0.1.6
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 2eb302e into linuxdeepin:master Dec 5, 2024
8 of 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