Uploaded image for project: 'Lustre'
  1. Lustre
  2. LU-2753 Tracking bug for static code analysis fixes.
  3. LU-4048

Fix 'control flow' issues

Details

    • 10861

    Description

      Thanks to the Coverity tool, we found some control flow issues in the Lustre code. This is typically the case of structurally dead code.

      I will propose a patch to address the issues.

      Attachments

        Activity

          [LU-4048] Fix 'control flow' issues
          emoly.liu Emoly Liu added a comment -

          Landed for 2.9.0.

          emoly.liu Emoly Liu added a comment - Landed for 2.9.0.
          gerrit Gerrit Updater added a comment -

          Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/7824/
          Subject: LU-4048 build: fix 'control flow' errors
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: 5295e62a00e5d3a81171d424d4fed16e86cd4739

          gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/7824/ Subject: LU-4048 build: fix 'control flow' errors Project: fs/lustre-release Branch: master Current Patch Set: Commit: 5295e62a00e5d3a81171d424d4fed16e86cd4739
          sebastien.buisson Sebastien Buisson (Inactive) added a comment -

          Hi,

          Sorry for the delay, I just pushed a new version of the patch following Jinshan's comment.

          Cheers,
          Sebastien.

          sebastien.buisson Sebastien Buisson (Inactive) added a comment - Hi, Sorry for the delay, I just pushed a new version of the patch following Jinshan's comment. Cheers, Sebastien.
          pjones Peter Jones added a comment -

          Emoly

          Could you please look after this patch?

          Thanks

          Peter

          pjones Peter Jones added a comment - Emoly Could you please look after this patch? Thanks Peter
          sebastien.buisson Sebastien Buisson (Inactive) added a comment -

          The patch is at:
          http://review.whamcloud.com/7824

          Thanks,
          Sebastien.

          sebastien.buisson Sebastien Buisson (Inactive) added a comment - The patch is at: http://review.whamcloud.com/7824 Thanks, Sebastien.

          People

            emoly.liu Emoly Liu
            sebastien.buisson Sebastien Buisson (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: