storage_fun.c修改

Question

从1065行开始

#ifdef WITH_HTTPD
                {
                char *pHttpTrunkSize;
                int64_t http_trunk_size;

                if ((result=fdfs_http_params_load(&iniContext,
                                filename, &g_http_params)) != 0)
                {
                        break;
                }

                               if(g_http_params.server_port > 0 )
                                               g_http_port = g_http_params.server_port;

                pHttpDomain = iniGetStrValue(NULL,
                        "http.domain_name", &iniContext);
                if (pHttpDomain == NULL)
                {
                        *g_http_domain = '';
                }
                else
                {
                        snprintf(g_http_domain, sizeof(g_http_domain),
                                "%s", pHttpDomain);
                }

Answer 1

哦，谢谢斑竹答疑，这段时间在努力的学习:wink:

Answer 2

斑竹，文件ni_file_reader.c中下函数应增加strlen(pValue)的检测，不然为空也不会返回nDefaultValue的

...
liuxy 发表于 2010-05-24 16:27

目前的设计是这样的：只有当配置项不存在时，才会返回缺省值。
配置项存在，但值为空，不会返回缺省值。

Answer 3

验证了一下，其中：
iniGetDoubleValue
iniGetInt64Value
都存在这样的问题

Answer 4

斑竹，文件ni_file_reader.c中下函数应增加strlen(pValue)的检测，不然为空也不会返回nDefaultValue的

int iniGetIntValue(const char *szSectionName, const char *szItemName,
                IniContext *pContext, const int nDefaultValue)
{
        char *pValue;

        pValue = iniGetStrValue(szSectionName, szItemName, pContext);
        if (pValue != NULL && strlen(pValue)>0)
        {
                return atoi(pValue);
        }
       
  return nDefaultValue;
}

Answer 5

感觉是需要修改啊，即使在编译屏蔽 WITH_HTTPD时，程序中有好几处都会读取http port 和http.domain_name
斑 ...
liuxy 发表于 2010-05-22 18:09

是会读取，但不会要求一定要配置这两个参数啊。
storage server上可以安装nginx等其他web server，和tracker server内置的HTTP server配合使用。

Answer 6

Answer 7

感觉是需要修改啊，即使在编译屏蔽 WITH_HTTPD时，程序中有好几处都会读取http port 和http.domain_name
斑竹，有空答疑一下么