Skip to content

Commit 427f2e8

Browse files
committed
fixed:sometimes read configure memory leak
1 parent ec297a4 commit 427f2e8

File tree

1 file changed

+3
-12
lines changed

1 file changed

+3
-12
lines changed

XEngine_Source/XEngine_ModuleConfigure/ModuleConfigure_Json/ModuleConfigure_Json.cpp

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -61,21 +61,12 @@ bool CModuleConfigure_Json::ModuleConfigure_Json_File(LPCXSTR lpszConfigFile, XE
6161
Config_dwErrorCode = ERROR_MODULE_CONFIGURE_JSON_OPENFILE;
6262
return false;
6363
}
64-
size_t nCount = 0;
65-
XCHAR tszMsgBuffer[4096];
66-
while (1)
67-
{
68-
size_t nRet = fread(tszMsgBuffer + nCount, 1, 2048, pSt_File);
69-
if (nRet <= 0)
70-
{
71-
break;
72-
}
73-
nCount += nRet;
74-
}
64+
XCHAR tszMsgBuffer[8192];
65+
size_t nRet = fread(tszMsgBuffer, 1, sizeof(tszMsgBuffer), pSt_File);
7566
fclose(pSt_File);
7667
//开始解析配置文件
7768
std::unique_ptr<Json::CharReader> const pSt_JsonReader(st_JsonBuilder.newCharReader());
78-
if (!pSt_JsonReader->parse(tszMsgBuffer, tszMsgBuffer + nCount, &st_JsonRoot, &st_JsonError))
69+
if (!pSt_JsonReader->parse(tszMsgBuffer, tszMsgBuffer + nRet, &st_JsonRoot, &st_JsonError))
7970
{
8071
Config_IsErrorOccur = true;
8172
Config_dwErrorCode = ERROR_MODULE_CONFIGURE_JSON_PARSE;

0 commit comments

Comments
 (0)