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

Fix 'error handling' issues

Details

    • Technical task
    • Resolution: Won't Fix
    • Minor
    • None
    • Lustre 2.4.0
    • 7552

    Description

      Following LU-2444, Coverity found a few more error handling issues in the Lustre code.

      I will propose a patch to address the issues.

      Attachments

        Activity

          [LU-3108] Fix 'error handling' issues
          pjones Peter Jones added a comment -

          Thanks Sebastien

          pjones Peter Jones added a comment - Thanks Sebastien
          sebastien.buisson Sebastien Buisson (Inactive) added a comment -

          Ok, given the comments in Gerrit, these issues should have been addressed by other code modifications.
          I will rerun Coverity and open a new Jira ticket if needed.

          Thanks,
          Sebastien.

          sebastien.buisson Sebastien Buisson (Inactive) added a comment - Ok, given the comments in Gerrit, these issues should have been addressed by other code modifications. I will rerun Coverity and open a new Jira ticket if needed. Thanks, Sebastien.
          pjones Peter Jones added a comment -

          Sebastien

          Although I see that you did not get an answer about the ticket tracking the reworking of the affected code, we are almost a year on now so is this ticket still relevant?

          Peter

          pjones Peter Jones added a comment - Sebastien Although I see that you did not get an answer about the ticket tracking the reworking of the affected code, we are almost a year on now so is this ticket still relevant? Peter
          pjones Peter Jones added a comment -

          Keith

          Could you please take care of this one?

          Thanks

          Peter

          pjones Peter Jones added a comment - Keith Could you please take care of this one? Thanks Peter
          sebastien.buisson Sebastien Buisson (Inactive) added a comment -

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

          Thanks,
          Sebastien.

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

          People

            pjones Peter Jones
            sebastien.buisson Sebastien Buisson (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: