Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-122619

Review Axivion Scan results for QtCore and fix if needed



    • User Story
    • Resolution: Unresolved
    • P1: Critical
    • None
    • None
    • Core: Other
    • None
    • 25
    • Foundation Sprint 105, Foundation Sprint 106, Foundation Sprint 107, Foundation Sprint 108, Foundation Sprint 109, Foundation Sprint 110, Foundation Sprint 111, Foundation Sprint 112
    • 738914e78 (dev), 21ef6d930 (dev), b68f10091 (dev), 35205ef16 (dev), e1da249be (dev), 644eca89e (6.7), 4c25cb76e (6.7), db7b707d8 (6.7), 84d5754a8 (6.7), a07c718c1 (6.7), cf420dc98 (dev), c6a2f7a70 (dev), b9cb2f30f (dev), 4be9bb498 (tqtc/lts-6.5)


      We now have a permanent server that is used to scan Qt products with the Axivion static analyzer: https://axivion-srv.ci.qt.io/ . For Qt 6 modules specifically, we check for the new Qt rule set, as well as clone analysis.

      Definition of Done

      We will go through all scan results for QtCore as part of this task. Checking of other modules will happen later once the rule sets have been confirmed for Qt Core.

      If issues are deemed fixable, they should be fixed in QtCore. False positives should be handled by either marking the code to suppress the failure case or (if the underlying rule) is not applicable to Qt be removed from the scanner setup. Active engagement with the Axivion team is encouraged.

      This task has two purposes. We want to fix any scan results genuinely worth fixing and secondly, we want to verify/fine-tune the rule sets for Qt applicability as those will likely be adjusted to have the most suitable default configuration for Axivion's static analyzer when targeting Qt application developers.

      The Axivion team is very interested in false positives regarding the new Qt specific rules before Annotations are done in code. They would like to fix rules and adjust configurations if possible.


        Issue Links

          For Gerrit Dashboard: QTBUG-122619
          # Subject Branch Project Status CR V



              ablasche Alex Blasche
              ablasche Alex Blasche
              Vladimir Minenko Vladimir Minenko
              Alex Blasche Alex Blasche
              0 Vote for this issue
              4 Start watching this issue



                Gerrit Reviews

                  There are 8 open Gerrit changes