{"id":416533,"date":"2024-06-30T01:00:48","date_gmt":"2024-06-30T01:00:48","guid":{"rendered":"http:\/\/savepearlharbor.com\/?p=416533"},"modified":"-0001-11-30T00:00:00","modified_gmt":"2024-06-30T01:00:48","slug":"","status":"publish","type":"post","link":"https:\/\/savepearlharbor.com\/?p=416533","title":{"rendered":"\u0422\u043e\u043f-10 \u043e\u0448\u0438\u0431\u043e\u043a \u0432 C++ \u043f\u0440\u043e\u0435\u043a\u0442\u0430\u0445 \u0437\u0430 2022 \u0433\u043e\u0434<\/span>"},"content":{"rendered":"
<\/div>\n
\n
\n
\n
\n

\u0414\u0435\u043b\u043e \u0438\u0434\u0451\u0442 \u043a \u041d\u043e\u0432\u043e\u043c\u0443 \u0433\u043e\u0434\u0443, \u0430 \u0437\u043d\u0430\u0447\u0438\u0442, \u0441\u0430\u043c\u043e\u0435 \u0432\u0440\u0435\u043c\u044f \u0442\u0440\u0430\u0434\u0438\u0446\u0438\u043e\u043d\u043d\u043e \u0432\u0441\u043f\u043e\u043c\u043d\u0438\u0442\u044c \u0434\u0435\u0441\u044f\u0442\u044c \u0441\u0430\u043c\u044b\u0445 \u0438\u043d\u0442\u0435\u0440\u0435\u0441\u043d\u044b\u0445 \u0441\u0440\u0430\u0431\u0430\u0442\u044b\u0432\u0430\u043d\u0438\u0439, \u043a\u043e\u0442\u043e\u0440\u044b\u0435 \u043d\u0430\u0448\u0451\u043b PVS-Studio \u0432 2022 \u0433\u043e\u0434\u0443.<\/p>\n

<\/p>\n

\"1021_Top_10_CPP_Bugs_2022_ru\/image1.png\"<\/p>\n

<\/a> <\/p>\n

\u0421\u0442\u043e\u0438\u0442 \u043e\u0442\u043c\u0435\u0442\u0438\u0442\u044c, \u0447\u0442\u043e \u0432 \u044d\u0442\u043e\u043c \u0433\u043e\u0434\u0443 \u0431\u044b\u043b\u043e \u043d\u0435 \u0442\u0430\u043a \u043c\u043d\u043e\u0433\u043e \u0441\u0442\u0430\u0442\u0435\u0439 \u043f\u0440\u043e \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0443 \u043f\u0440\u043e\u0435\u043a\u0442\u043e\u0432, \u043a\u0430\u043a \u0432 \u043f\u0440\u043e\u0448\u043b\u044b\u0445. \u0421\u0442\u0430\u0442\u044c\u0438 \u0432 \u043d\u0430\u0448\u0435\u043c \u0431\u043b\u043e\u0433\u0435<\/a> \u0441\u0442\u0430\u043b\u0438 \u0440\u0430\u0437\u043d\u043e\u043e\u0431\u0440\u0430\u0437\u043d\u0435\u0439, \u0430 \u0442\u0430\u043a\u0436\u0435 \u043f\u043e\u044f\u0432\u0438\u043b\u0438\u0441\u044c \u043f\u0435\u0440\u0435\u0432\u043e\u0434\u044b \u0441\u0442\u0430\u0442\u0435\u0439<\/a> \u0441\u0442\u043e\u0440\u043e\u043d\u043d\u0438\u0445 \u0430\u0432\u0442\u043e\u0440\u043e\u0432. \u041e\u0434\u043d\u0430\u043a\u043e, \u0443 \u043c\u0435\u043d\u044f, \u043a\u0430\u043a \u0443 \u043e\u0434\u043d\u043e\u0433\u043e \u0438\u0437 \u0447\u0438\u0442\u0430\u0442\u0435\u043b\u0435\u0439 \u043d\u0430\u0448\u0435\u0433\u043e \u0431\u043b\u043e\u0433\u0430 (\u0430 \u0442\u0430\u043a\u0436\u0435 \u0430\u0432\u0442\u043e\u0440\u0430 \u043d\u0435\u043a\u043e\u0442\u043e\u0440\u044b\u0445 \u0441\u0442\u0430\u0442\u0435\u0439<\/a> \u0438\u0437 \u043d\u0435\u0433\u043e), \u0432\u0441\u0451 \u0440\u0430\u0432\u043d\u043e \u0441\u0444\u043e\u0440\u043c\u0438\u0440\u043e\u0432\u0430\u043b\u0441\u044f \u0442\u043e\u043f \u043e\u0448\u0438\u0431\u043e\u043a, \u043d\u0430\u0439\u0434\u0435\u043d\u043d\u044b\u0445 \u0430\u043d\u0430\u043b\u0438\u0437\u0430\u0442\u043e\u0440\u043e\u043c PVS-Studio \u0438 \u043e\u043f\u0438\u0441\u0430\u043d\u043d\u044b\u0445 \u0432 \u043d\u0430\u0448\u0438\u0445 \u0441\u0442\u0430\u0442\u044c\u044f\u0445 \u043f\u0440\u043e \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0443 \u043f\u0440\u043e\u0435\u043a\u0442\u043e\u0432. \u041e\u0442\u043c\u0435\u0447\u0443, \u0447\u0442\u043e \u044d\u0442\u043e\u0442 \u0442\u043e\u043f \u0434\u043e\u0432\u043e\u043b\u044c\u043d\u043e \u0441\u0443\u0431\u044a\u0435\u043a\u0442\u0438\u0432\u043d\u044b\u0439 \u2014 \u0432 \u043d\u0451\u043c \u043c\u043d\u043e\u0433\u043e \u043e\u0448\u0438\u0431\u043e\u043a \u0438\u0437 \u0441\u0442\u0430\u0442\u0435\u0439, \u043a\u043e\u0442\u043e\u0440\u044b\u0435 \u043f\u0438\u0441\u0430\u043b \u044f :). <\/p>\n

<\/p>\n

\u0415\u0441\u043b\u0438 \u0443 \u0432\u0430\u0441, \u0443\u0432\u0430\u0436\u0430\u0435\u043c\u044b\u0435 \u0447\u0438\u0442\u0430\u0442\u0435\u043b\u0438, \u0435\u0441\u0442\u044c \u0441\u0432\u043e\u0451 \u0432\u0438\u0434\u0435\u043d\u0438\u0435 \u0442\u043e\u0433\u043e, \u043a\u0430\u043a \u0434\u043e\u043b\u0436\u0435\u043d \u0432\u044b\u0433\u043b\u044f\u0434\u0435\u0442\u044c \u044d\u0442\u043e\u0442 \u0442\u043e\u043f, \u0442\u043e \u043f\u0438\u0448\u0438\u0442\u0435 \u043e\u0431 \u044d\u0442\u043e\u043c \u0432 \u043a\u043e\u043c\u043c\u0435\u043d\u0442\u0430\u0440\u0438\u044f\u0445. \u0418\u0442\u0430\u043a, \u043f\u0440\u0438\u0441\u0442\u0443\u043f\u0438\u043c, \u0432\u0441\u0435\u043c \u043f\u0440\u0438\u044f\u0442\u043d\u043e\u0433\u043e \u0447\u0442\u0435\u043d\u0438\u044f!<\/p>\n

<\/p>\n

\u0414\u0435\u0441\u044f\u0442\u043e\u0435 \u043c\u0435\u0441\u0442\u043e: \u041a\u043b\u0430\u0441\u0441\u0438\u0447\u0435\u0441\u043a\u0430\u044f \u043e\u043f\u0435\u0447\u0430\u0442\u043a\u0430<\/h2>\n

<\/p>\n

V501<\/a> [CWE-570] There are identical sub-expressions ‘(SrcTy.isPointer() && DstTy.isScalar())’ to the left and to the right of the ‘||’ operator. CallLowering.cpp 1198<\/p>\n

<\/p>\n

static bool isCopyCompatibleType(LLT SrcTy, LLT DstTy)  {   if (SrcTy == DstTy)     return true;    if (SrcTy.getSizeInBits() != DstTy.getSizeInBits())     return false;    SrcTy = SrcTy.getScalarType();   DstTy = DstTy.getScalarType();    return (SrcTy.isPointer() && DstTy.isScalar()) ||          (DstTy.isScalar() && SrcTy.isPointer()); }<\/code><\/pre>\n
  <\/p>\n
\u0412 \u0434\u0430\u043d\u043d\u043e\u043c \u0444\u0440\u0430\u0433\u043c\u0435\u043d\u0442\u0435 \u043a\u043e\u0434\u0430 \u0434\u043e\u043f\u0443\u0449\u0435\u043d\u0430 \u043a\u043b\u0430\u0441\u0441\u0438\u0447\u0435\u0441\u043a\u0430\u044f \u043e\u043f\u0435\u0447\u0430\u0442\u043a\u0430:<\/p>\n
  <\/p>\n
(SrcTy.isPointer() && DstTy.isScalar()) || (DstTy.isScalar()  && SrcTy.isPointer())<\/code><\/pre>\n
  <\/p>\n
\u041e\u0448\u0438\u0431\u043a\u0430 \u0437\u0434\u0435\u0441\u044c:<\/p>\n
  <\/p>\n
(SrcTy.isPointer() && DstTy.isScalar()) || (DstTy.isScalar()  && SrcTy.isPointer())<\/code><\/pre>\n
  <\/p>\n
\u041f\u0440\u043e\u0433\u0440\u0430\u043c\u043c\u0438\u0441\u0442 \u043e\u0434\u043d\u043e\u0432\u0440\u0435\u043c\u0435\u043d\u043d\u043e \u043f\u043e\u043c\u0435\u043d\u044f\u043b \u0432\u043e \u0432\u0442\u043e\u0440\u043e\u0439 \u0441\u0442\u0440\u043e\u0447\u043a\u0435 Src<\/em> \u043d\u0430 Dst<\/em> \u0438 Pointer<\/em> \u043d\u0430 Scalar<\/em>. \u0412 \u0440\u0435\u0437\u0443\u043b\u044c\u0442\u0430\u0442\u0435 \u043f\u043e\u043b\u0443\u0447\u0430\u0435\u0442\u0441\u044f, \u0447\u0442\u043e \u0434\u0432\u0430 \u0440\u0430\u0437\u0430 \u043f\u0440\u043e\u0432\u0435\u0440\u044f\u0435\u0442\u0441\u044f \u043e\u0434\u043d\u043e \u0438 \u0442\u043e \u0436\u0435! \u041a\u043e\u0434 \u0432\u044b\u0448\u0435 \u044d\u043a\u0432\u0438\u0432\u0430\u043b\u0435\u043d\u0442\u0435\u043d:<\/p>\n
  <\/p>\n
(SrcTy.isPointer() && DstTy.isScalar()) || (SrcTy.isPointer() && DstTy.isScalar())<\/code><\/pre>\n
  <\/p>\n
\u041f\u0440\u0430\u0432\u0438\u043b\u044c\u043d\u044b\u0439 \u0432\u0430\u0440\u0438\u0430\u043d\u0442:<\/p>\n
  <\/p>\n
(SrcTy.isPointer() && DstTy.isScalar()) || (SrcTy.isScalar()  && DstTy.isPointer())<\/code><\/pre>\n
  <\/p>\n
\u042d\u0442\u0430 \u043e\u0448\u0438\u0431\u043a\u0430 \u0432\u043e\u0448\u043b\u0430 \u0432 \u0442\u043e\u043f \u0438\u0437 \u0441\u0442\u0430\u0442\u044c\u0438: «\u041f\u0440\u0438\u043c\u0435\u0440\u044b \u043e\u0448\u0438\u0431\u043e\u043a, \u043a\u043e\u0442\u043e\u0440\u044b\u0435 \u043c\u043e\u0436\u0435\u0442 \u043e\u0431\u043d\u0430\u0440\u0443\u0436\u0438\u0442\u044c PVS-Studio \u0432 \u043a\u043e\u0434\u0435 LLVM 15.0<\/a>«.<\/p>\n
  <\/p>\n
\u0414\u0435\u0432\u044f\u0442\u043e\u0435 \u043c\u0435\u0441\u0442\u043e: \u041a\u0443\u0434\u0430 \u0443\u0436 \u0431\u0435\u0437 \u0432\u044b\u0445\u043e\u0434\u0430 \u0437\u0430 \u0433\u0440\u0430\u043d\u0438\u0446\u0443 \u043a\u043e\u043d\u0442\u0435\u0439\u043d\u0435\u0440\u0430<\/h2>\n
  <\/p>\n
V557<\/a> Array overrun is possible. The ‘j’ index is pointing beyond array bound. OgreAnimationTrack.cpp 219<\/p>\n
  <\/p>\n
void AnimationTrack::_buildKeyFrameIndexMap(   const std::vector<Real>& keyFrameTimes) {    \/\/ ....    size_t i = 0, j = 0;   while (j <= keyFrameTimes.size())                    \/\/ <=   {     mKeyFrameIndexMap[j] = static_cast<ushort>(i);     while (i < mKeyFrames.size()       && mKeyFrames[i]->getTime() <= keyFrameTimes[j]) \/\/ <=       ++i;     ++j;   } }<\/code><\/pre>\n
  <\/p>\n
\u0417\u0434\u0435\u0441\u044c \u0438\u043d\u0434\u0435\u043a\u0441 j<\/em>, \u043f\u043e \u043a\u043e\u0442\u043e\u0440\u043e\u043c\u0443 \u043c\u044b \u043f\u043e\u043b\u0443\u0447\u0430\u0435\u043c \u0434\u043e\u0441\u0442\u0443\u043f \u043a \u044d\u043b\u0435\u043c\u0435\u043d\u0442\u0430\u043c \u043a\u043e\u043d\u0442\u0435\u0439\u043d\u0435\u0440\u0430 keyFrameTimes<\/em>, \u0438\u043d\u043a\u0440\u0435\u043c\u0435\u043d\u0442\u0438\u0440\u0443\u0435\u0442\u0441\u044f \u0434\u043e \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u044f, \u0440\u0430\u0432\u043d\u043e\u0433\u043e \u0440\u0430\u0437\u043c\u0435\u0440\u0443 \u043a\u043e\u043d\u0442\u0435\u0439\u043d\u0435\u0440\u0430. \u041f\u043e\u043f\u0440\u0430\u0432\u0438\u0442\u044c \u043e\u0448\u0438\u0431\u043a\u0443 \u043c\u043e\u0436\u043d\u043e \u0431\u044b\u043b\u043e \u0431\u044b \u0442\u0430\u043a\u0438\u043c \u043e\u0431\u0440\u0430\u0437\u043e\u043c:<\/p>\n
  <\/p>\n
while (j < keyFrameTimes.size()) {   \/\/ .... }<\/code><\/pre>\n
  <\/p>\n
\u042d\u0442\u0430 \u043e\u0448\u0438\u0431\u043a\u0430 \u0432\u043e\u0448\u043b\u0430 \u0432 \u0442\u043e\u043f \u0438\u0437 \u0441\u0442\u0430\u0442\u044c\u0438: «\u041f\u0440\u043e\u0432\u0435\u0440\u043a\u0430 \u0444\u0440\u0435\u0439\u043c\u0432\u043e\u0440\u043a\u0430 Ogre3D \u0441\u0442\u0430\u0442\u0438\u0447\u0435\u0441\u043a\u0438\u043c \u0430\u043d\u0430\u043b\u0438\u0437\u0430\u0442\u043e\u0440\u043e\u043c PVS-Studio»<\/a><\/p>\n
  <\/p>\n
\u0412\u043e\u0441\u044c\u043c\u043e\u0435 \u043c\u0435\u0441\u0442\u043e: \u042f\u0432\u043d\u044b\u0439 \u043f\u0440\u0438\u043c\u0435\u0440 \u0440\u0430\u0437\u044b\u043c\u0435\u043d\u043e\u0432\u0430\u043d\u0438\u044f \u043d\u0443\u043b\u0435\u0432\u043e\u0433\u043e \u0443\u043a\u0430\u0437\u0430\u0442\u0435\u043b\u044f<\/h2>\n
  <\/p>\n
V522<\/a> [CERT-EXP34-C] Dereferencing of the null pointer ‘document’ might take place. TextBlockCursor.cpp 332<\/p>\n
  <\/p>\n
auto BlockCursor::begin() -> std::list<TextBlock>::iterator {   return document == nullptr              ? document->blocks.end() : document->blocks.begin(); }<\/code><\/pre>\n
  <\/p>\n
\u041f\u043e\u043c\u043d\u044e, \u0447\u0442\u043e \u043a\u043e\u0433\u0434\u0430 \u044f \u0432 \u043f\u0435\u0440\u0432\u044b\u0439 \u0440\u0430\u0437 \u0443\u0432\u0438\u0434\u0435\u043b \u0434\u0430\u043d\u043d\u044b\u0439 \u0444\u0440\u0430\u0433\u043c\u0435\u043d\u0442 \u043a\u043e\u0434\u0430, \u0442\u043e \u0434\u0430\u0436\u0435 \u043d\u0435 \u0443\u0434\u0435\u0440\u0436\u0430\u043b\u0441\u044f \u043e\u0442 \u0444\u0435\u0439\u0441\u043f\u0430\u043b\u043c\u0430. \u0414\u0430\u0432\u0430\u0439\u0442\u0435 \u0440\u0430\u0437\u0431\u0435\u0440\u0451\u043c\u0441\u044f, \u0447\u0442\u043e \u0442\u0443\u0442 \u043f\u0440\u043e\u0438\u0441\u0445\u043e\u0434\u0438\u0442: \u043c\u044b \u0432\u0438\u0434\u0438\u043c \u044f\u0432\u043d\u0443\u044e \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0443 \u0443\u043a\u0430\u0437\u0430\u0442\u0435\u043b\u044f document<\/em> \u043d\u0430 nullptr<\/em> \u0438 \u0435\u0433\u043e \u0440\u0430\u0437\u044b\u043c\u0435\u043d\u043e\u0432\u0430\u043d\u0438\u0435 \u0432 \u043e\u0431\u043e\u0438\u0445 \u0432\u0435\u0442\u043a\u0430\u0445 \u0442\u0435\u0440\u043d\u0430\u0440\u043d\u043e\u0433\u043e \u043e\u043f\u0435\u0440\u0430\u0442\u043e\u0440\u0430.<\/p>\n
  <\/p>\n
\u0414\u0430\u043d\u043d\u044b\u0439 \u043a\u043e\u0434 \u0431\u044b\u043b \u0431\u044b \u043a\u043e\u0440\u0440\u0435\u043a\u0442\u0435\u043d \u0442\u043e\u043b\u044c\u043a\u043e \u0432 \u0442\u043e\u043c \u0441\u043b\u0443\u0447\u0430\u0435, \u0435\u0441\u043b\u0438 \u0446\u0435\u043b\u044c\u044e \u043f\u0440\u043e\u0433\u0440\u0430\u043c\u043c\u0438\u0441\u0442\u0430 \u0431\u044b\u043b\u043e \u0431\u044b \u0443\u0440\u043e\u043d\u0438\u0442\u044c \u043f\u0440\u043e\u0433\u0440\u0430\u043c\u043c\u0443.<\/p>\n
  <\/p>\n
\u042d\u0442\u0430 \u043e\u0448\u0438\u0431\u043a\u0430 \u0432\u043e\u0448\u043b\u0430 \u0432 \u0442\u043e\u043f \u0438\u0437 \u0441\u0442\u0430\u0442\u044c\u0438: «MuditaOS: \u0437\u0430\u0437\u0432\u043e\u043d\u0438\u0442 \u043b\u0438 \u0432\u0430\u0448 \u0431\u0443\u0434\u0438\u043b\u044c\u043d\u0438\u043a? \u0427\u0430\u0441\u0442\u044c 1<\/a>«.<\/p>\n
  <\/p>\n
\"getTrialImageLink\"<\/a><\/p>\n
  <\/p>\n
\u0421\u0435\u0434\u044c\u043c\u043e\u0435 \u043c\u0435\u0441\u0442\u043e: \u0423\u0447\u0438\u043c\u0441\u044f \u0441\u0447\u0438\u0442\u0430\u0442\u044c \u0434\u043e \u0441\u0435\u043c\u0438<\/h2>\n
  <\/p>\n
V557<\/a> [CWE-787] Array overrun is possible. The ‘dynamicStateCount ++’ index is pointing beyond array bound. VltGraphics.cpp 157<\/p>\n
  <\/p>\n
VkPipeline VltGraphicsPipeline::createPipeline(\/* .... *\/) const {   \/\/ ....   std::array<VkDynamicState, 6> dynamicStates;   uint32_t                      dynamicStateCount = 0;   dynamicStates[dynamicStateCount++] = VK_DYNAMIC_STATE_VIEWPORT;   dynamicStates[dynamicStateCount++] = VK_DYNAMIC_STATE_SCISSOR;   if (state.useDynamicDepthBias())     dynamicStates[dynamicStateCount++] = VK_DYNAMIC_STATE_DEPTH_BIAS;   if (state.useDynamicDepthBounds())   {     dynamicStates[dynamicStateCount++] = VK_DYNAMIC_STATE_DEPTH_BOUNDS;     dynamicStates[dynamicStateCount++] =                              VK_DYNAMIC_STATE_DEPTH_BOUNDS_TEST_ENABLE;   }   if (state.useDynamicBlendConstants())     dynamicStates[dynamicStateCount++] = VK_DYNAMIC_STATE_BLEND_CONSTANTS;   if (state.useDynamicStencilRef())     dynamicStates[dynamicStateCount++] = VK_DYNAMIC_STATE_STENCIL_REFERENCE;   \/\/ .... }<\/code><\/pre>\n
  <\/p>\n
\u0410\u043d\u0430\u043b\u0438\u0437\u0430\u0442\u043e\u0440 \u043f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0430\u0435\u0442, \u0447\u0442\u043e \u043c\u043e\u0436\u0435\u0442 \u043f\u0440\u043e\u0438\u0437\u043e\u0439\u0442\u0438 \u043f\u0435\u0440\u0435\u043f\u043e\u043b\u043d\u0435\u043d\u0438\u0435 \u043c\u0430\u0441\u0441\u0438\u0432\u0430 dynamicStates<\/em>. \u0412 \u0434\u0430\u043d\u043d\u043e\u043c \u0444\u0440\u0430\u0433\u043c\u0435\u043d\u0442\u0435 \u043a\u043e\u0434\u0430 \u0435\u0441\u0442\u044c 4 \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0438:<\/p>\n
  <\/p>\n