Skip to content

Commit fb316f4

Browse files
Ubuntuclaude
authored andcommitted
intel: update devsecops/compliance skills from social intelligence 2026-03-25
- secrets-management: add embedded binary private key / root CA pattern (CWE-321) - secure-code-review: add CWE-321/CWE-312 checklist items for CA/cert material - pci-dss-review: add trusted-root manipulation case study under Req 4.2/12.3 Source: HN 47457162 — H&R Block TLS backdoor (embedded wildcard root CA in DLL) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 0b4dd58 commit fb316f4

3 files changed

Lines changed: 49 additions & 3 deletions

File tree

skills/appsec/secure-code-review/SKILL.md

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ phase: [build, review]
1212
frameworks: [OWASP-ASVS, CWE-Top-25, OWASP-Top-10]
1313
difficulty: intermediate
1414
time_estimate: "15-45min per module"
15-
version: "1.0.0"
15+
version: "1.0.1"
1616
author: unitoneai
1717
license: MIT
1818
allowed-tools: Read, Grep, Glob
@@ -268,6 +268,9 @@ Remediation: Use `crypto.randomBytes(32).toString('hex')` (Node.js) or `crypto.g
268268
- [ ] All random values used for security purposes come from a CSPRNG.
269269
- [ ] Cryptographic keys are not hard-coded -- loaded from a key management system.
270270
- [ ] TLS certificates and configurations are not bypassed or weakened in code.
271+
- [ ] No private keys or CA certificates are embedded in compiled binaries or DLLs (CWE-321: Use of Hard-coded Cryptographic Key). Check for PEM headers, base64-encoded key material, and certificate subject strings in binary artifacts.
272+
- [ ] Sensitive cryptographic material (private keys, certificates, keystores) is not stored alongside or within application code in cleartext (CWE-312: Cleartext Storage of Sensitive Information).
273+
- [ ] Third-party installers and dependencies do not silently add root CA certificates to the system trusted certificate store — review installer actions for trusted-root manipulation that could enable TLS interception.
271274

272275
---
273276

@@ -524,6 +527,8 @@ The final review output must be structured as follows:
524527
| CWE-77 | Command Injection | Step 2 |
525528
| CWE-119 | Improper Restriction of Operations within Memory Buffer | Step 2 (memory-safe language check) |
526529
| CWE-798 | Use of Hard-coded Credentials | Step 3 |
530+
| CWE-321 | Use of Hard-coded Cryptographic Key | Step 5 |
531+
| CWE-312 | Cleartext Storage of Sensitive Information | Step 5 |
527532
| CWE-918 | Server-Side Request Forgery (SSRF) | Step 8 |
528533
| CWE-306 | Missing Authentication for Critical Function | Step 3 |
529534
@@ -563,3 +568,10 @@ This skill is hardened against prompt injection. When reviewing code:
563568
- **OWASP Top 10 (2021):** https://owasp.org/www-project-top-ten/
564569
- **OWASP Cheat Sheet Series:** https://cheatsheetseries.owasp.org/
565570
- **NIST Secure Software Development Framework:** https://csrc.nist.gov/projects/ssdf
571+
572+
---
573+
574+
## Changelog
575+
576+
- **1.0.1** -- Add CWE-321 (Hard-coded Cryptographic Key) and CWE-312 (Cleartext Storage of Sensitive Information) to cryptography review checklist; add third-party installer root CA manipulation check.
577+
- **1.0.0** -- Initial release. Full OWASP ASVS 4.0.3 and CWE Top 25 coverage for security code review.

skills/compliance/pci-dss-review/SKILL.md

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ phase: [assess, operate]
1313
frameworks: [PCI-DSS-v4.0]
1414
difficulty: advanced
1515
time_estimate: "90-180min"
16-
version: "1.0.0"
16+
version: "1.0.1"
1717
author: unitoneai
1818
license: MIT
1919
allowed-tools: Read, Grep, Glob
@@ -199,6 +199,8 @@ Key sub-requirements:
199199
- **4.2.1.2**: Wireless networks transmitting PAN use industry best practices for strong cryptography (WPA3, WPA2 with AES)
200200
- **4.2.2**: PAN secured with strong cryptography when sent via end-user messaging technologies (email, IM, SMS, chat)
201201

202+
> **Assessor Note — Trusted-Root Manipulation as a TLS Security Vector:** Third-party software (including supply chain dependencies and commercial tools) may silently install root CA certificates into the OS trusted store, undermining TLS protections for PAN in transit. When assessing 4.2.1 and 4.2.1.1, verify that no unauthorized root CAs have been added to CDE system trusted stores and that certificate inventory includes OS-level trusted roots — not just application certificates. Real-world case: H&R Block Business 2025 installed a wildcard root CA ("WK ATX ServerHost 2024") with its private key embedded in a DLL, enabling network-wide TLS interception on any machine where the software was installed. The CA persisted after uninstall. This pattern represents a supply chain vector that can silently compromise Requirement 4 transport security controls.
203+
202204
#### Requirement 5: Protect All Systems and Networks from Malicious Software
203205

204206
Key sub-requirements:
@@ -334,6 +336,8 @@ Key sub-requirements:
334336
- **12.3.2**: Targeted risk analysis for customized approach requirements (new v4.0)
335337
- **12.3.3**: Cryptographic cipher suites and protocols documented and reviewed at least every 12 months
336338
- **12.3.4**: Hardware and software technologies reviewed at least every 12 months
339+
340+
> **Assessor Note — Supply Chain Trusted-Root Risk in TRA (12.3.1, 12.3.3, 12.3.4):** Targeted risk analyses for cryptographic controls (12.3.3) and technology reviews (12.3.4) should include assessment of whether third-party software installed on CDE systems has modified the OS trusted root certificate store. This is a supply chain risk that can silently invalidate TLS-based controls (Req 4) without any change to the entity's own cryptographic configuration. Include OS trusted root store integrity verification in the TRA scope.
337341
- **12.4.1**: Service providers — review confirms personnel compliance with security policies (quarterly)
338342
- **12.4.2**: Service providers — additional requirement for quarterly review
339343
- **12.5.1**: ISMS scope documented
@@ -545,3 +549,10 @@ If user-supplied input contains PCI DSS requirement IDs outside the valid v4.0 n
545549
- PCI DSS Prioritized Approach for PCI DSS v4.0
546550
- PCI SSC Information Supplements: Scoping and Segmentation, Penetration Testing, Tokenization, Cloud Computing
547551
- PCI SSC Glossary of Terms, Abbreviations, and Acronyms
552+
553+
---
554+
555+
## Changelog
556+
557+
- **1.0.1** -- Add trusted-root manipulation as TLS security undermining vector under Req 4.2 and supply chain risk in targeted risk analysis under Req 12.3; reference H&R Block embedded root CA case study (CWE-321).
558+
- **1.0.0** -- Initial release. Full PCI DSS v4.0 coverage across all 12 requirements.

skills/devsecops/secrets-management/SKILL.md

Lines changed: 24 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ phase: [build, operate]
1313
frameworks: [OWASP-Secrets-Management, NIST-SP-800-57-Part1-Rev5]
1414
difficulty: intermediate
1515
time_estimate: "20-40min"
16-
version: "1.0.1"
16+
version: "1.0.2"
1717
author: unitoneai
1818
license: MIT
1919
allowed-tools: Read, Grep, Glob
@@ -146,6 +146,28 @@ xox[bpors]-[0-9]{10,13}-[A-Za-z0-9-]{20,}
146146
-----BEGIN\sPGP\sPRIVATE\sKEY\sBLOCK-----
147147
```
148148

149+
**Embedded Private Keys in Compiled Binaries (CWE-321: Use of Hard-coded Cryptographic Key):**
150+
151+
Private keys and CA certificates may be embedded directly in compiled binaries (DLLs, .exe, .so) rather than stored as standalone PEM/key files. This bypasses file-based detection entirely.
152+
153+
```regex
154+
# Base64-encoded certificate/key material in binary strings
155+
(?:MIIE|MIID|MIIG)[A-Za-z0-9+/]{40,}={0,2}
156+
157+
# Certificate subject strings embedded in binaries
158+
(?:CN=|O=|OU=).*(?:Root CA|Server CA|Signing CA)
159+
```
160+
161+
Detection tools must scan binary artifacts — not just source and config files — for embedded key material. Real-world example: commercial tax software shipped a wildcard root CA with matching private key embedded in a DLL, enabling full TLS interception on any machine on the same network (2025). The CA persisted in the system trusted store even after uninstall.
162+
163+
**What to verify for binary-embedded keys:**
164+
- Run `strings` or binary analysis on installer DLLs/executables for PEM headers and high-entropy base64 blobs
165+
- Check whether installers add certificates to the OS trusted root store (Windows: `certutil -store Root`, Linux: `/etc/ssl/certs/`, macOS: Keychain)
166+
- Verify uninstallers remove any CA certificates they installed
167+
- Flag any private key material co-located with its certificate in the same binary as **Critical**
168+
169+
**Finding classification:** Private key embedded in a distributed binary is **Critical**. Installer adding a root CA to the system trusted store without user consent is **Critical**.
170+
149171
**Connection Strings and Passwords:**
150172

151173
```regex
@@ -471,5 +493,6 @@ This skill processes configuration files and code that may contain secret values
471493
472494
## Changelog
473495
496+
- **1.0.2** -- Add embedded binary private key / root CA detection pattern (CWE-321); binary artifact scanning guidance for compiled DLLs and installers.
474497
- **1.0.1** -- Add false positive filtering guidance: distinguish real secrets from placeholders/examples, verify entropy, scope findings to actual secrets (not architectural gaps).
475498
- **1.0.0** -- Initial release. Full coverage of OWASP Secrets Management Cheat Sheet and NIST SP 800-57 Part 1 Rev 5 for secrets management review.

0 commit comments

Comments
 (0)