17 Star 26 Fork 9

OpenHarmony / security_services_app_verify

日志打印不合理

Task
Done
Gymee  Opened this issue

该问题是怎么引起的?

不涉及

重现步骤

不涉及

报错信息

不涉及

int InitVerify(FileRead *file, const char *filePath, int *handle)
{
    if (handle == NULL || file == NULL || filePath == NULL) {
        LOG_ERROR("file open error"); // 应该是invalid parameters之类的,而不应该是open error
        return V_ERR_FILE_OPEN;  // 应该也不是V_ERR_FILE_OPEN
    }

    // ....

    if (*handle < 0) {
        APPV_FREE(path);
        LOG_PRINT_STR("file open error %s", path); // 这个才是open error
        return V_ERR_FILE_OPEN;
    }
Attachments
total 3 participants

Comments (4)

Ma Peng Fei 2020-09-17 15:54

我觉得没问题

Gymee 2020-09-17 16:05

我觉得没问题

@Ma Peng Fei 这也不是啥大问题 不同的人看法不一样哈

HJ 2020-09-17 16:35 member

@Ma Peng Fei 这也不是啥大问题 不同的人看法不一样哈

@Gymee 日志打印内容需要明确,这个还是要改的。谢谢反馈!这个回头我改改,也非常欢迎你共享代码 :smile:

Gymee 2020-09-17 16:37

@Gymee 日志打印内容需要明确,这个还是要改的。谢谢反馈!这个回头我改改,也非常欢迎你共享代码 :smile:

@HJ 好的 谢谢你们

Sign in to comment

Assignees
Labels
Not set
Projects
Milestones
Branches
Planed to start
Not set
Planed to end
Not set
Top level
Priority
1
https://git.oschina.net/openharmony/security_services_app_verify.git
git@git.oschina.net:openharmony/security_services_app_verify.git
openharmony
security_services_app_verify
security_services_app_verify

Search

103611 48b8ff67 1899542 103622 4d02230c 1899542