Details

    • Improvement
    • Resolution: Unresolved
    • Minor
    • None
    • None
    • 3
    • 9223372036854775807

    Description

      I've been experimenting with more static analyzers (Coverity in this case). While it does have some false positives, it looks like it found a number of legitimate bugs. See https://scan.coverity.com/projects/lustre for details.

      It's seems like Coverity has been used with Lustre in the past (see: https://wiki.lustre.org/images/8/8a/LUG2013-Lustre_Static_Code_Analysis-Bull.pdf). Although, it doesn't seem like it's been run in a while.

      Also, there's a number of Coverity comments that seem out-dated and should probably just be cleaned up.

      Attachments

        Issue Links

          Activity

            [LU-17000] Coverity static analysis issues
            gerrit Gerrit Updater added a comment -

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/59499/
            Subject: LU-17000 obdclass: Fix mem leak in lcfg_setparam_client
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 20e9f9bd769bcd10033fe3b70cfe257ff899893d

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/59499/ Subject: LU-17000 obdclass: Fix mem leak in lcfg_setparam_client Project: fs/lustre-release Branch: master Current Patch Set: Commit: 20e9f9bd769bcd10033fe3b70cfe257ff899893d
            gerrit Gerrit Updater added a comment -

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/59456/
            Subject: LU-17000 llite: Handle not NUL terminated buffer in ll_statahead_info
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 24a2a4ed6b2120aa21cbaad9496281ff69d1d7c2

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/59456/ Subject: LU-17000 llite: Handle not NUL terminated buffer in ll_statahead_info Project: fs/lustre-release Branch: master Current Patch Set: Commit: 24a2a4ed6b2120aa21cbaad9496281ff69d1d7c2

            "Arshad Hussain <arshad.hussain@aeoncomputing.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/59516
            Subject: LU-17000 utils: For test
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 6e3dc1c3bb9eef4f7dc85b76c69958f8e8e34255

            gerrit Gerrit Updater added a comment - "Arshad Hussain <arshad.hussain@aeoncomputing.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/59516 Subject: LU-17000 utils: For test Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 6e3dc1c3bb9eef4f7dc85b76c69958f8e8e34255

            "Arshad Hussain <arshad.hussain@aeoncomputing.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/59499
            Subject: LU-17000 obdclass: Fix mem leak in lcfg_setparam_client
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 477bebf6053a98a6cc24a1c40893c4625ea8e626

            gerrit Gerrit Updater added a comment - "Arshad Hussain <arshad.hussain@aeoncomputing.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/59499 Subject: LU-17000 obdclass: Fix mem leak in lcfg_setparam_client Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 477bebf6053a98a6cc24a1c40893c4625ea8e626

            "Arshad Hussain <arshad.hussain@aeoncomputing.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/59477
            Subject: LU-17000 obdclass: Fix mem leak in pool_add_objects
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 0b773cb1e85e583843ad2906cacf37aef798b8ba

            gerrit Gerrit Updater added a comment - "Arshad Hussain <arshad.hussain@aeoncomputing.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/59477 Subject: LU-17000 obdclass: Fix mem leak in pool_add_objects Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 0b773cb1e85e583843ad2906cacf37aef798b8ba

            "Arshad Hussain <arshad.hussain@aeoncomputing.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/59456
            Subject: LU-17000 llite: Handle not NUL buffer in ll_statahead_info
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 3088783b720f54ec4c03b18ab91e0908a8e71ac2

            gerrit Gerrit Updater added a comment - "Arshad Hussain <arshad.hussain@aeoncomputing.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/59456 Subject: LU-17000 llite: Handle not NUL buffer in ll_statahead_info Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 3088783b720f54ec4c03b18ab91e0908a8e71ac2

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/55377/
            Subject: LU-17000 llite: Handle not NUL terminated buffer
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 517e76cbefe800251d6d132c7f1d8a0d965d089e

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/55377/ Subject: LU-17000 llite: Handle not NUL terminated buffer Project: fs/lustre-release Branch: master Current Patch Set: Commit: 517e76cbefe800251d6d132c7f1d8a0d965d089e

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/58360/
            Subject: LU-17000 mgs: check lcfg parameters
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 5e4456e9dfeb9ef513ed26eb06ce9251888c991d

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/58360/ Subject: LU-17000 mgs: check lcfg parameters Project: fs/lustre-release Branch: master Current Patch Set: Commit: 5e4456e9dfeb9ef513ed26eb06ce9251888c991d

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/58362/
            Subject: LU-17000 misc: avoid memory leaks in error handling
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: f641b364c200762eb35e78607efb6d4dbbf9906f

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/58362/ Subject: LU-17000 misc: avoid memory leaks in error handling Project: fs/lustre-release Branch: master Current Patch Set: Commit: f641b364c200762eb35e78607efb6d4dbbf9906f

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/58361/
            Subject: LU-17000 llite: fix memory leaks in error handling
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 39a4e71c532b3f0180ef59bc8cb78aa21b85256f

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/58361/ Subject: LU-17000 llite: fix memory leaks in error handling Project: fs/lustre-release Branch: master Current Patch Set: Commit: 39a4e71c532b3f0180ef59bc8cb78aa21b85256f

            People

              timday Tim Day
              timday Tim Day
              Votes:
              0 Vote for this issue
              Watchers:
              8 Start watching this issue

              Dates

                Created:
                Updated: