mirror of
https://github.com/openharmony/ability_ability_runtime.git
synced 2026-08-24 22:21:36 -04:00
!19396 merge revert into master
revert customized buffer consumer Created-by: zhangzhichao0091 Commit-by: zhangzhichao0091 Merged-by: openharmony_ci Description: **IssueNo**:https://gitcode.com/openharmony/ability_ability_runtime/issues/15305 **Description**: **稳定性自检:** | 自检项 | 自检结果 | | ------------------------------------------------------------ | -------- | | 涉及跨进程调用的相关操作需要抛至主线程或加锁防止并发 |pass | | 成员变量进行赋值或创建需要排查并发 | pass | | 谨慎在lambda表达式中使用引用捕获 | pass | | 谨慎在未经拷贝的情况下使用外部传入的string、C字符串 | pass | | map\vector\list\set等stl模板类使用时需要排查并发 | pass | | 谨慎考虑加锁范围 | pass | | 在IPC通信中谨慎使用同步通信方式 | pass | | 禁止传递this指针至其他模块或线程(特别是eventhandler任务) | pass | | 禁止将外部传入的裸指针在内部直接构造智能指针 | pass | | 禁止多个独立创建的智能指针管理同一地址 | pass | | 禁止在析构函数中抛异步任务 | pass | | 禁止js对象在非js线程(例如在IPC线程)创建、使用或销毁 | pass | | 禁止在对外接口中未经判空直接使用外部传入的指针 | pass | | 禁止接口返回局部变量引用 | pass | | 禁止在信号函数中加锁 | pass | | 禁止在关键流程(SA启动、应用启动等主流程)执行耗时的操作 | pass | | 禁止将同一个cpp编译在不同的so中 | pass | **安全编码自检:** | 自检项 | 自检结果 | | -------------------------------------------------------------- | -------- | | 裸指针避免通过隐式转换构造为sptr | pass | | json对象在取值之前必须先判断类型,避免类型不匹配 | pass | | 序列化时必须对传入的数组大小进行校验,避免出现超大数组 | pass | | 避免使用未明确位宽的整型,选择使用int8_t、uint8_t等类型 | pass | | 外部传入的路径要做规范化校验,对路径中的.、..、../等特殊字符严格校验 | pass | | 指针变量、表示资源描述符的变量、bool变量必须赋初值 | pass | | readParcelable获取的对象使用前需要判空 | pass | | 分配和释放内存的函数需要成对出现 | pass | | 申请内存后异常退出前需要及时进行内存释放 | pass | | 内存申请前必须对内存大小进行合法性校验 | pass | | 内存分配后必须判断是否成功 | pass | | 禁止使用realloc、alloca函数 | pass | | 禁止打印文件路径、口令等敏感信息,如有需要,使用private修饰 | pass | | 禁止打印内存地址 | pass | | 整数之间运算时必须严格检查,确保不会出现溢出、反转、除0 | pass | | 禁止对有符号整数进行位操作符运算 | pass | | 禁止对指针进行逻辑或位运算 | pass | | 循环次数如果收外部数据控制,需要检验其合法性 | pass | | 禁止使用内存操作类危险函数,需要使用安全函数 | pass | | 谨慎使用不可重入函数 | pass | | 必须检查安全函数的返回值,并进行正确处理 | pass | | 禁止仅通过TokenType类型判断绕过权限校验 | pass | **TDD Result**: **XTS Result**: ### 是否已执行L0用例 - [ ] 已验证 - [x] 不涉及。如不涉及,请写明理由 See merge request: openharmony/ability_ability_runtime!19396
This commit is contained in:
@@ -2273,13 +2273,6 @@ void MainThread::ProcessExit(const ProcessExitInfo& info)
|
||||
}
|
||||
|
||||
#if defined(NWEB) && defined(NWEB_GRAPHIC)
|
||||
class CustomizedBufferConsumerListener : public IBufferConsumerListener {
|
||||
public:
|
||||
CustomizedBufferConsumerListener() {}
|
||||
~CustomizedBufferConsumerListener() {}
|
||||
|
||||
void OnBufferAvailable() override {}
|
||||
};
|
||||
void MainThread::HandleNWebPreload()
|
||||
{
|
||||
if (!mainHandler_) {
|
||||
@@ -2292,21 +2285,21 @@ void MainThread::HandleNWebPreload()
|
||||
TAG_LOGE(AAFwkTag::APPKIT, "init NWebEngine failed");
|
||||
return;
|
||||
}
|
||||
cSurface_ = IConsumerSurface::Create(NWEB_SURFACE_NODE_NAME);
|
||||
if (cSurface_ == nullptr) {
|
||||
TAG_LOGE(AAFwkTag::APPKIT, "create cSurface failed");
|
||||
Rosen::RSSurfaceNodeConfig config;
|
||||
config.SurfaceNodeName = NWEB_SURFACE_NODE_NAME;
|
||||
preloadSurfaceNode_ = Rosen::RSSurfaceNode::Create(config, false);
|
||||
if (!preloadSurfaceNode_) {
|
||||
TAG_LOGE(AAFwkTag::APPKIT, "preload surface node is nullptr");
|
||||
return;
|
||||
}
|
||||
auto producer = cSurface_->GetProducer();
|
||||
pSurface_ = Surface::CreateSurfaceAsProducer(producer);
|
||||
if (pSurface_ == nullptr) {
|
||||
TAG_LOGE(AAFwkTag::APPKIT, "create pSurface failed");
|
||||
auto surface = preloadSurfaceNode_->GetSurface();
|
||||
if (!surface) {
|
||||
TAG_LOGE(AAFwkTag::APPKIT, "preload surface is nullptr");
|
||||
preloadSurfaceNode_ = nullptr;
|
||||
return;
|
||||
}
|
||||
sptr<IBufferConsumerListener> listener = sptr<CustomizedBufferConsumerListener>::MakeSptr();
|
||||
cSurface_->RegisterConsumerListener(listener);
|
||||
auto initArgs = std::make_shared<NWeb::NWebEngineInitArgsImpl>();
|
||||
preloadNWeb_ = NWeb::NWebAdapterHelper::Instance().CreateNWeb(pSurface_, initArgs,
|
||||
preloadNWeb_ = NWeb::NWebAdapterHelper::Instance().CreateNWeb(surface, initArgs,
|
||||
NWEB_SURFACE_SIZE, NWEB_SURFACE_SIZE, false);
|
||||
if (!preloadNWeb_) {
|
||||
TAG_LOGE(AAFwkTag::APPKIT, "create preLoadNWeb failed");
|
||||
|
||||
@@ -790,8 +790,7 @@ private:
|
||||
std::string pluginDefaultNamespaceLdDictionary_ = "";
|
||||
RuntimeUpdateParam runtimeUpdateParam_;
|
||||
#if defined(NWEB) && defined(NWEB_GRAPHIC)
|
||||
sptr<OHOS::IConsumerSurface> cSurface_ = nullptr;
|
||||
sptr<OHOS::Surface> pSurface_ = nullptr;
|
||||
Rosen::RSSurfaceNode::SharedPtr preloadSurfaceNode_ = nullptr;
|
||||
std::shared_ptr<NWeb::NWeb> preloadNWeb_ = nullptr;
|
||||
#endif
|
||||
|
||||
|
||||
@@ -66,8 +66,7 @@ public:
|
||||
MOCK_METHOD1(NotifyMemoryLevel, void(int32_t level));
|
||||
MOCK_METHOD1(SetAppWindowTitle, void(const std::string &title));
|
||||
MOCK_METHOD1(SetAppWindowIcon, void(const std::shared_ptr<Media::PixelMap> &pixelMap));
|
||||
MOCK_METHOD4(PreInitializeForm, void(OHOS::Rosen::Window *window, const std::string &url, napi_value storage,
|
||||
sptr<IRemoteObject> connectToRender));
|
||||
MOCK_METHOD3(PreInitializeForm, void(OHOS::Rosen::Window *window, const std::string &url, napi_value storage));
|
||||
MOCK_METHOD0(RunFormPage, void());
|
||||
MOCK_METHOD0(GetFormRootNode, std::shared_ptr<Rosen::RSSurfaceNode>());
|
||||
MOCK_METHOD1(UpdateFormData, void(const std::string &data));
|
||||
|
||||
Reference in New Issue
Block a user