11 Star 87 Fork 746

OpenHarmony / startup_init

 / 详情

init_read_cfg.c的宏设置建议放到同名.h头文件中

已拒绝
缺陷
创建于  
2020-09-18 23:18

该问题是怎么引起的?

static const long MAX_JSON_FILE_LEN = 102400; // max init.cfg size 100KB
#define MAX_SERVICES_CNT_IN_FILE 100
#define MAX_CAPS_CNT_FOR_ONE_SERVICE 100
#define UID_STR_IN_CFG "uid"
#define GID_STR_IN_CFG "gid"
#define ONCE_STR_IN_CFG "once"
#define IMPORTANT_STR_IN_CFG "importance"
#define BIN_SH_NOT_ALLOWED "/bin/sh"

重现步骤

报错信息

评论 (2)

mxs65221447 创建了缺陷
展开全部操作日志

我觉得在init_read_cfg.c中定义上面这些常量并没有啥不妥哈:

1, .h里面的东西是对外的,比如接口、类型等;我不对外的都是定义在.c里面的,我内部的函数、类型、常量是不需要在.h里面的;.h和.c是有明确分工的:一个对外接口和一个实现封装,所有.h不能啥都放哈

2,另外static const long MAX_JSON_FILE_LEN = 102400; // max init.cfg size 100KB,这个不适合放在头文件中

@mxs65221447

模块内部使用的,不建议公开出去。

handy 负责人设置为熊磊
熊磊 任务状态待办的 修改为已拒绝

登录 后才可以发表评论

状态
负责人
项目
里程碑
Pull Requests
关联的 Pull Requests 被合并后可能会关闭此 issue
分支
开始日期   -   截止日期
-
置顶选项
优先级
预计工期 (小时)
参与者(3)
8039110 gymee 1642069305
C
1
https://gitee.com/openharmony/startup_init.git
git@gitee.com:openharmony/startup_init.git
openharmony
startup_init
startup_init

搜索帮助