Skip to content

chore: Update version to 0.1.21 - #34

Merged
lzwind merged 1 commit into
linuxdeepin:masterfrom
wangrong1069:pr0122
Jan 22, 2026
Merged

chore: Update version to 0.1.21#34
lzwind merged 1 commit into
linuxdeepin:masterfrom
wangrong1069:pr0122

Conversation

@wangrong1069

Copy link
Copy Markdown
Contributor

As title.

Log: update version to 0.1.21

As title.

Log: update version to 0.1.21
@github-actions

Copy link
Copy Markdown

TAG Bot

TAG: 0.1.21
EXISTED: no
DISTRIBUTION: unstable

@deepin-ci-robot

Copy link
Copy Markdown

deepin pr auto review

这份 debian/changelog 的 diff 内容主要记录了软件包版本从 0.1.20 升级到 0.1.21 的变更。以下是对该变更的审查意见,涵盖语法逻辑、代码质量、性能和安全四个方面:

1. 语法逻辑

  • 格式规范:该变更严格遵循了 Debian changelog 文件的格式标准(符合 dpkg-parsechangelog 的解析规则)。
    • 包名:deepin-scanner
    • 版本号:0.1.21(遵循了数字递增逻辑)。
    • 发行版:unstable
    • 优先级:medium
    • 维护者信息:格式正确,包含姓名、邮箱和时间戳。
  • 时间戳问题:时间戳显示为 2026 年(Thu, 22 Jan 2026)。
    • 建议:请核实是否为笔误。如果是未来的日期,可能会导致某些自动化构建工具或发布检查脚本产生警告或错误。

2. 代码质量

  • 变更描述清晰度:日志条目为 fix: Fix sw64 gcc not support pie by default
    • 优点:使用了 fix: 前缀,清晰地表明了这是一个修复类型的提交。
    • 改进建议:描述略显口语化且存在语法瑕疵。
      • "not support pie" 建议改为 "does not support PIE" 或 "PIE not supported"。
      • "by default" 可能会产生歧义,是指编译器默认不支持,还是指默认情况下开启了 PIE 导致编译失败?
    • 优化后的描述建议
      • fix: Disable PIE for sw64 architecture due to lack of compiler support (更专业,明确指出了是禁用 PIE 以及原因)。
      • 或者:fix: Fix build failure on sw64 caused by unsupported PIE flags

3. 代码性能

  • 影响分析:该变更涉及构建配置(针对 sw64 架构禁用 PIE - Position Independent Executable)。
    • 性能影响:禁用 PIE 通常会对程序的启动性能有极微小的正面影响(因为不需要地址重定位),但这在现代操作系统上通常可以忽略不计。
    • 架构特定性:变更仅针对 sw64 架构,对 x86_64 或 ARM64 等主流架构的性能无影响。

4. 代码安全

  • 安全风险评估这是该变更最需要关注的部分
    • PIE 的作用:PIE(位置无关可执行)是 ASLR(地址空间布局随机化)的基础技术之一。启用 PIE 可以让程序在内存中的加载地址随机化,从而增加攻击者进行 ROP(面向返回编程)等攻击的难度。
    • 禁用 PIE 的后果:针对 sw64 架构禁用 PIE,意味着在该架构上运行的 deepin-scanner 将失去 ASLR 的保护,安全性相对降低。
    • 改进建议
      • 确认 sw64 的 GCC 工具链确实完全不支持 PIE。如果只是旧版本不支持,建议在 debian/rules 中增加版本检测逻辑,仅在旧版本上禁用。
      • 如果必须禁用,请确保该软件包在 sw64 上的使用场景风险较低,或者通过其他安全加固手段(如 Stack Canaries, Fortify Source)来弥补。
      • 建议在 debian/control 或相关构建文件中明确注释说明为什么在此架构上必须关闭此安全特性,以便后续维护者知晓。

总结

该变更符合 Debian 打包规范,但在时间戳准确性安全性说明上需要改进。建议修正时间年份,并优化变更日志的措辞,使其更加专业和明确。同时,鉴于禁用了安全特性(PIE),请务必确认这是技术上的必要妥协,并做好相关记录。

@deepin-ci-robot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: lzwind, wangrong1069

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

@lzwind
lzwind merged commit a98077b into linuxdeepin:master Jan 22, 2026
15 checks passed
@wangrong1069
wangrong1069 deleted the pr0122 branch January 22, 2026 12:17
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