Details

    • 7520

    Description

      Thanks to the Coverity tool, we found some control flow issues in the Lustre code. This is typically the case of a missing break statement in a switch case, or when the tool found some structurally dead code.

      I will propose a patch to address the issues.

      Attachments

        Activity

          [LU-3097] Fix 'control flow' issues

          I looks all patches were landed. So, close the ticket.

          dmiter Dmitry Eremin (Inactive) added a comment - I looks all patches were landed. So, close the ticket.

          Hi,

          Following Oleg's comment in http://review.whamcloud.com/5933, I have split the patch into four parts, and abandoned the initial one.

          The new patches are at:
          http://review.whamcloud.com/7164
          http://review.whamcloud.com/7165
          http://review.whamcloud.com/7166
          http://review.whamcloud.com/7167

          Thanks,
          Sebastien.

          sebastien.buisson Sebastien Buisson (Inactive) added a comment - Hi, Following Oleg's comment in http://review.whamcloud.com/5933 , I have split the patch into four parts, and abandoned the initial one. The new patches are at: http://review.whamcloud.com/7164 http://review.whamcloud.com/7165 http://review.whamcloud.com/7166 http://review.whamcloud.com/7167 Thanks, Sebastien.
          pjones Peter Jones added a comment -

          Dmitry

          Could you please take care of this one?

          Thanks

          Peter

          pjones Peter Jones added a comment - Dmitry Could you please take care of this one? Thanks Peter

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

          Thanks,
          Sebastien.

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

          People

            dmiter Dmitry Eremin (Inactive)
            sebastien.buisson Sebastien Buisson (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: