storage_fun.c修改

liuxy

从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 = '\0';
                }
                else
                {
                        snprintf(g_http_domain, sizeof(g_http_domain), \
                                "%s", pHttpDomain);
                }

liuxy

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

lulu019

happy_fish100

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

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

liuxy

斑竹，文件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;
}

liuxy

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

happy_fish100

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


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

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

liuxy

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