工控方向学习文章推送到新建的 hc-ics 文件夹中 - #201
Open
Suc2es2 wants to merge 2 commits into
Open
Conversation
ProbiusOfficial
left a comment
Owner
There was a problem hiding this comment.
lgtm
确定完全添加后即可发起推送
如果还需要补充 请在当前PR中继续提交
Author
哦耶 |
Collaborator
|
还缺少对 mkdocs.yml 的修改,不然 ICS 部分内容无法呈现在网站上 |
Randark-JMT
requested review from
Randark-JMT
and removed request for
Randark-JMT
August 21, 2026 13:12
Randark-JMT
reviewed
Aug 22, 2026
Collaborator
There was a problem hiding this comment.
mkdocs.yml 导航我已在最新 commit 中补上(docs(nav): 将 hc-ics 工控安全章节接入 mkdocs.yml 导航),ICS 章节会出现在「MISC | 杂项」之后
剩余还有几处建议在当前 PR 中继续补充/修正:
未完成(阻断)
LSIS FEnet 协议.md末尾只有## 实战案例标题,正文缺失,文章在半截截断SCD 配置信息.md末尾悬空### 黑客的大意标题,正文同样缺失Omron Fins 协议.md有两处 AI 对话残留,需要删除:- FINS/UDP 报头表格前的「好的,这是您要求的 Markdown 格式表格:」
- 功能码表格前的「以下是您需要的 Markdown 表格:」
技术性错误
JT T 808 协议.md校验码描述「按字节异或,再与 0x7E 异或」有误——标准校验就是纯异或。文中自己的脚本jt808_checksum实现的也是纯异或,正文与脚本矛盾JT T 808 协议.md消息体属性 bit 位图中 bit14 标为「加密」,但实战案例与脚本按 2019 版解析(bit14=协议版本标识,bit13-10=加密方式),图与文不符MMS 协议.md顶层 PDU 表中unconfirmed-PDU标签写 A2 有误——A2 是confirmed-ErrorPDU,unconfirmed-PDU(信息报告)应为 A3IC 卡.md「扇区 0:000–0CF」有误,应为 000–03F(4 块 × 16 字节 = 64 字节即 0x40 长,后文也自述了这一点)
小问题
S7comm 协议.md:「得到 `flag{flag_is_here}」反引号未闭合,渲染会把后续文字吞进代码块;另外「S7 协议恶意攻击分析」案例开头重复粘贴了一遍上方已有的功能码表,建议删掉MQTT 协议.md:实战案例结尾只给出半个 flag(flag{21png_),收尾缺失1527 芯片.md:脚本中path = "iiiiicccccssssshhhhheeeeexxxxx"占位文件名建议换成更明确的示例名- 10 个文件中 7 个末尾缺换行符(
\ No newline at end of file)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.