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

Review Axivion Scan results for QtNetwork and fix if needed



    • User Story
    • Resolution: Unresolved
    • P1: Critical
    • None
    • None
    • Network
    • None
    • 21
    • 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.

      Definition of Done

      We will go through all scan results for QtNetwork as part of this task. Since QtCore has progressed somewhat, please be mindful of issues reported there already.

      If issues are deemed fixable, they should be fixed in QtNetwork. 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-125026
          # Subject Branch Project Status CR V



              mate Máté Barany
              ablasche Alex Blasche
              Vladimir Minenko Vladimir Minenko
              Alex Blasche Alex Blasche
              0 Vote for this issue
              2 Start watching this issue



                Gerrit Reviews

                  There are 7 open Gerrit changes