{"id":409703,"date":"2024-06-29T20:57:15","date_gmt":"2024-06-29T20:57:15","guid":{"rendered":"http:\/\/savepearlharbor.com\/?p=409703"},"modified":"-0001-11-30T00:00:00","modified_gmt":"-0001-11-29T21:00:00","slug":"","status":"publish","type":"post","link":"https:\/\/savepearlharbor.com\/?p=409703","title":{"rendered":"\u041f\u0440\u0438\u043a\u043b\u044e\u0447\u0435\u043d\u0438\u044f \u043a\u0430\u043f\u0438\u0442\u0430\u043d\u0430 \u0411\u043b\u0430\u0434\u0430: \u043f\u043e\u0442\u043e\u043d\u0435\u0442 \u043b\u0438 \u0410\u0440\u0430\u0431\u0435\u043b\u043b\u0430?<\/span>"},"content":{"rendered":"
<\/div>\n
\n
\n
\n
\n

\u041d\u0435\u0434\u0430\u0432\u043d\u043e \u0432 \u0441\u0435\u0442\u0438 \u043f\u043e\u044f\u0432\u0438\u043b\u0430\u0441\u044c \u043d\u043e\u0432\u043e\u0441\u0442\u044c \u043e \u0442\u043e\u043c, \u0447\u0442\u043e \u0431\u044b\u043b \u043e\u0442\u043a\u0440\u044b\u0442 \u0438\u0441\u0445\u043e\u0434\u043d\u044b\u0439 \u043a\u043e\u0434 \u0438\u0433\u0440\u044b «\u041f\u0440\u0438\u043a\u043b\u044e\u0447\u0435\u043d\u0438\u044f \u043a\u0430\u043f\u0438\u0442\u0430\u043d\u0430 \u0411\u043b\u0430\u0434\u0430». \u041c\u044b \u043d\u0435 \u0441\u043c\u043e\u0433\u043b\u0438 \u043f\u0440\u043e\u0439\u0442\u0438 \u043c\u0438\u043c\u043e \u0438 \u043f\u0440\u043e\u0432\u0435\u0440\u0438\u043b\u0438 \u0435\u0433\u043e \u043a\u0430\u0447\u0435\u0441\u0442\u0432\u043e \u0441 \u043f\u043e\u043c\u043e\u0449\u044c\u044e PVS-Studio. \u041f\u043e\u0442\u043e\u043d\u0435\u0442 \u043b\u0438 \u043b\u0435\u0433\u0435\u043d\u0434\u0430\u0440\u043d\u044b\u0439 \u043a\u043e\u0440\u0430\u0431\u043b\u044c \u043a\u0430\u043f\u0438\u0442\u0430\u043d\u0430 \u0411\u043b\u0430\u0434\u0430 \u043e\u0442 \u043d\u0430\u0439\u0434\u0435\u043d\u043d\u044b\u0445 \u0431\u0430\u0433\u043e\u0432? \u0414\u0430\u0432\u0430\u0439\u0442\u0435 \u0443\u0437\u043d\u0430\u0435\u043c!<\/p>\n

<\/figcaption><\/figure>\n

\u041e \u043f\u0440\u043e\u0435\u043a\u0442\u0435<\/h3>\n

\u0418\u0433\u0440\u0430 «\u041f\u0440\u0438\u043a\u043b\u044e\u0447\u0435\u043d\u0438\u044f \u043a\u0430\u043f\u0438\u0442\u0430\u043d\u0430 \u0411\u043b\u0430\u0434\u0430» \u0441\u043e\u0437\u0434\u0430\u043d\u0430 \u0441\u0442\u0443\u0434\u0438\u0435\u0439 «1C \u041c\u043e\u0440\u0441\u043a\u043e\u0439 \u0432\u043e\u043b\u043a» \u043f\u043e \u043c\u043e\u0442\u0438\u0432\u0430\u043c \u043a\u043d\u0438\u0433 \u0420\u0430\u0444\u0430\u044d\u043b\u044f \u0421\u0430\u0431\u0430\u0442\u0438\u043d\u0438. \u041e\u043d\u0430 \u043f\u043e\u0432\u0435\u0441\u0442\u0432\u0443\u0435\u0442 \u043e \u043f\u0440\u0438\u043a\u043b\u044e\u0447\u0435\u043d\u0438\u044f\u0445 \u0433\u043b\u0430\u0432\u043d\u043e\u0433\u043e \u0433\u0435\u0440\u043e\u044f \u044d\u0442\u0438\u0445 \u043f\u0440\u043e\u0438\u0437\u0432\u0435\u0434\u0435\u043d\u0438\u0439, \u043a\u0430\u043f\u0438\u0442\u0430\u043d\u0430 \u041f\u0438\u0442\u0435\u0440\u0430 \u0411\u043b\u0430\u0434\u0430. \u0414\u0435\u0439\u0441\u0442\u0432\u0438\u0435 \u0438\u0433\u0440\u044b \u0440\u0430\u0437\u0432\u043e\u0440\u0430\u0447\u0438\u0432\u0430\u0435\u0442\u0441\u044f \u0432 \u0441\u0440\u0435\u0434\u043d\u0435\u0432\u0435\u043a\u043e\u0432\u043e\u0439 \u043d\u043e\u0432\u043e\u0439 \u0410\u043d\u0433\u043b\u0438\u0438. \u0413\u043b\u0430\u0432\u043d\u044b\u0439 \u0433\u0435\u0440\u043e\u0439, \u0432\u043e\u0435\u043d\u043d\u044b\u0439 \u0432\u0440\u0430\u0447 \u043f\u043e \u043e\u0431\u0440\u0430\u0437\u043e\u0432\u0430\u043d\u0438\u044e, \u0431\u044b\u043b \u043d\u0435\u0441\u043f\u0440\u0430\u0432\u0435\u0434\u043b\u0438\u0432\u043e \u043e\u0441\u0443\u0436\u0434\u0451\u043d \u0438 \u0441\u043e\u0441\u043b\u0430\u043d \u043d\u0430 \u043e\u0441\u0442\u0440\u043e\u0432 \u0411\u0430\u0440\u0431\u0430\u0434\u043e\u0441. \u0421 \u043e\u0441\u0442\u0440\u043e\u0432\u0430 \u0435\u043c\u0443 \u0443\u0434\u0430\u0451\u0442\u0441\u044f \u0431\u0435\u0436\u0430\u0442\u044c, \u043f\u043e\u0441\u043b\u0435 \u0447\u0435\u0433\u043e, \u0437\u0430\u0445\u0432\u0430\u0442\u0438\u0432 \u0438\u0441\u043f\u0430\u043d\u0441\u043a\u0438\u0439 \u043a\u043e\u0440\u0430\u0431\u043b\u044c, \u043e\u043d \u0441\u0442\u0430\u043d\u043e\u0432\u0438\u0442\u0441\u044f \u0441\u0430\u043c\u044b\u043c \u0437\u043d\u0430\u043c\u0435\u043d\u0438\u0442\u044b\u043c \u043f\u0438\u0440\u0430\u0442\u043e\u043c \u0442\u043e\u0433\u043e \u0432\u0440\u0435\u043c\u0435\u043d\u0438.<\/p>\n

\u041f\u0440\u043e\u0435\u043a\u0442 \u0431\u044b\u043b \u0437\u0430\u0432\u0435\u0440\u0448\u0451\u043d \u0432 2010 \u0433\u043e\u0434\u0443, \u043d\u043e \u0442\u0430\u043a \u0438 \u043d\u0435 \u0443\u0432\u0438\u0434\u0435\u043b \u0441\u0432\u0435\u0442. \u0412\u0441\u0451 \u043f\u043e\u0442\u043e\u043c\u0443, \u0447\u0442\u043e \u043a\u043e\u043c\u043f\u0430\u043d\u0438\u0438 1\u0421 \u0438 Playlogic \u043d\u0435 \u0441\u043c\u043e\u0433\u043b\u0438 \u0440\u0435\u0448\u0438\u0442\u044c, \u043a\u043e\u043c\u0443 \u043f\u0440\u0438\u043d\u0430\u0434\u043b\u0435\u0436\u0430\u0442 \u043f\u0440\u0430\u0432\u0430 \u043d\u0430 \u0438\u0433\u0440\u0443. \u0414\u0430\u043b\u0435\u0435 \u0445\u043e\u0434\u0438\u043b\u0438 \u043d\u043e\u0432\u043e\u0441\u0442\u0438, \u0447\u0442\u043e \u0444\u0438\u043d\u0430\u043b\u044c\u043d\u0430\u044f \u0441\u0431\u043e\u0440\u043a\u0430 \u0438\u0433\u0440\u044b \u0431\u0435\u0437\u0432\u043e\u0437\u0432\u0440\u0430\u0442\u043d\u043e \u0443\u0442\u0435\u0440\u044f\u043d\u0430. \u041e\u0434\u043d\u0430\u043a\u043e \u043d\u0435 \u0442\u0430\u043a \u0434\u0430\u0432\u043d\u043e \u043f\u043e\u044f\u0432\u0438\u043b\u043e\u0441\u044c \u043d\u0435\u0441\u043a\u043e\u043b\u044c\u043a\u043e \u043d\u043e\u0432\u043e\u0441\u0442\u0435\u0439 \u043e \u0442\u043e\u043c, \u0447\u0442\u043e \u044d\u0442\u043e \u043d\u0435 \u0442\u0430\u043a. \u041f\u043e\u044f\u0432\u0438\u043b\u043e\u0441\u044c \u0432\u0438\u0434\u0435\u043e \u0441 \u043f\u0440\u043e\u0445\u043e\u0436\u0434\u0435\u043d\u0438\u0435\u043c \u043d\u0430 YouTube \u0438 \u0440\u0430\u0431\u043e\u0442\u0430\u044e\u0449\u0438\u0439 \u0431\u0438\u043b\u0434 \u0438\u0433\u0440\u044b \u043d\u0430 \u043e\u0434\u043d\u043e\u043c \u0438\u0437\u0432\u0435\u0441\u0442\u043d\u043e\u043c \u0442\u0440\u0435\u043a\u0435\u0440\u0435. \u041f\u043e\u0437\u0436\u0435 \u043d\u0430 GitHub \u0432\u044b\u043b\u043e\u0436\u0438\u043b\u0438 \u0438\u0441\u0445\u043e\u0434\u043d\u0438\u043a\u0438<\/a> \u043f\u043e\u0434 \u043b\u0438\u0446\u0435\u043d\u0437\u0438\u0435\u0439 GPLv3.<\/p>\n

<\/div>\n

\u0418\u0442\u0430\u043a, \u043f\u043e\u0441\u043b\u0435 \u0442\u043e\u0433\u043e \u043a\u0430\u043a \u043c\u044b \u043f\u043e\u0437\u043d\u0430\u043a\u043e\u043c\u0438\u043b\u0438\u0441\u044c \u0441 \u043f\u0440\u043e\u0435\u043a\u0442\u043e\u043c, \u043f\u0440\u0435\u0434\u043b\u0430\u0433\u0430\u044e \u043f\u043e\u0441\u043c\u043e\u0442\u0440\u0435\u0442\u044c \u043a\u0430\u043a\u0438\u0435 \u0438\u043d\u0442\u0435\u0440\u0435\u0441\u043d\u044b\u0435 \u043e\u0448\u0438\u0431\u043a\u0438 \u0432 \u043d\u0451\u043c \u0443\u0434\u0430\u043b\u043e\u0441\u044c \u043e\u0431\u043d\u0430\u0440\u0443\u0436\u0438\u0442\u044c.<\/p>\n

\u0420\u0435\u0437\u0443\u043b\u044c\u0442\u0430\u0442\u044b \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0438<\/h3>\n

\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435 N1<\/h4>\n

V1075<\/a> The function expects the file to be opened for writing, but it was opened for reading. Debugger.cpp 172<\/p>\n

\u041c\u043e\u0436\u0435\u043c \u043f\u043e\u0437\u0434\u0440\u0430\u0432\u0438\u0442\u044c \u043c\u043e\u0435\u0433\u043e \u043a\u043e\u043b\u043b\u0435\u0433\u0443 \u0421\u0435\u0440\u0433\u0435\u044f \u041b\u0430\u0440\u0438\u043d\u0430 \u0441 \u043f\u0435\u0440\u0432\u044b\u043c \u0442\u0440\u043e\u0444\u0435\u0439\u043d\u044b\u043c \u0441\u0440\u0430\u0431\u0430\u0442\u044b\u0432\u0430\u043d\u0438\u0435\u043c \u0435\u0433\u043e \u0434\u0438\u0430\u0433\u043d\u043e\u0441\u0442\u0438\u0447\u0435\u0441\u043a\u043e\u0433\u043e \u043f\u0440\u0430\u0432\u0438\u043b\u0430, \u0432\u044b\u0448\u0435\u0434\u0448\u0435\u0433\u043e \u0432 \u0440\u0435\u043b\u0438\u0437\u0435 PVS-Studio 7.15. \u0414\u0430\u043d\u043d\u043e\u0435 \u0434\u0438\u0430\u0433\u043d\u043e\u0441\u0442\u0438\u0447\u0435\u0441\u043a\u043e\u0435 \u043f\u0440\u0430\u0432\u0438\u043b\u043e \u0440\u0443\u0433\u0430\u0435\u0442\u0441\u044f \u043d\u0430 \u0437\u0430\u043f\u0438\u0441\u044c \u0432 \u0444\u0430\u0439\u043b\u044b, \u043e\u0442\u043a\u0440\u044b\u0442\u044b\u0435 \u0434\u043b\u044f \u0447\u0442\u0435\u043d\u0438\u044f, \u0438 \u043d\u0430\u043e\u0431\u043e\u0440\u043e\u0442. \u0427\u0435\u0441\u0442\u043d\u043e, \u043c\u044b \u0434\u0430\u0436\u0435 \u043d\u0435 \u0434\u0443\u043c\u0430\u043b\u0438, \u0447\u0442\u043e \u043e\u043d\u043e \u043a\u043e\u0433\u0434\u0430-\u043d\u0438\u0431\u0443\u0434\u044c \u0441\u0440\u0430\u0431\u043e\u0442\u0430\u0435\u0442. \u041f\u0440\u0438\u0432\u0435\u0434\u0443 \u043f\u0440\u043e\u0431\u043b\u0435\u043c\u043d\u0443\u044e \u0444\u0443\u043d\u043a\u0446\u0438\u044e \u043f\u043e\u043b\u043d\u043e\u0441\u0442\u044c\u044e:<\/p>\n

void appDebuger::CopyDump(const char * src, DWORD from, const char * dst) {   FILE * file = fopen(src, \"rb\");   if(file)   {     fseek(file, 0, SEEK_END);     DWORD size = ftell(file);     fseek(file, from, SEEK_SET);     if(size > from)     {       FILE * to = fopen(dst, \"a+b\");       if(to)       {         char buf[128];         while(from < size)         {                     DWORD s = size - from;           if(s > sizeof(buf)) s = sizeof(buf);           memset(buf, ' ', sizeof(buf));           fread(buf, s, 1, file);           fwrite(buf, s, 1, file);            \/\/ <=           from += s;         }         fclose(to);       }     }     fclose(file);   } } <\/code><\/pre>\n
\u0418\u0442\u0430\u043a, \u0444\u0443\u043d\u043a\u0446\u0438\u044f \u0434\u043e\u043b\u0436\u043d\u0430 \u043f\u0440\u043e\u0447\u0438\u0442\u0430\u0442\u044c \u0434\u0430\u043d\u043d\u044b\u0435 \u0438\u0437 \u043e\u0434\u043d\u043e\u0433\u043e \u0444\u0430\u0439\u043b\u0430 \u043f\u043e \u043f\u0443\u0442\u0438 src<\/em>, \u043d\u0430\u0447\u0438\u043d\u0430\u044f \u0441 \u043f\u043e\u0437\u0438\u0446\u0438\u0438* from*, \u0438 \u0437\u0430\u043f\u0438\u0441\u0430\u0442\u044c \u0438\u0445 \u0432 \u0434\u0440\u0443\u0433\u043e\u0439 \u0444\u0430\u0439\u043b \u043f\u043e \u043f\u0443\u0442\u0438 dst<\/em>. \u0417\u0430 \u0438\u0441\u0445\u043e\u0434\u043d\u044b\u0439 \u0444\u0430\u0439\u043b \u043e\u0442\u0432\u0435\u0447\u0430\u0435\u0442 \u043f\u0435\u0440\u0435\u043c\u0435\u043d\u043d\u0430\u044f file<\/em>, \u0437\u0430 \u0440\u0435\u0437\u0443\u043b\u044c\u0442\u0438\u0440\u0443\u044e\u0449\u0438\u0439 \u0444\u0430\u0439\u043b \u043f\u0435\u0440\u0435\u043c\u0435\u043d\u043d\u0430\u044f to<\/em>.<\/p>\n
\u041a \u0441\u043e\u0436\u0430\u043b\u0435\u043d\u0438\u044e, \u0432 \u043a\u043e\u0434\u0435 \u043f\u0440\u043e\u0438\u0441\u0445\u043e\u0434\u0438\u0442 \u0447\u0442\u0435\u043d\u0438\u0435 \u0438\u0437 \u043f\u0435\u0440\u0435\u043c\u0435\u043d\u043d\u043e\u0439 file<\/em> \u0438 \u0437\u0430\u043f\u0438\u0441\u044c \u0432 \u043f\u0435\u0440\u0435\u043c\u0435\u043d\u043d\u0443\u044e file<\/em>. \u041f\u043e\u043f\u0440\u0430\u0432\u0438\u0442\u044c \u043a\u043e\u0434 \u043c\u043e\u0436\u043d\u043e \u0442\u0430\u043a\u0438\u043c \u043e\u0431\u0440\u0430\u0437\u043e\u043c:<\/p>\n
fwrite(buf, s, 1, to); <\/code><\/pre>\n
\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435 N2<\/h4>\n
V1065<\/a> [CWE-682] Expression can be simplified, check ‘bi’ and similar operands. PreviewAnimation.cpp 146<\/p>\n
PreviewAnimation::PreviewAnimation(....) {   \/\/ ....   const int bw = 30;   const int bh = 30;   const int bi = 3;   const int cn = 9;    bar.r.w = 7 + (bw + bi)*cn + 7 - bi + 1 + (bw + bi) + 7 - bi + 1;   \/\/ .... } <\/code><\/pre>\n
\u0410 \u044d\u0442\u043e \u043e\u0434\u043d\u043e \u0438\u0437 \u043c\u043e\u0438\u0445 \u0441\u0430\u043c\u044b\u0445 \u043b\u044e\u0431\u0438\u043c\u044b\u0445 \u0434\u0438\u0430\u0433\u043d\u043e\u0441\u0442\u0438\u0447\u0435\u0441\u043a\u0438\u0445 \u043f\u0440\u0430\u0432\u0438\u043b! \u041e\u043d\u043e \u0433\u043e\u0432\u043e\u0440\u0438\u0442 \u043d\u0430\u043c \u043e \u0442\u043e\u043c, \u0447\u0442\u043e \u0432 \u043f\u0440\u0435\u0434\u0441\u0442\u0430\u0432\u043b\u0435\u043d\u043d\u043e\u043c \u0432\u044b\u0440\u0430\u0436\u0435\u043d\u0438\u0438 \u043c\u043e\u0436\u043d\u043e \u0441\u043e\u043a\u0440\u0430\u0442\u0438\u0442\u044c \u043f\u0435\u0440\u0435\u043c\u0435\u043d\u043d\u0443\u044e bi<\/em>:<\/p>\n
bar.r.w = 7 + (bw + bi)*cn + 7 - bi + 1 + bw + 7 + 1; <\/code><\/pre>\n
\u0412\u043e\u0437\u043c\u043e\u0436\u043d\u043e, \u043d\u0430 \u043c\u0435\u0441\u0442\u0435 \u043e\u0434\u043d\u043e\u0433\u043e \u0438\u0437 bi<\/em> \u0434\u043e\u043b\u0436\u043d\u0430 \u0431\u044b\u0442\u044c \u043a\u0430\u043a\u0430\u044f-\u0442\u043e \u0434\u0440\u0443\u0433\u0430\u044f \u043f\u0435\u0440\u0435\u043c\u0435\u043d\u043d\u0430\u044f. \u0418\u043b\u0438 \u0436\u0435 \u0432\u044b\u0440\u0430\u0436\u0435\u043d\u0438\u0435 (bw + bi)<\/em> \u0437\u0430\u0431\u044b\u043b\u0438 \u043d\u0430 \u0447\u0442\u043e-\u0442\u043e \u0443\u043c\u043d\u043e\u0436\u0438\u0442\u044c. \u0418\u043d\u0442\u0435\u0440\u0435\u0441\u043d\u043e, \u0430 \u043a\u0430\u043a \u0431\u044b \u043f\u043e\u043c\u0435\u043d\u044f\u043b\u0438\u0441\u044c \u0430\u043d\u0438\u043c\u0430\u0446\u0438\u0438 \u0432 \u0438\u0433\u0440\u0435 \u043f\u043e\u0441\u043b\u0435 \u044d\u0442\u043e\u0433\u043e?<\/p>\n
\u0422\u0430\u043a\u0436\u0435, \u043e\u0442\u043c\u0435\u0447\u0443, \u0447\u0442\u043e \u0434\u0430\u0432\u0430\u0442\u044c \u043f\u043e\u0434\u043e\u0431\u043d\u044b\u0435 \u0438\u043c\u0435\u043d\u0430 \u043a\u043e\u043d\u0441\u0442\u0430\u043d\u0442\u0430\u043c: bw, bh, bi, cn<\/em> \u2013 \u043f\u043b\u043e\u0445\u0430\u044f \u043f\u0440\u0430\u043a\u0442\u0438\u043a\u0430. \u042d\u0442\u043e \u0437\u0430\u043f\u0443\u0442\u044b\u0432\u0430\u0435\u0442 \u043a\u043e\u0434 \u0434\u043b\u044f \u0447\u0438\u0442\u0430\u044e\u0449\u0435\u0433\u043e \u0435\u0433\u043e \u0447\u0435\u043b\u043e\u0432\u0435\u043a\u0430. \u041a\u0443\u0434\u0430 \u0443\u0436 \u043b\u0443\u0447\u0448\u0435 \u0431\u044b\u043b\u043e \u0431\u044b \u043d\u0430\u0437\u0432\u0430\u0442\u044c \u043f\u0435\u0440\u0435\u043c\u0435\u043d\u043d\u044b\u0435 \u043e\u0441\u043c\u044b\u0441\u043b\u0435\u043d\u043d\u0435\u0435 \u0438\u043b\u0438 \u0445\u043e\u0442\u044f \u0431\u044b \u0441\u043e\u043f\u0440\u043e\u0432\u043e\u0434\u0438\u0442\u044c \u0438\u0445 \u043e\u043f\u0440\u0435\u0434\u0435\u043b\u0435\u043d\u0438\u044f \u043f\u043e\u044f\u0441\u043d\u044f\u044e\u0449\u0438\u043c\u0438 \u043a\u043e\u043c\u043c\u0435\u043d\u0442\u0430\u0440\u0438\u044f\u043c\u0438.<\/p>\n
\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435 N3<\/h4>\n
V570<\/a> [CWE-480] The ‘ldMaxChars’ variable is assigned to itself. StringAttr.cpp 198<\/p>\n
void StringAttribute::LoadFromFile (....) {   int ldMinChars, ldMaxChars;   \/\/ ....   minChars = ldMinChars;   ldMaxChars = ldMaxChars;   \/\/ .... }  class StringAttribute : public BaseAttribute {   int minChars;   int maxChars; } <\/code><\/pre>\n
\u0422\u0443\u0442 \u043c\u044b \u0432\u0438\u0434\u0438\u043c \u043a\u043b\u0430\u0441\u0441\u0438\u0447\u0435\u0441\u043a\u043e\u0435 \u043f\u0440\u0438\u0441\u0432\u0430\u0438\u0432\u0430\u043d\u0438\u0435 \u043f\u0435\u0440\u0435\u043c\u0435\u043d\u043d\u043e\u0439 \u0441\u0430\u043c\u043e\u0439 \u0441\u0435\u0431\u0435. \u0427\u0442\u043e\u0431\u044b \u043f\u043e\u0441\u043c\u043e\u0442\u0440\u0435\u0442\u044c, \u0447\u0442\u043e \u0434\u043e\u043b\u0436\u043d\u043e \u0431\u044b\u0442\u044c \u043d\u0430 \u043c\u0435\u0441\u0442\u0435 \u043f\u0435\u0440\u0435\u043c\u0435\u043d\u043d\u043e\u0439, \u0432 \u043a\u043e\u0442\u043e\u0440\u0443\u044e \u043f\u0440\u043e\u0438\u0441\u0445\u043e\u0434\u0438\u0442 \u043f\u0440\u0438\u0441\u0432\u0430\u0438\u0432\u0430\u043d\u0438\u0435, \u0437\u0430\u0433\u043b\u044f\u043d\u0435\u043c \u0432 \u043e\u043f\u0440\u0435\u0434\u0435\u043b\u0435\u043d\u0438\u0435 \u043a\u043b\u0430\u0441\u0441\u0430 StringAttribute<\/em>. \u0412 \u043d\u0451\u043c \u0432\u0438\u0434\u043d\u043e \u043f\u043e\u0445\u043e\u0436\u0435\u0435 \u043f\u043e\u043b\u0435 maxChars<\/em>. \u041e\u0447\u0435\u0432\u0438\u0434\u043d\u043e, \u0447\u0442\u043e \u043f\u0440\u043e\u0433\u0440\u0430\u043c\u043c\u0438\u0441\u0442 \u043e\u043f\u0435\u0447\u0430\u0442\u0430\u043b\u0441\u044f \u0438 \u043d\u0430 \u0441\u0430\u043c\u043e\u043c \u0434\u0435\u043b\u0435 \u0434\u043e\u043b\u0436\u0435\u043d \u0431\u044b\u043b \u043f\u0440\u0438\u0441\u0432\u043e\u0438\u0442\u044c \u043f\u0435\u0440\u0435\u043c\u0435\u043d\u043d\u043e\u0439 \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435 ldMaxChars<\/em>. \u041f\u043e\u043f\u0440\u0430\u0432\u043b\u0435\u043d\u043d\u044b\u0439 \u0444\u0440\u0430\u0433\u043c\u0435\u043d\u0442 \u043a\u043e\u0434\u0430:<\/p>\n
void StringAttribute::LoadFromFile (....) {   int ldMinChars, ldMaxChars;   \/\/ ....   minChars = ldMinChars;   maxChars = ldMaxChars;   \/\/ .... } <\/code><\/pre>\n
\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435 N4<\/h4>\n
V781<\/a> [CWE-20, CERT-API00-C] The value of the ‘start’ index is checked after it was used. Perhaps there is a mistake in program logic. coreloader.h 136<\/p>\n
__forceinline void GetBootParams(const char * bootIni) {   long start = 0;   \/\/ ....   if (bootIni[start] && start > 0)   {     \/\/ ....   }   \/\/ .... } <\/code><\/pre>\n
\u0412 \u0434\u0430\u043d\u043d\u043e\u043c \u0444\u0440\u0430\u0433\u043c\u0435\u043d\u0442\u0435 \u043a\u043e\u0434\u0430 \u043f\u0440\u043e\u0433\u0440\u0430\u043c\u043c\u0438\u0441\u0442 \u0441\u043d\u0430\u0447\u0430\u043b\u0430 \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u043b \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435 start<\/em> \u0434\u043b\u044f \u0438\u043d\u0434\u0435\u043a\u0441\u0430\u0446\u0438\u0438 \u043f\u043e \u043c\u0430\u0441\u0441\u0438\u0432\u0443 bootIni<\/em>, \u0430 \u0442\u043e\u043b\u044c\u043a\u043e \u0437\u0430\u0442\u0435\u043c \u043f\u0440\u043e\u0432\u0435\u0440\u0438\u043b \u0435\u0433\u043e \u043d\u0430 \u0431\u043e\u043b\u044c\u0448\u0435* 0*. \u041f\u0440\u043e\u0432\u0435\u0440\u043a\u0430 \u043c\u043e\u0436\u0435\u0442 \u0431\u044b\u0442\u044c \u0431\u0435\u0441\u0441\u043c\u044b\u0441\u043b\u0435\u043d\u043d\u0430, \u0442\u043e\u0433\u0434\u0430 \u0441\u0442\u043e\u0438\u0442 \u0443\u0431\u0440\u0430\u0442\u044c \u0435\u0451. \u0422\u0430\u043a\u0436\u0435 \u043e\u043d\u0430 \u043c\u043e\u0436\u0435\u0442 \u0438\u043c\u0435\u0442\u044c \u0441\u043c\u044b\u0441\u043b, \u0442\u043e\u0433\u0434\u0430 \u0435\u0451 \u0441\u0442\u043e\u0438\u0442 \u043f\u043e\u043c\u0435\u043d\u044f\u0442\u044c \u043c\u0435\u0441\u0442\u0430\u043c\u0438 \u0441 \u043e\u043f\u0435\u0440\u0430\u0446\u0438\u0435\u0439 \u0432\u0437\u044f\u0442\u0438\u044f \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u044f \u043f\u043e \u0438\u043d\u0434\u0435\u043a\u0441\u0443.<\/p>\n
if (start > 0 && bootIni[start]) {   \/\/ .... } <\/code><\/pre>\n
\u0412 \u043f\u0440\u043e\u0442\u0438\u0432\u043d\u043e\u043c \u0441\u043b\u0443\u0447\u0430\u0435, \u043f\u0440\u043e\u0433\u0440\u0430\u043c\u043c\u0438\u0441\u0442 \u0440\u0438\u0441\u043a\u0443\u0435\u0442 \u0443\u0440\u043e\u043d\u0438\u0442\u044c \u0438\u0433\u0440\u0443. \u041a\u0441\u0442\u0430\u0442\u0438, \u0442\u0430\u043a\u0438\u0445 \u0441\u0440\u0430\u0431\u0430\u0442\u044b\u0432\u0430\u043d\u0438\u0439 \u0432 \u043f\u0440\u043e\u0433\u0440\u0430\u043c\u043c\u0435 \u0431\u044b\u043b\u043e \u0434\u043e\u0432\u043e\u043b\u044c\u043d\u043e \u043c\u043d\u043e\u0433\u043e. \u0412\u043e\u0442 \u043d\u0435\u0441\u043a\u043e\u043b\u044c\u043a\u043e, \u043d\u0430 \u043a\u043e\u0442\u043e\u0440\u044b\u0435 \u0441\u0442\u043e\u0438\u0442 \u043e\u0431\u0440\u0430\u0442\u0438\u0442\u044c \u0432\u043d\u0438\u043c\u0430\u043d\u0438\u0435 \u0432 \u043f\u0435\u0440\u0432\u0443\u044e \u043e\u0447\u0435\u0440\u0435\u0434\u044c:<\/p>\n