{"id":257911,"date":"2015-05-25T18:39:02","date_gmt":"2015-05-25T14:39:02","guid":{"rendered":"http:\/\/savepearlharbor.com\/?p=257911"},"modified":"-0001-11-30T00:00:00","modified_gmt":"-0001-11-29T21:00:00","slug":"","status":"publish","type":"post","link":"https:\/\/savepearlharbor.com\/?p=257911","title":{"rendered":"\u041f\u0440\u043e\u0432\u0435\u0440\u044f\u0435\u043c \u043e\u0442\u043a\u0440\u044b\u0442\u044b\u0439 \u0438\u0441\u0445\u043e\u0434\u043d\u044b\u0439 \u043a\u043e\u0434 UEFI \u0434\u043b\u044f Intel Galileo \u043f\u0440\u0438 \u043f\u043e\u043c\u043e\u0449\u0438 PVS-Studio"},"content":{"rendered":"<p>     \t<img decoding=\"async\" align=\"left\" src=\"https:\/\/habrastorage.org\/files\/46f\/80b\/7c0\/46f80b7c024a4994bf5dee9057407b7d.png\"\/>\u0420\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u043a\u0430 \u043f\u0440\u043e\u0448\u0438\u0432\u043e\u043a, \u0434\u0430\u0436\u0435 \u0435\u0441\u043b\u0438 \u043e\u043d\u0430 \u0432\u0435\u0434\u0435\u0442\u0441\u044f \u043d\u0435 \u043d\u0430 \u0430\u0441\u0441\u0435\u043c\u0431\u043b\u0435\u0440\u0435 \u0434\u043b\u044f \u044d\u043a\u0437\u043e\u0442\u0438\u0447\u0435\u0441\u043a\u0438\u0445 \u0430\u0440\u0445\u0438\u0442\u0435\u043a\u0442\u0443\u0440, \u0430 \u043d\u0430 C \u0434\u043b\u044f i386\/amd64 \u2014 \u0434\u0435\u043b\u043e \u0432\u0435\u0441\u044c\u043c\u0430 \u043d\u0435\u043f\u0440\u043e\u0441\u0442\u043e\u0435, \u0434\u0430 \u0438 \u0446\u0435\u043d\u0430 \u043e\u0448\u0438\u0431\u043a\u0438 \u043c\u043e\u0436\u0435\u0442 \u0431\u044b\u0442\u044c \u043a\u0440\u0430\u0439\u043d\u0435 \u0432\u044b\u0441\u043e\u043a\u043e\u0439, \u0432\u043f\u043b\u043e\u0442\u044c \u0434\u043e \u0432\u044b\u0445\u043e\u0434\u0430 \u0446\u0435\u043b\u0435\u0432\u043e\u0439 \u0430\u043f\u043f\u0430\u0440\u0430\u0442\u043d\u043e\u0439 \u043f\u043b\u0430\u0442\u0444\u043e\u0440\u043c\u044b \u0438\u0437 \u0441\u0442\u0440\u043e\u044f, \u043f\u043e\u044d\u0442\u043e\u043c\u0443 \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u043d\u0438\u0435 \u0440\u0430\u0437\u043b\u0438\u0447\u043d\u044b\u0445 \u0442\u0435\u0445\u043d\u0438\u043a \u043f\u0440\u0435\u0434\u043e\u0442\u0432\u0440\u0430\u0449\u0435\u043d\u0438\u044f \u043e\u0448\u0438\u0431\u043e\u043a \u043d\u0430 \u0441\u0430\u043c\u044b\u0445 \u0440\u0430\u043d\u043d\u0438\u0445 \u044d\u0442\u0430\u043f\u0430\u0445 \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u043a\u0438 \u2014 \u043d\u0435\u043e\u0431\u0445\u043e\u0434\u0438\u043c\u043e\u0441\u0442\u044c. <br \/>  \u041a \u0441\u043e\u0436\u0430\u043b\u0435\u043d\u0438\u044e, \u043e \u0444\u043e\u0440\u043c\u0430\u043b\u044c\u043d\u043e\u0439 \u0432\u0435\u0440\u0438\u0444\u0438\u043a\u0430\u0446\u0438\u0438 \u0438\u043b\u0438 \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u043d\u0438\u0438 MISRA C \u0432 \u0441\u043b\u0443\u0447\u0430\u0435 UEFI-\u043f\u0440\u043e\u0448\u0438\u0432\u043e\u043a \u043e\u0441\u0442\u0430\u0435\u0442\u0441\u044f \u0442\u043e\u043b\u044c\u043a\u043e \u043c\u0435\u0447\u0442\u0430\u0442\u044c (\u0441 \u0434\u0440\u0443\u0433\u043e\u0439 \u0441\u0442\u0440\u043e\u043d\u044b, \u043c\u0430\u043b\u043e \u043a\u0442\u043e \u0445\u043e\u0447\u0435\u0442 \u0442\u0440\u0430\u0442\u0438\u0442\u044c \u043d\u0430 \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u043a\u0443 \u043f\u0440\u043e\u0448\u0438\u0432\u043a\u0438 \u043f\u0430\u0440\u0443 \u043b\u0435\u0442 \u0438 50% \u0431\u044e\u0442\u0436\u0435\u0442\u0430 \u043f\u0440\u043e\u0435\u043a\u0442\u0430), \u043f\u043e\u044d\u0442\u043e\u043c\u0443 \u0441\u0435\u0433\u043e\u0434\u043d\u044f \u043f\u043e\u0433\u043e\u0432\u043e\u0440\u0438\u043c \u043e \u0441\u0442\u0430\u0442\u0438\u0447\u0435\u0441\u043a\u043e\u043c \u0430\u043d\u0430\u043b\u0438\u0437\u0435, \u0430 \u0442\u043e\u0447\u043d\u0435\u0435 \u2014 \u043e \u043f\u043e\u043f\u0443\u043b\u044f\u0440\u043d\u043e\u043c \u043d\u0430 \u0425\u0430\u0431\u0440\u0435 \u0441\u0442\u0430\u0442\u0438\u0447\u0435\u0441\u043a\u043e\u043c \u0430\u043d\u0430\u043b\u0438\u0437\u0430\u0442\u043e\u0440\u0435 PVS-Studio, \u043a\u043e\u0442\u043e\u0440\u044b\u043c \u043f\u043e\u043f\u044b\u0442\u0430\u0435\u043c\u0441\u044f \u043d\u0430\u0439\u0442\u0438 \u043e\u0448\u0438\u0431\u043a\u0438 \u0432 \u043e\u0442\u043a\u0440\u044b\u0442\u043e\u043c \u043a\u043e\u0434\u0435 UEFI \u0434\u043b\u044f Intel Galileo. <br \/>  \u0417\u0430 \u0440\u0435\u0437\u0443\u043b\u044c\u0442\u0430\u0442\u0430\u043c\u0438 \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0438 \u043f\u043e\u043a\u043e\u0440\u043d\u0435\u0439\u0448\u0435 \u043f\u0440\u043e\u0448\u0443 \u043f\u043e\u0434 \u043a\u0430\u0442.<br \/>  <a name=\"habracut\"><\/a>  <\/p>\n<h4><b>\u041d\u0430\u0441\u0442\u0440\u043e\u0439\u043a\u0430 \u043e\u043a\u0440\u0443\u0436\u0435\u043d\u0438\u044f<\/b><\/h4>\n<p>\u041a\u0430\u043a \u043c\u043d\u0435 \u043f\u043e\u0434\u0441\u043a\u0430\u0437\u044b\u0432\u0430\u0435\u0442 \u043a\u0430\u043f\u0438\u0442\u0430\u043d \u041e\u0447\u0435\u0432\u0438\u0434\u043d\u043e\u0441\u0442\u044c, \u0434\u043b\u044f \u0430\u043d\u0430\u043b\u0438\u0437\u0430 \u043a\u0430\u043a\u043e\u0433\u043e-\u043b\u0438\u0431\u043e \u043a\u043e\u0434\u0430 \u043d\u0430\u043c \u043f\u043e\u043d\u0430\u0434\u043e\u0431\u0438\u0442\u0441\u044f \u0430\u043d\u0430\u043b\u0438\u0437\u0430\u0442\u043e\u0440, \u0441\u0430\u043c \u043a\u043e\u0434 \u0438 \u0441\u0431\u043e\u0440\u043e\u0447\u043d\u0430\u044f \u0441\u0440\u0435\u0434\u0430 \u0434\u043b\u044f \u043d\u0435\u0433\u043e.<br \/>  \u0410\u043d\u0430\u043b\u0438\u0437\u0430\u0442\u043e\u0440 \u043b\u0435\u0436\u0438\u0442 <a href=\"http:\/\/www.viva64.com\/ru\/pvs-studio-download\/\">\u043d\u0430 \u0441\u0430\u0439\u0442\u0435 \u0435\u0433\u043e \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u0430<\/a>, \u043f\u043e\u0441\u043b\u0435 \u0443\u0441\u0442\u0430\u043d\u043e\u0432\u043a\u0438 \u043f\u0438\u0448\u0435\u043c \u0435\u043c\u0443 \u043f\u0438\u0441\u044c\u043c\u043e \u0441 \u043f\u0440\u043e\u0441\u044c\u0431\u043e\u0439 \u043f\u0440\u0435\u0434\u043e\u0441\u0442\u0430\u0432\u0438\u0442\u044c \u043d\u0430 \u043a\u043e\u0440\u043e\u0442\u043a\u043e\u0435 \u0432\u0440\u0435\u043c\u044f \u043a\u043b\u044e\u0447, \u043f\u043e\u0437\u0432\u043e\u043b\u044f\u044e\u0449\u0438\u0439 \u0432\u0438\u0434\u0435\u0442\u044c \u043d\u0435 \u0442\u043e\u043b\u044c\u043a\u043e \u043f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u044f \u043f\u0435\u0440\u0432\u043e\u0433\u043e \u0443\u0440\u043e\u0432\u043d\u044f (\u0430 \u0438\u043c\u0435\u043d\u043d\u043e \u0442\u0430\u043a \u0441\u0434\u0435\u043b\u0430\u043d\u043e \u0432 \u043e\u0437\u043d\u0430\u043a\u043e\u043c\u0438\u0442\u0435\u043b\u044c\u043d\u044b\u0445 \u0432\u0435\u0440\u0441\u0438\u044f\u0445), \u0430 \u0432\u0441\u0435 \u0432\u043e\u043e\u0431\u0449\u0435 \u2014 \u0432 \u043d\u0430\u0448\u0435\u043c \u0441\u043b\u0443\u0447\u0430\u0435 \u043b\u0443\u0447\u0448\u0435 \u043f\u0435\u0440\u0435\u0431\u0434\u0435\u0442\u044c, \u0447\u0435\u043c \u043d\u0435\u0434\u043e\u0431\u0434\u0435\u0442\u044c.<br \/>  \u041a\u043e\u0434 \u043f\u0440\u043e\u0448\u0438\u0432\u043a\u0438 \u044f\u0432\u043b\u044f\u0435\u0442\u0441\u044f \u0447\u0430\u0441\u0442\u044c\u044e <a href=\"https:\/\/downloadcenter.intel.com\/downloads\/eula\/23197\/Intel-Quark-BSP?httpDown=http%3A%2F%2Fdownloadmirror.intel.com%2F23197%2Feng%2FBoard_Support_Package_Sources_for_Intel_Quark_v1.1.0.7z\">Quark BSP<\/a> \u0438 \u043e\u0441\u043d\u043e\u0432\u0430\u043d \u043d\u0430 <abbr title=\"EFI Development Kit\">EDK<\/abbr>2010.SR1, \u043a\u0430\u043a \u0438 \u0432\u0441\u0435 \u0441\u043e\u0432\u0440\u0435\u043c\u0435\u043d\u043d\u044b\u0435 \u0440\u0435\u0430\u043b\u0438\u0437\u0430\u0446\u0438\u0438 UEFI, \u0437\u0430 \u0438\u0441\u043a\u043b\u044e\u0447\u0435\u043d\u0438\u0435\u043c \u043f\u0440\u043e\u0434\u0443\u043a\u0446\u0438\u0438 \u043a\u043e\u043c\u043f\u0430\u043d\u0438\u0438 Apple. <br \/>  EDK \u0438\u043c\u0435\u0435\u0442 \u0441\u043e\u0431\u0441\u0442\u0432\u0443\u043d\u043d\u0443\u044e \u0441\u0431\u043e\u0440\u043e\u0447\u043d\u0443\u044e \u0441\u0438\u0441\u0442\u0435\u043c\u0443, \u043f\u043e\u044d\u0442\u043e\u043c\u0443 \u0434\u043b\u044f \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0438 \u0441\u043e\u0431\u0438\u0440\u0430\u0435\u043c\u043e\u0433\u043e \u0435\u0439 \u043a\u043e\u0434\u0430 \u0432\u043e\u0441\u043f\u043e\u043b\u044c\u0437\u0443\u0435\u043c\u0441\u044f PVS-Studio Standalone. \u041a\u0430\u043a \u043f\u043e\u0434\u0433\u043e\u0442\u043e\u0432\u0438\u0442\u044c \u043f\u0430\u043a\u0435\u0442 Quark_EDKII \u043a \u0441\u0431\u043e\u0440\u043a\u0435 \u2014 <a href=\"https:\/\/downloadcenter.intel.com\/downloads\/eula\/23197\/Intel-Quark-BSP?httpDown=http%3A%2F%2Fdownloadmirror.intel.com%2F23197%2Feng%2Fquark-x1000-bsp-build-sw-rel-user-guide..pdf\">\u0447\u0438\u0442\u0430\u0439\u0442\u0435 \u0432 \u044d\u0442\u043e\u043c \u0434\u043e\u043a\u0443\u043c\u0435\u043d\u0442\u0435<\/a>, \u043f\u043e\u0434\u0440\u043e\u0431\u043d\u0435\u0435 \u043e\u0441\u0442\u0430\u043d\u0430\u0432\u043b\u0438\u0432\u0430\u0442\u044c\u0441\u044f \u043d\u0430 \u043d\u0435\u0439 \u0437\u0434\u0435\u0441\u044c \u044f \u043d\u0435 \u0431\u0443\u0434\u0443.<\/p>\n<h4><b>\u0417\u0430\u043f\u0443\u0441\u043a \u0430\u043d\u0430\u043b\u0438\u0437\u0430\u0442\u043e\u0440\u0430<\/b><\/h4>\n<p>\u0417\u0430\u043f\u0443\u0441\u043a\u0430\u0435\u043c PVS-Studio Standalone, \u043d\u0430\u0436\u0438\u043c\u0430\u0435\u043c \u043a\u043d\u043e\u043f\u043a\u0443 Analyze your files&#8230;, \u043e\u0442\u043a\u0440\u044b\u0432\u0430\u0435\u0442\u0441\u044f \u043e\u043a\u043d\u043e Compiler Monitoring, \u0432 \u043a\u043e\u0442\u043e\u0440\u043e\u043c \u043d\u0430\u0436\u0438\u043c\u0430\u0435\u043c \u0435\u0434\u0438\u043d\u0441\u0442\u0432\u0435\u043d\u043d\u0443\u044e \u043a\u043d\u043e\u043f\u043a\u0443 Start Monitoring. \u0422\u0435\u043f\u0435\u0440\u044c \u043e\u0442\u043a\u0440\u044b\u0432\u0430\u0435\u043c \u043a\u043e\u043d\u0441\u043e\u043b\u044c \u0432 \u043f\u0430\u043f\u043a\u0435 \u0441 Quark_EDKII \u0438 \u0432\u044b\u043f\u043e\u043b\u043d\u044f\u0435\u043c \u043a\u043e\u043c\u0430\u043d\u0434\u0443 <i>quarkbuild -r32 S QuarkPlatform<\/i> \u0434\u043b\u044f \u0441\u0431\u043e\u0440\u043a\u0438 \u0440\u0435\u043b\u0438\u0437\u043d\u043e\u0439 \u0432\u0435\u0440\u0441\u0438\u0438 \u043f\u0440\u043e\u0448\u0438\u0432\u043a\u0438. \u0416\u0434\u0435\u043c \u043e\u043a\u043e\u043d\u0447\u0430\u043d\u0438\u044f \u0441\u0431\u043e\u0440\u043a\u0438, \u0432 \u043e\u043a\u043d\u0435 Compiler Monitoring \u043d\u0430\u0431\u043b\u044e\u0434\u0430\u0435\u043c \u0437\u0430 \u0440\u043e\u0441\u0442\u043e\u043c \u0447\u0438\u0441\u043b\u0430 \u043e\u0431\u043d\u0430\u0440\u0443\u0436\u0435\u043d\u043d\u044b\u0445 \u0432\u044b\u0437\u043e\u0432\u043e\u0432 \u043a\u043e\u043c\u043f\u0438\u043b\u044f\u0442\u043e\u0440\u0430, \u043f\u043e\u0441\u043b\u0435 \u043e\u043a\u043e\u043d\u0447\u0430\u043d\u0438\u044f \u0441\u0431\u043e\u0440\u043a\u0438 \u043d\u0430\u0436\u0438\u043c\u0430\u0435\u043c \u043a\u043d\u043e\u043f\u043a\u0443 Stop Monitoring \u0438 \u0436\u0434\u0435\u043c \u043e\u043a\u043e\u043d\u0447\u0430\u043d\u0438\u044f \u0430\u043d\u0430\u043b\u0438\u0437\u0430. <\/p>\n<h4><b>\u0420\u0435\u0437\u0443\u043b\u044c\u0442\u0430\u0442\u044b \u0430\u043d\u0430\u043b\u0438\u0437\u0430<\/b><\/h4>\n<p>\u0414\u043b\u044f \u0442\u0435\u043a\u0443\u0449\u0435\u0439 \u0432\u0435\u0440\u0441\u0438\u0438 Quark_EDKII_v1.1.0 \u0430\u043d\u0430\u043b\u0438\u0437\u0430\u0442\u043e\u0440 \u043f\u043e\u043a\u0430\u0437\u044b\u0432\u0430\u0435\u0442 96 \u043f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0439 \u043f\u0435\u0440\u0432\u043e\u0433\u043e \u0443\u0440\u043e\u0432\u043d\u044f, 100 \u2014 \u0432\u0442\u043e\u0440\u043e\u0433\u043e \u0438 63-\u0442\u0440\u0435\u0442\u044c\u0435\u0433\u043e (\u0441 \u043d\u0430\u0439\u0441\u0442\u0440\u043e\u0439\u043a\u0430\u043c\u0438 \u043f\u043e \u0443\u043c\u043e\u043b\u0447\u0430\u043d\u0438\u044e, \u0442.\u0435. \u043f\u0440\u0438 \u0432\u044b\u0431\u043e\u0440\u0435 \u0442\u043e\u043b\u044c\u043a\u043e General Analysis). \u041e\u0442\u0441\u043e\u0440\u0442\u0438\u0440\u0443\u0435\u043c \u0438\u0445 \u043f\u043e \u043d\u043e\u043c\u0435\u0440\u0443 \u043f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u044f \u0438 \u043f\u0440\u0438\u0441\u0442\u0443\u043f\u0438\u043c \u043a \u043f\u043e\u0438\u0441\u043a\u0443 \u043e\u0448\u0438\u0431\u043e\u043a.<\/p>\n<p>  <b>\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435<\/b>: V521 Such expressions using the &#8216;,&#8217; operator are dangerous. Make sure the expression is correct.<br \/>  <b>\u0424\u0430\u0439\u043b<\/b>: quarkplatformpkg\\pci\\dxe\\pcihostbridge\\pcihostbridge.c, 181, 272<br \/>  <b>\u041a\u043e\u0434<\/b>:<\/p>\n<pre><code>for (TotalRootBridgeFound = 0, IioResourceMapEntry = 0;  TotalRootBridgeFound &lt; HostBridge-&gt;RootBridgeCount, IioResourceMapEntry &lt; MaxIIO;  IioResourceMapEntry++) {<\/code><\/pre>\n<p><b>\u041a\u043e\u043c\u043c\u0435\u043d\u0442\u0430\u0440\u0438\u0439<\/b>: \u043d\u0435\u043f\u0440\u0430\u0432\u0438\u043b\u044c\u043d\u043e\u0435 \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u043d\u0438\u0435 \u043e\u043f\u0435\u0440\u0430\u0442\u043e\u0440\u0430 \u00ab\u0437\u0430\u043f\u044f\u0442\u0430\u044f\u00bb \u0432 \u0443\u0441\u043b\u043e\u0432\u0438\u0438. \u041d\u0430\u043f\u043e\u043c\u043d\u044e, \u0447\u0442\u043e \u044d\u0442\u043e\u0442 \u043e\u043f\u0435\u0440\u0430\u0442\u043e\u0440 \u0438\u043c\u0435\u0435\u0442 \u043c\u0438\u043d\u0438\u043c\u0430\u043b\u044c\u043d\u044b\u0439 \u043f\u0440\u0438\u043e\u0440\u0438\u0442\u0435\u0442, \u0432\u044b\u0447\u0438\u0441\u043b\u044f\u0435\u0442 \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u044f \u043e\u0431\u043e\u0438\u0445 \u043e\u043f\u0435\u0440\u0430\u043d\u0434\u043e\u0432, \u043d\u043e \u0441\u0430\u043c \u043f\u0440\u0438\u043d\u0438\u043c\u0430\u0435\u0442 \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435 \u043f\u0440\u0430\u0432\u043e\u0433\u043e. \u0412 \u0434\u0430\u043d\u043d\u043e\u043c \u0441\u043b\u0443\u0447\u0430\u0435 \u0443\u0441\u043b\u043e\u0432\u0438\u0435 \u043f\u043e\u043b\u043d\u043e\u0441\u0442\u044c\u044e \u0430\u043d\u0430\u043b\u043e\u0433\u0438\u0447\u043d\u043e IioResourceMapEntry &lt; MaxIIO, \u0430 \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0430 TotalRootBridgeFound &lt; HostBridge-&gt;RootBridgeCount \u0445\u043e\u0442\u044c \u0438 \u0432\u044b\u043f\u043e\u043b\u043d\u044f\u0435\u0442\u0441\u044f, \u043d\u043e \u043d\u0430 \u043f\u0440\u043e\u0434\u043e\u043b\u0436\u0435\u043d\u0438\u0435 \u0438\u043b\u0438 \u043f\u0440\u0435\u0440\u044b\u0432\u0430\u043d\u0438\u0435 \u0446\u0438\u043a\u043b\u0430 \u043d\u0435 \u0432\u043b\u0438\u044f\u0435\u0442.<br \/>  <b>\u041f\u0440\u0435\u0434\u043b\u0430\u0433\u0430\u0435\u043c\u044b\u0439 \u0444\u0438\u043a\u0441<\/b>: \u0437\u0430\u043c\u0435\u043d\u0438\u0442\u044c \u0437\u0430\u043f\u044f\u0442\u0443\u044e \u043d\u0430 &#038;&#038; \u0432 \u0443\u0441\u043b\u043e\u0432\u0438\u0438.<\/p>\n<p>  <b>\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435<\/b>: V524 It is odd that the body of &#8216;AllocateRuntimePages&#8217; function is fully equivalent to the body of &#8216;AllocatePages&#8217; function.<br \/>  <b>\u0424\u0430\u0439\u043b<\/b>: mdepkg\\library\\smmmemoryallocationlib\\memoryallocationlib.c, 208 \u0438 \u0434\u0430\u043b\u0435\u0435 <br \/>  <b>\u041a\u043e\u0434<\/b>:<\/p>\n<pre><code>\/**   Allocates one or more 4KB pages of type EfiBootServicesData.    Allocates the number of 4KB pages of type EfiBootServicesData and returns a pointer    to the allocated buffer.  The buffer returned is aligned on a 4KB boundary.  If    Pages is 0, then NULL is returned.  If there is not enough memory remaining to    satisfy the request, then NULL is returned.    @param  Pages                 The number of 4 KB pages to allocate.    @return A pointer to the allocated buffer or NULL if allocation fails.  **\/ VOID * EFIAPI AllocatePages (   IN UINTN  Pages   ) {   return InternalAllocatePages (EfiRuntimeServicesData, Pages); }<\/code><\/pre>\n<p><b>\u041a\u043e\u043c\u043c\u0435\u043d\u0442\u0430\u0440\u0438\u0439<\/b>: \u043a\u043e\u0434 \u043f\u0440\u043e\u0442\u0438\u0432\u043e\u0440\u0435\u0447\u0438\u0442 \u043a\u043e\u043c\u043c\u0435\u043d\u0442\u0430\u0440\u0438\u044e \u0438 \u0432\u044b\u0434\u0435\u043b\u044f\u0435\u0442 \u043f\u0430\u043c\u044f\u0442\u044c \u0442\u0438\u043f\u0430 EfiRuntimeServicesData \u0432\u043c\u0435\u0441\u0442\u043e \u043f\u043e\u0434\u0440\u0430\u0437\u0443\u043c\u0435\u0432\u0430\u0435\u043c\u043e\u0433\u043e EfiBootServicesData. \u0420\u0430\u0437\u043d\u0438\u0446\u0430 \u043c\u0435\u0436\u0434\u0443 \u043d\u0438\u043c\u0438 \u0432 \u0442\u043e\u043c, \u0447\u0442\u043e \u043f\u0430\u043c\u044f\u0442\u044c \u0432\u0442\u043e\u0440\u043e\u0433\u043e \u0442\u0438\u043f\u0430 \u0431\u0443\u0434\u0435\u0442 \u0430\u0432\u0442\u043e\u043c\u0430\u0442\u0438\u0447\u0435\u0441\u043a\u0438 \u043e\u0441\u0432\u043e\u0431\u043e\u0436\u0434\u0435\u043d\u0430 \u043f\u043e \u043e\u043a\u043e\u043d\u0447\u0430\u043d\u0438\u044e \u0444\u0430\u0437\u044b <abbr title=\"Boot Device Selection\">BDS<\/abbr>, \u0430 \u043f\u0430\u043c\u044f\u0442\u044c \u043f\u0435\u0440\u0432\u043e\u0433\u043e \u0442\u0438\u043f\u0430 \u0434\u043e\u043b\u0436\u043d\u0430 \u0431\u044b\u0442\u044c \u043e\u0441\u0432\u043e\u0431\u043e\u0436\u0434\u0435\u043d\u0430 \u044f\u0432\u043d\u044b\u043c \u0432\u044b\u0437\u043e\u0432\u043e\u043c FreeMem \u0434\u043e \u043e\u043a\u043e\u043d\u0447\u0430\u043d\u0438\u044f \u0432\u044b\u0448\u0435\u0443\u043f\u043e\u043c\u044f\u043d\u0443\u0442\u043e\u0439 \u0444\u0430\u0437\u044b, \u0438\u043d\u0430\u0447\u0435 \u043e\u043d\u0430 \u043e\u0441\u0442\u0430\u043d\u0435\u0442\u0441\u044f \u043d\u0435\u0434\u043e\u0441\u0442\u0443\u043f\u043d\u043e\u0439 \u0434\u043b\u044f \u041e\u0421 \u043d\u0430\u0432\u0441\u0435\u0433\u0434\u0430. \u0412 \u0438\u0442\u043e\u0433\u0435 \u043c\u0435\u043b\u043a\u0430\u044f \u0432\u0440\u043e\u0434\u0435 \u0431\u044b \u043e\u0448\u0438\u0431\u043a\u0430 \u043c\u043e\u0436\u0435\u0442 \u043f\u0440\u0438\u0432\u043e\u0434\u0438\u0442\u044c \u043a \u0441\u043e\u0432\u0435\u0440\u0448\u0435\u043d\u043d\u043e \u043d\u0435\u043f\u043e\u043d\u044f\u0442\u043d\u044b\u043c \u0443\u0442\u0435\u0447\u043a\u0430\u043c \u043f\u0430\u043c\u044f\u0442\u0438 \u0438 \u0444\u0440\u0430\u0433\u043c\u0435\u043d\u0442\u0430\u0446\u0438\u0438 \u0434\u043e\u0441\u0442\u0443\u043f\u043d\u043e\u0433\u043e \u041e\u0421 \u0430\u0434\u0440\u0435\u0441\u043d\u043e\u0433\u043e \u043f\u0440\u043e\u0441\u0442\u0440\u0430\u043d\u0441\u0442\u0432\u0430.<br \/>  <b>\u041f\u0440\u0435\u0434\u043b\u0430\u0433\u0430\u0435\u043c\u044b\u0439 \u0444\u0438\u043a\u0441<\/b>: \u0437\u0430\u043c\u0435\u043d\u0430 \u0442\u0438\u043f\u0430 \u043f\u0430\u043c\u044f\u0442\u0438 \u043d\u0430 EfiBootServicesData \u0432\u043e \u0432\u0441\u0435\u0445 \u043d\u0435-Runtime \u0444\u0443\u043d\u043a\u0446\u0438\u044f\u0445 \u044d\u0442\u043e\u0433\u043e \u0444\u0430\u0439\u043b\u0430.<\/p>\n<p>  <b>\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435<\/b>: V524 It is odd that the body of &#8216;OhciSetLsThreshold&#8217; function is fully equivalent to the body of &#8216;OhciSetPeriodicStart&#8217; function. <br \/>  <b>\u0424\u0430\u0439\u043b<\/b>: quarksocpkg\\quarksouthcluster\\usb\\ohci\\pei\\ohcireg.c, 1010, 1015 \u0438 quarksocpkg\\quarksouthcluster\\usb\\ohci\\dxe\\ohcireg.c, 1010, 1040<br \/>  <b>\u041a\u043e\u0434<\/b>:<\/p>\n<pre><code>EFI_STATUS OhciSetLsThreshold (   IN USB_OHCI_HC_DEV      *Ohc,   IN UINT32               Value   ) {   EFI_STATUS              Status;     Status = OhciSetOperationalReg (Ohc-&gt;PciIo, HC_PERIODIC_START, &Value);    return Status; }<\/code><\/pre>\n<p><b>\u041a\u043e\u043c\u043c\u0435\u043d\u0442\u0430\u0440\u0438\u0439<\/b>: \u0435\u0449\u0435 \u043e\u0434\u043d\u0430 \u0436\u0435\u0440\u0442\u0432\u0430 \u043a\u043e\u043f\u0438\u043f\u0430\u0441\u0442\u044b, \u043d\u0430 \u044d\u0442\u043e\u0442 \u0440\u0430\u0437 \u0432\u043c\u0435\u0441\u0442\u043e HC_LS_THREASHOLD \u0443\u0441\u0442\u0430\u043d\u0430\u0432\u043b\u0438\u0432\u0430\u0435\u0442\u0441\u044f \u0438 \u043f\u0440\u043e\u0432\u0435\u0440\u044f\u0435\u0442\u0441\u044f \u0431\u0438\u0442 HC_PERIODIC_START.<br \/>  <b>\u041f\u0440\u0435\u0434\u043b\u0430\u0433\u0430\u0435\u043c\u044b\u0439 \u0444\u0438\u043a\u0441<\/b>: \u0437\u0430\u043c\u0435\u043d\u0438\u0442\u044c \u0431\u0438\u0442 \u043d\u0430 \u043f\u0440\u0430\u0432\u0438\u043b\u044c\u043d\u044b\u0439.<\/p>\n<p>  <b>\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435<\/b>: V528 It is odd that pointer to &#8216;char&#8217; type is compared with the &#8216;\\0&#8217; value. Probably meant: *MatchLang != &#8216;\\0&#8217;. <br \/>  <b>\u0424\u0430\u0439\u043b<\/b>: quarkplatformpkg\\platform\\dxe\\smbiosmiscdxe\\miscnumberofinstallablelanguagesfunction.c, 95<br \/>  <b>\u041a\u043e\u0434<\/b>:<\/p>\n<pre><code>for (MatchLang = Languages, (*Offset) = 0; MatchLang != '\\0'; (*Offset)++) {       \/\/       \/\/ Seek to the end of current match language.        \/\/       for (EndMatchLang = MatchLang; *EndMatchLang != '\\0' && *EndMatchLang != ';'; EndMatchLang++);          if ((EndMatchLang == MatchLang + CompareLength) && AsciiStrnCmp(MatchLang, BestLanguage, CompareLength) == 0) {         \/\/         \/\/ Find the current best Language in the supported languages         \/\/         break;       }       \/\/       \/\/ best language match be in the supported language.       \/\/       ASSERT (*EndMatchLang == ';');       MatchLang = EndMatchLang + 1;     }<\/code><\/pre>\n<p><b>\u041a\u043e\u043c\u043c\u0435\u043d\u0442\u0430\u0440\u0438\u0439<\/b>: \u0432 \u0440\u0435\u0437\u0443\u043b\u044c\u0442\u0430\u0442\u0435 \u043e\u0448\u0438\u0431\u043a\u0438 \u0441 \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u043e\u0439 \u043d\u0435\u0440\u0430\u0437\u044b\u043c\u0435\u043d\u043e\u0432\u0430\u043d\u043d\u043e\u0433\u043e \u0443\u043a\u0430\u0437\u0430\u0442\u0435\u043b\u044f \u0446\u0438\u043a\u043b \u0441\u0442\u0430\u043b \u0431\u0435\u0441\u043a\u043e\u043d\u0435\u0447\u043d\u044b\u043c, \u0438 \u043e\u0442 \u0437\u0430\u0446\u0438\u043a\u043b\u0438\u0432\u0430\u043d\u0438\u044f \u0441\u043f\u0430\u0441\u0430\u0435\u0442 \u0442\u043e\u043b\u044c\u043a\u043e \u0442\u043e, \u0447\u0442\u043e \u0432\u043d\u0443\u0442\u0440\u0438 \u043d\u0430\u0448\u0435\u043b\u0441\u044f break. <br \/>  <b>\u041f\u0440\u0435\u0434\u043b\u0430\u0433\u0430\u0435\u043c\u044b\u0439 \u0444\u0438\u043a\u0441<\/b>: \u0434\u043e\u0431\u0430\u0432\u0438\u0442\u044c \u043d\u0435\u0434\u043e\u0441\u0442\u0430\u044e\u0449\u0435\u0435 \u0440\u0430\u0437\u044b\u043c\u0435\u043d\u043e\u0432\u0430\u043d\u0438\u0435.<\/p>\n<p>  <b>\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435<\/b>: V535 The variable &#8216;Index&#8217; is being used for this loop and for the outer loop.<br \/>  <b>\u0424\u0430\u0439\u043b<\/b>: mdemodulepkg\\core\\pismmcore\\dispatcher.c, 1233, 1269, 1316<br \/>  <b>\u041a\u043e\u0434<\/b>:<\/p>\n<pre><code>for (Index = 0; Index &lt; HandleCount; Index++) {     FvHandle = HandleBuffer[Index];     ...     for (Index = 0; Index &lt; sizeof (mSmmFileTypes)\/sizeof (EFI_FV_FILETYPE); Index++) {       ...     }     ...     for (Index = 0; Index &lt; AprioriEntryCount; Index++) {       ...     } } <\/code><\/pre>\n<p><b>\u041a\u043e\u043c\u043c\u0435\u043d\u0442\u0430\u0440\u0438\u0439<\/b>: \u043f\u0440\u0438\u043c\u0435\u0440 \u043a\u043e\u0434\u0430, \u043a\u043e\u0442\u043e\u0440\u044b\u0439 \u0440\u0430\u0431\u043e\u0442\u0430\u0435\u0442 \u043b\u0438\u0448\u044c \u0431\u043b\u0430\u0433\u043e\u0434\u0430\u0440\u044f \u0443\u0434\u0430\u0447\u043d\u043e\u043c\u0443 \u0441\u0442\u0435\u0447\u0435\u043d\u0438\u044e \u043e\u0431\u0441\u0442\u043e\u044f\u0442\u0435\u043b\u044c\u0441\u0442\u0432. HandleCount \u0432\u043e \u0432\u043d\u0435\u0448\u043d\u0435\u043c \u0446\u0438\u043a\u043b\u0435 \u043f\u043e\u0447\u0442\u0438 \u0432\u0441\u0435\u0433\u0434\u0430 \u0440\u0430\u0432\u0435\u043d 1, \u0432 \u043c\u0430\u0441\u0441\u0438\u0432\u0435 mSmmFileTypes \u0442\u043e\u0436\u0435 \u043d\u0430 \u0434\u0430\u043d\u043d\u044b\u0439 \u043c\u043e\u043c\u0435\u043d\u0442 \u0440\u043e\u0432\u043d\u043e \u043e\u0434\u0438\u043d \u044d\u043b\u0435\u043c\u0435\u043d\u0442, \u0430 AprioriEntryCount \u043d\u0435 \u043c\u0435\u043d\u044c\u0448\u0435 1, \u043f\u043e\u044d\u0442\u043e\u043c\u0443 \u0432\u043d\u0435\u0448\u043d\u0438\u0439 \u0446\u0438\u043a\u043b \u0437\u0430\u0432\u0435\u0440\u0448\u0430\u0435\u0442\u0441\u044f. \u042f\u0441\u043d\u043e, \u0447\u0442\u043e \u043f\u043e\u0434\u0440\u0430\u0437\u0443\u043c\u0435\u0432\u0430\u043b\u043e\u0441\u044c \u0441\u043e\u0432\u0441\u0435\u043c \u0438\u043d\u043e\u0435 \u043f\u043e\u0432\u0435\u0434\u0435\u043d\u0438\u0435, \u043d\u043e \u0443 \u043a\u043e\u043f\u0438\u043f\u0430\u0441\u0442\u0430 \u0441\u0432\u043e\u0435 \u043c\u043d\u0435\u043d\u0438\u0435 \u043d\u0430 \u044d\u0442\u043e \u0441\u0447\u0435\u0442.<br \/>  <b>\u041f\u0440\u0435\u0434\u043b\u0430\u0433\u0430\u0435\u043c\u044b\u0439 \u0444\u0438\u043a\u0441<\/b>: \u0432\u0432\u0435\u0441\u0442\u0438 \u043d\u0435\u0437\u0430\u0432\u0438\u0441\u0438\u043c\u044b\u0435 \u0441\u0447\u0435\u0442\u0447\u0438\u043a\u0438 \u0434\u043b\u044f \u043a\u0430\u0436\u0434\u043e\u0433\u043e \u0446\u0438\u043a\u043b\u0430.<\/p>\n<p>  <b>\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435<\/b>: V547 Expression &#8216;(0) &gt; (1 \u2014 Dtr1.field.tCMD)&#8217; is always false. Unsigned type value is never &lt; 0.<br \/>  <b>\u0424\u0430\u0439\u043b<\/b>: quarksocpkg\\quarknorthcluster\\memoryinit\\pei\\meminit.c, 483, 487<br \/>  <b>\u041a\u043e\u0434<\/b>:<\/p>\n<pre><code>#define MMAX(a,b)      ((a)&gt;(b)?(a):(b)) ... #pragma pack(1) typedef union {     uint32_t    raw;     struct {         ...         uint32_t tCMD               :2;             \/**&lt; bit [5:4] Command transport duration *\/         ...     } field; } RegDTR1;                                          \/**&lt; DRAM Timing Register 1 *\/ #pragma pack() ... if (mrc_params-&gt;ddr_speed == DDRFREQ_800)   {     Dtr3.field.tXP = MMAX(0, 1 - Dtr1.field.tCMD);   }   else   {     Dtr3.field.tXP = MMAX(0, 2 - Dtr1.field.tCMD);   }<\/code><\/pre>\n<p><b>\u041a\u043e\u043c\u043c\u0435\u043d\u0442\u0430\u0440\u0438\u0439<\/b>: \u043f\u0440\u043e\u0441\u0442\u0435\u0439\u0448\u0438\u0439 \u043c\u0430\u043a\u0440\u043e\u0441 \u0438 \u0430\u0432\u0442\u043e\u043c\u0430\u0442\u0438\u0447\u0435\u0441\u043a\u043e\u0435 \u043f\u0440\u0438\u0432\u0435\u0434\u0435\u043d\u0438\u0435 \u0442\u0438\u043f\u043e\u0432 \u043d\u0430\u043d\u043e\u0441\u044f\u0442 \u043e\u0442\u0432\u0435\u0442\u043d\u044b\u0439 \u0443\u0434\u0430\u0440. \u0422.\u043a. tCMD \u2014 \u044d\u0442\u043e \u0431\u0438\u0442\u043e\u0432\u043e\u0435 \u043f\u043e\u043b\u0435 \u0442\u0438\u043f\u0430 uint32_t, \u0442\u043e \u0432 \u0443\u0441\u043b\u043e\u0432\u0438\u0438 0 &gt; 1 \u2014 tCMD \u043e\u0431\u0435 \u0447\u0430\u0441\u0442\u0438 \u0431\u0443\u0434\u0443\u0442 \u0430\u0432\u0442\u043e\u043c\u0430\u0442\u0438\u0447\u0435\u0441\u043a\u0438 \u043f\u0440\u0438\u0432\u0435\u0434\u0435\u043d\u044b \u043a uint32_t, \u043e\u0442\u0447\u0435\u0433\u043e \u043e\u043d\u043e \u0441\u0442\u0430\u043d\u0435\u0442 \u043b\u043e\u0436\u043d\u044b\u043c \u043f\u0440\u0438 \u043b\u044e\u0431\u043e\u043c \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0438 tCMD.<br \/>  <b>\u041f\u0440\u0435\u0434\u043b\u0430\u0433\u0430\u0435\u043c\u044b\u0439 \u0444\u0438\u043a\u0441<\/b>:<\/p>\n<pre><code>if (mrc_params-&gt;ddr_speed == DDRFREQ_800)   {     Dtr3.field.tXP = Dtr1.field.tCMD &gt; 0 ? 0 : 1 ;   }   else   {     Dtr3.field.tXP = Dtr1.field.tCMD &gt; 1 ? 0 : 2 - Dtr1.field.tCMD;   }<\/code><\/pre>\n<p>  <b>\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435<\/b>: V547 Expression &#8216;PollCount &gt;= ((1000 * 1000) \/ 25)&#8217; is always false. The value range of unsigned char type: [0, 255].<br \/>  <b>\u0424\u0430\u0439\u043b<\/b>: quarksocpkg\\quarksouthcluster\\i2c\\common\\i2ccommon.c, 297<br \/>  <b>\u041a\u043e\u0434<\/b>:<\/p>\n<pre><code>UINT8       PollCount; ... do {     Data = *((volatile UINT32 *) (UINTN)(Addr));     if ((Data & I2C_REG_RAW_INTR_STAT_TX_ABRT) != 0) {       Status = EFI_ABORTED;       break;     }     if ((Data & I2C_REG_RAW_INTR_STAT_TX_OVER) != 0) {       Status = EFI_DEVICE_ERROR;       break;     }     if ((Data & I2C_REG_RAW_INTR_STAT_RX_OVER) != 0) {       Status = EFI_DEVICE_ERROR;       break;     }     if ((Data & I2C_REG_RAW_INTR_STAT_STOP_DET) != 0) {       Status = EFI_SUCCESS;       break;     }     MicroSecondDelay(TI2C_POLL);     PollCount++;     if (PollCount &gt;= MAX_STOP_DET_POLL_COUNT) {       Status = EFI_TIMEOUT;       break;     }    } while (TRUE);<\/code><\/pre>\n<p><b>\u041a\u043e\u043c\u043c\u0435\u043d\u0442\u0430\u0440\u0438\u0439<\/b>: \u043c\u0430\u043a\u0440\u043e\u0441 MAX_STOP_DET_POLL_COUNT \u0440\u0430\u0437\u0432\u043e\u0440\u0430\u0447\u0438\u0432\u0430\u0435\u0442\u0441\u044f \u0432 40000, \u0430 PollCount \u043d\u0435 \u043c\u043e\u0436\u0435\u0442 \u0431\u044b\u0442\u044c \u0431\u043e\u043b\u044c\u0448\u0435 255, \u0432 \u0440\u0435\u0437\u0443\u043b\u044c\u0442\u0430\u0442\u0435 \u0432\u043e\u0437\u043c\u043e\u0436\u0435\u043d \u0431\u0435\u0441\u043a\u043e\u043d\u0435\u0447\u043d\u044b\u0439 \u0446\u0438\u043a\u043b.<br \/>  <b>\u041f\u0440\u0435\u0434\u043b\u0430\u0433\u0430\u0435\u043c\u044b\u0439 \u0444\u0438\u043a\u0441<\/b>: \u0441\u043c\u0435\u043d\u0438\u0442\u044c \u0442\u0438\u043f PollCount \u043d\u0430 UINT32.<\/p>\n<p>  <b>\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435<\/b>: V560 A part of conditional expression is always true: (0x00040000).<br \/>  <b>\u0424\u0430\u0439\u043b<\/b>: quarksocpkg\\quarknorthcluster\\library\\intelqnclib\\pciexpress.c, 370<br \/>  <b>\u041a\u043e\u0434<\/b>:<\/p>\n<pre><code>if ((QNCMmPci32 (0, Bus, Device, Function, (CapOffset + PCIE_LINK_CAP_OFFSET))        && B_QNC_PCIE_LCAP_CPM) != B_QNC_PCIE_LCAP_CPM) {       return;     }<\/code><\/pre>\n<p><b>\u041a\u043e\u043c\u043c\u0435\u043d\u0442\u0430\u0440\u0438\u0439<\/b>: \u0432\u043c\u0435\u0441\u0442\u043e \u043f\u043e\u0431\u0438\u0442\u043e\u0432\u043e\u0433\u043e AND \u0432 \u0432\u044b\u0440\u0430\u0436\u0435\u043d\u0438\u0435 \u0437\u0430\u043a\u0440\u0430\u043b\u043e\u0441\u044c \u043b\u043e\u0433\u0438\u0447\u0435\u0441\u043a\u043e\u0435, \u0432 \u0440\u0435\u0437\u0443\u043b\u044c\u0442\u0430\u0442\u0435 \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0430 \u0441\u0442\u0430\u043b\u0430 \u0431\u0435\u0441\u043f\u043e\u043b\u0435\u0437\u043d\u043e\u0439.<br \/>  <b>\u041f\u0440\u0435\u0434\u043b\u0430\u0433\u0430\u0435\u043c\u044b\u0439 \u0444\u0438\u043a\u0441<\/b>: <\/p>\n<pre><code>if ((QNCMmPci32 (0, Bus, Device, Function, (CapOffset + PCIE_LINK_CAP_OFFSET)) & B_QNC_PCIE_LCAP_CPM)       != B_QNC_PCIE_LCAP_CPM) {       return;     }<\/code><\/pre>\n<p>  <b>\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435<\/b>: V560 A part of conditional expression is always true: 0x0FFFFF000.<br \/>  <b>\u0424\u0430\u0439\u043b<\/b>: quarksocpkg\\quarknorthcluster\\library\\intelqnclib\\intelqnclib.c, 378<br \/>  <b>\u041a\u043e\u0434<\/b>:<\/p>\n<pre><code>return  QNCPortRead(QUARK_NC_HOST_BRIDGE_SB_PORT_ID, QUARK_NC_HOST_BRIDGE_HMBOUND_REG) && HMBOUND_MASK;<\/code><\/pre>\n<p><b>\u041a\u043e\u043c\u043c\u0435\u043d\u0442\u0430\u0440\u0438\u0439<\/b>: \u0442\u043e \u0436\u0435 \u0441\u0430\u043c\u043e\u0435, \u0447\u0442\u043e \u0438 \u0432 \u043f\u0440\u0435\u0434\u044b\u0434\u0443\u0449\u0435\u043c \u0441\u043b\u0443\u0447\u0430\u0435, \u0442\u043e\u043b\u044c\u043a\u043e \u0435\u0449\u0435 \u0445\u0443\u0436\u0435, \u0432 \u044d\u0442\u043e\u0442 \u0440\u0430\u0437 \u043f\u043e\u0441\u0442\u0440\u0430\u0434\u0430\u043b\u043e \u0432\u043e\u0437\u0432\u0440\u0430\u0449\u0430\u0435\u043c\u043e\u0435 \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435<br \/>  <b>\u041f\u0440\u0435\u0434\u043b\u0430\u0433\u0430\u0435\u043c\u044b\u0439 \u0444\u0438\u043a\u0441<\/b>: <\/p>\n<pre><code>return  QNCPortRead(QUARK_NC_HOST_BRIDGE_SB_PORT_ID, QUARK_NC_HOST_BRIDGE_HMBOUND_REG) & HMBOUND_MASK;<\/code><\/pre>\n<p>  <b>\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435<\/b>: V560 A part of conditional expression is always true: 0x00400.<br \/>  <b>\u0424\u0430\u0439\u043b<\/b>: quarksocpkg\\quarksouthcluster\\usb\\ohci\\pei\\ohcireg.c, 1065 \u0438 quarksocpkg\\quarksouthcluster\\usb\\ohci\\dxe\\ohcireg.c, 1070<br \/>  <b>\u041a\u043e\u0434<\/b>:<\/p>\n<pre><code>if (Field & (RH_DEV_REMOVABLE || RH_PORT_PWR_CTRL_MASK)) {<\/code><\/pre>\n<p><b>\u041a\u043e\u043c\u043c\u0435\u043d\u0442\u0430\u0440\u0438\u0439<\/b>: \u043d\u0430 \u044d\u0442\u043e\u0442 \u0440\u0430\u0437 \u043d\u0435 \u043f\u043e\u0432\u0435\u0437\u043b\u043e \u043f\u043e\u0431\u0438\u0442\u043e\u0432\u043e\u043c\u0443 OR.<br \/>  <b>\u041f\u0440\u0435\u0434\u043b\u0430\u0433\u0430\u0435\u043c\u044b\u0439 \u0444\u0438\u043a\u0441<\/b>: <\/p>\n<pre><code>if (Field & (RH_DEV_REMOVABLE | RH_PORT_PWR_CTRL_MASK)) {<\/code><\/pre>\n<p>  <b>\u041f\u0440\u0435\u0434\u0443\u043f\u0440\u0435\u0436\u0434\u0435\u043d\u0438\u0435<\/b>: V649 There are two &#8216;if&#8217; statements with identical conditional expressions. The first &#8216;if&#8217; statement contains function return. This means that the second &#8216;if&#8217; statement is senseless.<br \/>  <b>\u0424\u0430\u0439\u043b<\/b>: s:\\quarkplatformpkg\\platform\\dxe\\smbiosmiscdxe\\miscsystemmanufacturerfunction.c, 155<br \/>  <b>\u041a\u043e\u0434<\/b>:<\/p>\n<pre><code>  SerialNumStrLen = StrLen(SerialNumberPtr);   if (SerialNumStrLen &gt; SMBIOS_STRING_MAX_LENGTH) {     return EFI_UNSUPPORTED;   }   ...   SKUNumStrLen = StrLen(SKUNumberPtr);   if (SerialNumStrLen &gt; SMBIOS_STRING_MAX_LENGTH) {     return EFI_UNSUPPORTED;   }   ...   FamilyStrLen = StrLen(FamilyPtr);   if (SerialNumStrLen &gt; SMBIOS_STRING_MAX_LENGTH) {     return EFI_UNSUPPORTED;   }<\/code><\/pre>\n<p><b>\u041a\u043e\u043c\u043c\u0435\u043d\u0442\u0430\u0440\u0438\u0439<\/b>: \u0441\u043d\u043e\u0432\u0430 \u043f\u043e\u0434\u0432\u0435\u043b \u043a\u043e\u043f\u0438\u043f\u0430\u0441\u0442, \u0431\u0443\u0434\u044c \u043e\u043d \u043d\u0435\u043b\u0430\u0434\u0435\u043d. \u041f\u043e\u043b\u0443\u0447\u0430\u0435\u043c \u043e\u0434\u043d\u043e \u0437\u043d\u0430\u0447\u0435\u043d\u0438\u0435, \u043f\u0440\u043e\u0432\u0435\u0440\u044f\u0435\u043c \u0434\u0440\u0443\u0433\u043e\u0435, \u0438\u043c\u0435\u0435\u043c \u043d\u0435\u043f\u043e\u043d\u044f\u0442\u043d\u043e\u0435 \u043f\u043e\u0432\u0435\u0434\u0435\u043d\u0438\u0435 \u0444\u0443\u043d\u043a\u0446\u0438\u0438.<br \/>  <b>\u041f\u0440\u0435\u0434\u043b\u0430\u0433\u0430\u0435\u043c\u044b\u0439 \u0444\u0438\u043a\u0441<\/b>:<\/p>\n<pre><code>  SerialNumStrLen = StrLen(SerialNumberPtr);   if (SerialNumStrLen &gt; SMBIOS_STRING_MAX_LENGTH) {     return EFI_UNSUPPORTED;   }   ...   SKUNumStrLen = StrLen(SKUNumberPtr);   if (SKUNumStrLen &gt; SMBIOS_STRING_MAX_LENGTH) {     return EFI_UNSUPPORTED;   }   ...   FamilyStrLen = StrLen(FamilyPtr);   if (FamilyStrLen &gt; SMBIOS_STRING_MAX_LENGTH) {     return EFI_UNSUPPORTED;   }<\/code><\/pre>\n<p>  <\/p>\n<h4><b>\u0417\u0430\u043a\u043b\u044e\u0447\u0435\u043d\u0438\u0435<\/b><\/h4>\n<p>\u042f \u0441\u0442\u0430\u0440\u0430\u043b\u0441\u044f \u043e\u0442\u043e\u0431\u0440\u0430\u0442\u044c \u0442\u043e\u043b\u044c\u043a\u043e \u044f\u0432\u043d\u043e \u043e\u0448\u0438\u0431\u043e\u0447\u043d\u044b\u0439 \u043a\u043e\u0434, \u043d\u0435 \u043e\u0431\u0440\u0430\u0449\u0430\u044f \u0432\u043d\u0438\u043c\u0430\u043d\u0438\u044f \u043d\u0430 \u043f\u0440\u043e\u0431\u043b\u0435\u043c\u044b \u0432\u0440\u043e\u0434\u0435 \u043e\u043f\u0430\u0441\u043d\u043e\u0433\u043e \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u043d\u0438\u044f \u043e\u043f\u0435\u0440\u0430\u0442\u043e\u0440\u043e\u0432 \u0441\u0434\u0432\u0438\u0433\u0430, \u043f\u043e\u0432\u0442\u043e\u0440\u043d\u043e\u0433\u043e \u043f\u0440\u0438\u0441\u0432\u0430\u0438\u0432\u0430\u043d\u0438\u044f \u043e\u0434\u043d\u043e\u0439 \u0438 \u0442\u043e\u0439 \u0436\u0435 \u043f\u0435\u0440\u0435\u043c\u0435\u043d\u043d\u043e\u0439, \u043f\u0440\u0435\u043e\u0431\u0440\u0430\u0437\u043e\u0432\u0430\u043d\u0438\u044f \u043b\u0438\u0442\u0435\u0440\u0430\u043b\u043e\u0432 \u0438 \u043f\u0435\u0440\u0435\u043c\u0435\u043d\u043d\u044b\u0445 \u0446\u0435\u043b\u043e\u0433\u043e \u0442\u0438\u043f\u0430 \u0432 \u0443\u043a\u0430\u0437\u0430\u0442\u0435\u043b\u0438 \u0438 \u0442.\u043f., \u0433\u043e\u0432\u043e\u0440\u044f\u0449\u0438\u0435 \u0441\u043a\u043e\u0440\u0435\u0435 \u043e \u043d\u0438\u0437\u043a\u043e\u043c \u043a\u0430\u0447\u0435\u0441\u0442\u0432\u0435 \u043a\u043e\u0434\u0430, \u0447\u0435\u043c \u043e \u043d\u0430\u043b\u0438\u0447\u0438\u0438 \u043e\u0448\u0438\u0431\u043a\u0438 \u0432 \u043d\u0435\u043c, \u043d\u043e \u0434\u0430\u0436\u0435 \u0442\u0430\u043a \u0441\u043f\u0438\u0441\u043e\u043a \u043f\u043e\u043b\u0443\u0447\u0438\u043b\u0441\u044f \u0434\u043e\u0432\u043e\u043b\u044c\u043d\u043e \u0432\u043d\u0443\u0448\u0438\u0442\u0435\u043b\u044c\u043d\u044b\u043c. \u041f\u0440\u043e\u0435\u043a\u0442\u044b \u0434\u043b\u044f \u0434\u0435\u0441\u043a\u0442\u043e\u043f\u043d\u044b\u0445 \u043c\u0430\u0442\u0435\u0440\u0438\u043d\u0441\u043a\u0438\u0445 \u043f\u043b\u0430\u0442 \u0432 \u0441\u0440\u0435\u0434\u043d\u0435\u043c \u0431\u043e\u043b\u044c\u0448\u0435 \u044d\u0442\u043e\u0433\u043e \u0432 4-5 \u0440\u0430\u0437 (\u043e\u043a\u043e\u043b\u043e 4000 \u0432\u044b\u0437\u043e\u0432\u043e\u0432 \u043a\u043e\u043c\u043f\u0438\u043b\u044f\u0442\u043e\u0440\u0430 \u043f\u043e \u0441\u0447\u0435\u0442\u0447\u0438\u043a\u0443 \u0432 \u043e\u043a\u043d\u0435 Monitoring \u0432\u043c\u0435\u0441\u0442\u043e 800 \u0432 \u043d\u0430\u0448\u0435\u043c \u0441\u043b\u0443\u0447\u0430\u0435), \u0438 \u0442\u0430\u043c \u0432\u0441\u0442\u0440\u0435\u0447\u0430\u044e\u0442\u0441\u044f \u0442\u0435 \u0436\u0435 \u0442\u0438\u043f\u0438\u0447\u043d\u044b\u0435 \u043e\u0448\u0438\u0431\u043a\u0438.<br \/>  \u041a \u0441\u043e\u0436\u0430\u043b\u0435\u043d\u0438\u044e, Intel \u0434\u043e \u0441\u0438\u0445 \u043f\u043e\u0440 \u043d\u0435 \u0432\u044b\u043b\u043e\u0436\u0438\u043b\u0430 \u0438\u0441\u0445\u043e\u0434\u043d\u044b\u0439 \u043a\u043e\u0434 Quark_EDKII \u043d\u0430 GitHub, \u043f\u043e\u044d\u0442\u043e\u043c\u0443 pull request&#8217;\u044b \u0434\u043b\u044f \u044d\u0442\u043e\u0433\u043e \u043f\u0440\u043e\u0435\u043a\u0442\u0430 \u044f \u043f\u043e\u043a\u0430 \u043d\u0438\u043a\u043e\u043c\u0443 \u043d\u0435 \u043e\u0442\u043f\u0440\u0430\u0432\u0438\u043b. \u0412\u043e\u0437\u043c\u043e\u0436\u043d\u043e,  <a href=\"http:\/\/habrahabr.ru\/users\/izard\/\" class=\"user_link\">izard<\/a> \u0432 \u043a\u0443\u0440\u0441\u0435, \u043a\u0442\u043e \u0438\u043c\u0435\u043d\u043d\u043e \u0432 Intel \u043e\u0442\u0432\u0435\u0447\u0430\u0435\u0442 \u0437\u0430 \u043f\u0440\u043e\u0435\u043a\u0442 \u0438 \u0432 \u043a\u043e\u0433\u043e \u0441\u043b\u0435\u0434\u0443\u0435\u0442 \u0431\u0440\u043e\u0441\u0438\u0442\u044c \u0441\u0441\u044b\u043b\u043a\u043e\u0439, \u0447\u0442\u043e\u0431\u044b \u0431\u0430\u0433\u0438 \u043a\u043e\u0433\u0434\u0430-\u043d\u0438\u0431\u0443\u0434\u044c \u0438\u0441\u043f\u0440\u0430\u0432\u0438\u043b\u0438.<\/p>\n<p>  \u0421\u043f\u0430\u0441\u0438\u0431\u043e \u0447\u0438\u0442\u0430\u0442\u0435\u043b\u044f\u043c \u0437\u0430 \u0432\u043d\u0438\u043c\u0430\u043d\u0438\u0435, \u0430 \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u0447\u0438\u043a\u0430\u043c PVS-Studio \u2014 \u0437\u0430 \u043f\u043e\u043b\u0435\u0437\u043d\u0443\u044e \u043f\u0440\u043e\u0433\u0440\u0430\u043c\u043c\u0443 \u0438 \u0442\u0435\u0441\u0442\u043e\u0432\u044b\u0439 \u043a\u043b\u044e\u0447.     \t<\/p>\n<div class=\"clear\"><\/div>\n<p> \u0441\u0441\u044b\u043b\u043a\u0430 \u043d\u0430 \u043e\u0440\u0438\u0433\u0438\u043d\u0430\u043b \u0441\u0442\u0430\u0442\u044c\u0438 <a href=\"http:\/\/habrahabr.ru\/post\/258721\/\"> http:\/\/habrahabr.ru\/post\/258721\/<\/a><\/p>\n","protected":false},"excerpt":{"rendered":"<p>     \t<img decoding=\"async\" align=\"left\" src=\"https:\/\/habrastorage.org\/files\/46f\/80b\/7c0\/46f80b7c024a4994bf5dee9057407b7d.png\"\/>\u0420\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u043a\u0430 \u043f\u0440\u043e\u0448\u0438\u0432\u043e\u043a, \u0434\u0430\u0436\u0435 \u0435\u0441\u043b\u0438 \u043e\u043d\u0430 \u0432\u0435\u0434\u0435\u0442\u0441\u044f \u043d\u0435 \u043d\u0430 \u0430\u0441\u0441\u0435\u043c\u0431\u043b\u0435\u0440\u0435 \u0434\u043b\u044f \u044d\u043a\u0437\u043e\u0442\u0438\u0447\u0435\u0441\u043a\u0438\u0445 \u0430\u0440\u0445\u0438\u0442\u0435\u043a\u0442\u0443\u0440, \u0430 \u043d\u0430 C \u0434\u043b\u044f i386\/amd64 \u2014 \u0434\u0435\u043b\u043e \u0432\u0435\u0441\u044c\u043c\u0430 \u043d\u0435\u043f\u0440\u043e\u0441\u0442\u043e\u0435, \u0434\u0430 \u0438 \u0446\u0435\u043d\u0430 \u043e\u0448\u0438\u0431\u043a\u0438 \u043c\u043e\u0436\u0435\u0442 \u0431\u044b\u0442\u044c \u043a\u0440\u0430\u0439\u043d\u0435 \u0432\u044b\u0441\u043e\u043a\u043e\u0439, \u0432\u043f\u043b\u043e\u0442\u044c \u0434\u043e \u0432\u044b\u0445\u043e\u0434\u0430 \u0446\u0435\u043b\u0435\u0432\u043e\u0439 \u0430\u043f\u043f\u0430\u0440\u0430\u0442\u043d\u043e\u0439 \u043f\u043b\u0430\u0442\u0444\u043e\u0440\u043c\u044b \u0438\u0437 \u0441\u0442\u0440\u043e\u044f, \u043f\u043e\u044d\u0442\u043e\u043c\u0443 \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u043d\u0438\u0435 \u0440\u0430\u0437\u043b\u0438\u0447\u043d\u044b\u0445 \u0442\u0435\u0445\u043d\u0438\u043a \u043f\u0440\u0435\u0434\u043e\u0442\u0432\u0440\u0430\u0449\u0435\u043d\u0438\u044f \u043e\u0448\u0438\u0431\u043e\u043a \u043d\u0430 \u0441\u0430\u043c\u044b\u0445 \u0440\u0430\u043d\u043d\u0438\u0445 \u044d\u0442\u0430\u043f\u0430\u0445 \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u043a\u0438 \u2014 \u043d\u0435\u043e\u0431\u0445\u043e\u0434\u0438\u043c\u043e\u0441\u0442\u044c. <br \/>  \u041a \u0441\u043e\u0436\u0430\u043b\u0435\u043d\u0438\u044e, \u043e \u0444\u043e\u0440\u043c\u0430\u043b\u044c\u043d\u043e\u0439 \u0432\u0435\u0440\u0438\u0444\u0438\u043a\u0430\u0446\u0438\u0438 \u0438\u043b\u0438 \u0438\u0441\u043f\u043e\u043b\u044c\u0437\u043e\u0432\u0430\u043d\u0438\u0438 MISRA C \u0432 \u0441\u043b\u0443\u0447\u0430\u0435 UEFI-\u043f\u0440\u043e\u0448\u0438\u0432\u043e\u043a \u043e\u0441\u0442\u0430\u0435\u0442\u0441\u044f \u0442\u043e\u043b\u044c\u043a\u043e \u043c\u0435\u0447\u0442\u0430\u0442\u044c (\u0441 \u0434\u0440\u0443\u0433\u043e\u0439 \u0441\u0442\u0440\u043e\u043d\u044b, \u043c\u0430\u043b\u043e \u043a\u0442\u043e \u0445\u043e\u0447\u0435\u0442 \u0442\u0440\u0430\u0442\u0438\u0442\u044c \u043d\u0430 \u0440\u0430\u0437\u0440\u0430\u0431\u043e\u0442\u043a\u0443 \u043f\u0440\u043e\u0448\u0438\u0432\u043a\u0438 \u043f\u0430\u0440\u0443 \u043b\u0435\u0442 \u0438 50% \u0431\u044e\u0442\u0436\u0435\u0442\u0430 \u043f\u0440\u043e\u0435\u043a\u0442\u0430), \u043f\u043e\u044d\u0442\u043e\u043c\u0443 \u0441\u0435\u0433\u043e\u0434\u043d\u044f \u043f\u043e\u0433\u043e\u0432\u043e\u0440\u0438\u043c \u043e \u0441\u0442\u0430\u0442\u0438\u0447\u0435\u0441\u043a\u043e\u043c \u0430\u043d\u0430\u043b\u0438\u0437\u0435, \u0430 \u0442\u043e\u0447\u043d\u0435\u0435 \u2014 \u043e \u043f\u043e\u043f\u0443\u043b\u044f\u0440\u043d\u043e\u043c \u043d\u0430 \u0425\u0430\u0431\u0440\u0435 \u0441\u0442\u0430\u0442\u0438\u0447\u0435\u0441\u043a\u043e\u043c \u0430\u043d\u0430\u043b\u0438\u0437\u0430\u0442\u043e\u0440\u0435 PVS-Studio, \u043a\u043e\u0442\u043e\u0440\u044b\u043c \u043f\u043e\u043f\u044b\u0442\u0430\u0435\u043c\u0441\u044f \u043d\u0430\u0439\u0442\u0438 \u043e\u0448\u0438\u0431\u043a\u0438 \u0432 \u043e\u0442\u043a\u0440\u044b\u0442\u043e\u043c \u043a\u043e\u0434\u0435 UEFI \u0434\u043b\u044f Intel Galileo. <br \/>  \u0417\u0430 \u0440\u0435\u0437\u0443\u043b\u044c\u0442\u0430\u0442\u0430\u043c\u0438 \u043f\u0440\u043e\u0432\u0435\u0440\u043a\u0438 \u043f\u043e\u043a\u043e\u0440\u043d\u0435\u0439\u0448\u0435 \u043f\u0440\u043e\u0448\u0443 \u043f\u043e\u0434 \u043a\u0430\u0442.  <\/p>\n","protected":false},"author":1,"featured_media":0,"comment_status":"open","ping_status":"open","sticky":false,"template":"","format":"standard","meta":{"footnotes":""},"categories":[],"tags":[],"class_list":["post-257911","post","type-post","status-publish","format-standard","hentry"],"_links":{"self":[{"href":"https:\/\/savepearlharbor.com\/index.php?rest_route=\/wp\/v2\/posts\/257911","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/savepearlharbor.com\/index.php?rest_route=\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/savepearlharbor.com\/index.php?rest_route=\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/savepearlharbor.com\/index.php?rest_route=\/wp\/v2\/users\/1"}],"replies":[{"embeddable":true,"href":"https:\/\/savepearlharbor.com\/index.php?rest_route=%2Fwp%2Fv2%2Fcomments&post=257911"}],"version-history":[{"count":0,"href":"https:\/\/savepearlharbor.com\/index.php?rest_route=\/wp\/v2\/posts\/257911\/revisions"}],"wp:attachment":[{"href":"https:\/\/savepearlharbor.com\/index.php?rest_route=%2Fwp%2Fv2%2Fmedia&parent=257911"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/savepearlharbor.com\/index.php?rest_route=%2Fwp%2Fv2%2Fcategories&post=257911"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/savepearlharbor.com\/index.php?rest_route=%2Fwp%2Fv2%2Ftags&post=257911"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}