mirror of
https://github.com/openharmony/ability_form_fwk.git
synced 2026-08-24 07:47:15 -04:00
openharmony_ci
2abbb074c8
代码检视问题修改 Created-by: yzy-15 Commit-by: yzy-15 Merged-by: openharmony_ci Description: **IssueNo**: #3164 **Description**: 代码检视问题修改 **稳定性自检：** | 自检项 | 自检结果 | | ------------------------------------------------------------ | -------- | | 涉及跨进程调用的相关操作需要抛至主线程或加锁防止并发 | 已自检：P1-12 ParamCommonEvent加mutex，P1-11 LoadDataFromDb加锁，P1-9 configMutex_保护configUpdateTime_ | | 成员变量进行赋值或创建需要排查并发 | 已自检：atTimerWakeUpTime_/limiterTimerReportId_/reportDiskUseTimerId_/isInitialized_/hasCachedConfig_改为atomic | | 谨慎在lambda表达式中使用引用捕获 | 已自检：P1-5 FormProxyRegistry callback从[this]改为[weak=weak_from_this()] | | 谨慎在未经拷贝的情况下使用外部传入的string、C字符串 | 不涉及 | | map\vector\list\set等stl模板类使用时需要排查并发 | 已自检：P1-3 FormRouterProxyMgr TOCTOU修复，P1-11 LoadDataFromDb map加锁，P1-12 eventHandles_加mutex | | 谨慎考虑加锁范围 | 已自检：P1-5析构函数移除冗余锁，P1-8 CreateLimiterTimer锁内check-create，P1-12 handler在锁外调用 | | 在IPC通信中谨慎使用同步通信方式 | 不涉及 | | 禁止传递this指针至其他模块或线程（特别是eventhandler任务） | 已自检：P1-5 callback不再捕获this，改用weak_ptr | | 禁止将外部传入的裸指针在内部直接构造智能指针 | 不涉及 | | 禁止多个独立创建的智能指针管理同一地址 | 不涉及 | | 禁止在析构函数中抛异步任务 | 已自检：P5-3 FormShareMgr析构移除shared_from_this()，改用OnStop()中调UnregisterObserver() | | 禁止js对象在非js线程（例如在IPC线程）创建、使用或销毁 | 不涉及 | | 禁止在对外接口中未经判空直接使用外部传入的指针 | 已自检：P2-9 callerToken判空，P4-1 connect判空 | | 禁止接口返回局部变量引用 | 不涉及 | | 禁止在信号函数中加锁 | 不涉及 | | 禁止在关键流程（SA启动、应用启动等主流程）执行耗时的操作 | 不涉及 | | 禁止将同一个cpp编译在不同的so中 | 不涉及 | **安全编码自检：** | 自检项 | 自检结果 | | -------------------------------------------------------------- | -------- | | 裸指针避免通过隐式转换构造为sptr | 不涉及 | | json对象在取值之前必须先判断类型，避免类型不匹配 | 不涉及 | | 序列化时必须对传入的数组大小进行校验，避免出现超大数组 | 不涉及 | | 避免使用未明确位宽的整型，选择使用int8_t、uint8_t等类型 | 已自检：P1-1 atTimerWakeUpTime_从long改为int64_t | | 外部传入的路径要做规范化校验，对路径中的.、..、../等特殊字符严格校验 | 不涉及 | | 指针变量、表示资源描述符的变量、bool变量必须赋初值 | 已自检：P4-2 castFormsCount/tempFormsCount赋初值0 | | readParcelable获取的对象使用前需要判空 | 不涉及 | | 分配和释放内存的函数需要成对出现 | 不涉及 | | 申请内存后异常退出前需要及时进行内存释放 | 不涉及 | | 内存申请前必须对内存大小进行合法性校验 | 不涉及 | | 内存分配后必须判断是否成功 | 不涉及 | | 禁止使用realloc、alloca函数 | 不涉及 | | 禁止打印文件路径、口令等敏感信息，如有需要，使用private修饰 | 不涉及 | | 禁止打印内存地址 | 不涉及 | | 整数之间运算时必须严格检查，确保不会出现溢出、反转、除0 | 已自检：P3-1/P3-2 errno检查+范围校验，P3-3 NANO_PER_SECOND溢出检查 | | 禁止对有符号整数进行位操作符运算 | 不涉及 | | 禁止对指针进行逻辑或位运算 | 不涉及 | | 循环次数如果收外部数据控制，需要检验其合法性 | 不涉及 | | 禁止使用内存操作类危险函数，需要使用安全函数 | 不涉及 | | 谨慎使用不可重入函数 | 已自检：P3-1/P3-2 errno为thread-local，strtol/strtoll线程安全 | | 必须检查安全函数的返回值，并进行正确处理 | 已自检：P2-6 GetOsAccountLocalIdFromUid，P2-8 GetFormLock，P2-10 GetFormRecord，P5-1 PushFormEvent | | 禁止仅通过TokenType类型判断绕过权限校验 | 不涉及 | **TDD Result**: 已更新测试文件：fms_form_timer_mgr_test.cpp删除IsDynamicTimerExpired用例，formtimermgrtwo_fuzzer.cpp删除fuzzer引用，fms_form_surface_info_test.cpp更新ReadFromParcel测试 **XTS Result**: 低风险。改动为内部实现变更（atomic类型、mutex锁范围、输入校验），对XTS接口不可见 ### 是否已执行L0用例 - [ ] 已验证 - [x] 不涉及。如不涉及，请写明理由：本PR为代码检视问题修复，涉及并发安全加固、输入校验、整数溢出防护等内部实现变更，不改变对外接口签名和行为 See merge request: openharmony/ability_form_fwk!3815