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

Fix 'NULL pointer dereference' errors

Details

    • Technical task
    • Resolution: Duplicate
    • Minor
    • None
    • Lustre 2.4.0
    • 6681

    Description

      Here is a new defect that aims at addressing Null pointer dereference errors found by Coverity in the Lustre code.

      I will propose a patch to address the issues.

      Attachments

        Activity

          [LU-2751] Fix 'NULL pointer dereference' errors
          pjones Peter Jones added a comment -

          ok thanks Sebastien!

          pjones Peter Jones added a comment - ok thanks Sebastien!
          sebastien.buisson Sebastien Buisson (Inactive) added a comment -

          Hi,

          After reviews, I removed unnecessary assertions from the patch. As the resulting patch is a lot smaller, I integrated all the modifications into the patch LU-2217, pushed here:
          http://review.whamcloud.com/4720

          This ticket can be closed.

          Sebastien.

          sebastien.buisson Sebastien Buisson (Inactive) added a comment - Hi, After reviews, I removed unnecessary assertions from the patch. As the resulting patch is a lot smaller, I integrated all the modifications into the patch LU-2217 , pushed here: http://review.whamcloud.com/4720 This ticket can be closed. Sebastien.
          pjones Peter Jones added a comment -

          Thanks Sebastien!

          Keith

          Can you please look into this one?

          Thanks

          Peter

          pjones Peter Jones added a comment - Thanks Sebastien! Keith Can you please look into this one? Thanks Peter
          sebastien.buisson Sebastien Buisson (Inactive) added a comment -

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

          Thanks,
          Sebastien.

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

          People

            keith Keith Mannthey (Inactive)
            sebastien.buisson Sebastien Buisson (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: