openharmony-security-review

Identify security vulnerabilities in OpenHarmony C++ system service code handling IPC inputs.

31|6|Updated Jan 29, 2026
One-click install
npx skills add https://github.com/openharmonyinsight/openharmony-skills --skill openharmony-security-review
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: openharmony-security-review
Source: https://github.com/openharmonyinsight/openharmony-skills/tree/main/skills/openharmony-security-review
Command: npx skills add https://github.com/openharmonyinsight/openharmony-skills --skill openharmony-security-review

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

It helps you systematically identify security vulnerabilities in privileged OpenHarmony system service C++ code, especially where untrusted data enters through IPC or network interfaces.

Core Features & Use Cases

  • IPC & Deserialization Safety: Verifies MessageParcel reads are checked and that post-deserialization validation prevents size/logic exploitation.
  • Multithreading & Concurrency Hardening: Detects container thread-safety issues, iterator invalidation, deadlock risk, and TOCTOU patterns.
  • Sensitive Data & Permission Validation: Flags PII/pointer/address leakage in logs and ensures privileged operations validate caller permissions.

Quick Start

Use openharmony-security-review to audit an xxxService.cpp and its corresponding xxxStub.cpp for IPC input handling, concurrency hazards, sensitive logging, and permission checks.

Frequently Asked Questions about openharmony-security-review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I find IPC security vulnerabilities in OpenHarmony C++ system services?

To find IPC security vulnerabilities in OpenHarmony C++ system services, audit MessageParcel deserialization paths to verify read integrity, bounds checking, and logical validation against untrusted inputs.

What is thread safety validation for OpenHarmony C++ shared state?

Thread safety validation for OpenHarmony C++ shared state detects container thread-safety issues, iterator invalidation, deadlock risks, and TOCTOU patterns by enforcing consistent locking mechanisms.

How do I prevent PII and pointer leakage in OpenHarmony C++ logs?

Prevent PII and pointer leakage in OpenHarmony C++ logs by enforcing strict redaction of sensitive data and memory addresses before outputting any diagnostic logging information.

Can I use this security review to audit an OpenHarmony xxxService.cpp and xxxStub.cpp pair?

Yes, you can audit an xxxService.cpp and its corresponding xxxStub.cpp pair to systematically identify IPC input handling issues, concurrency hazards, sensitive logging, and strict permission validation checks.

What is the best way to validate caller permissions for privileged OpenHarmony operations?

The best way to validate caller permissions for privileged OpenHarmony operations is to systematically flag and enforce strict permission checks across all code paths handling untrusted IPC or network inputs.