Uploaded image for project: 'Lustre'
  1. Lustre
  2. LU-10811

New static analysis issues in v2_10_59_0-0-g108333a

Details

    • Bug
    • Resolution: Not a Bug
    • Major
    • None
    • None
    • 3
    • 9223372036854775807

    Description

      Found 1 new static analysis issues in v2_10_59_0-0-g108333a:

      1. Null pointer may be passed to function that may dereference it
        • lustre/utils/lfs.c: in lfs_quota, Null pointer 'name' that comes from line 6018 may be passed to function and can be dereferenced there by passing argument 2 to function 'get_print_quota' at line 6156.

      TRACEBACK:
      An event which alters the program's state, leading to the defect
      lfs.c:6018: 'name' has been assigned a NULL value.
      A condition which occurs, leading to the defect
      lfs.c:6028: Entering loop, because (c=getopt(argc, argv, "gi:I:o:pqtuvh" ) ) != -1 is true
      An event which alters the program's state, leading to the defect
      lfs.c:6032: Goto statement causes loop exit.
      An event which alters the program's state, leading to the defect
      lfs.c:6035: Goto statement causes loop exit.
      A condition which occurs, leading to the defect
      lfs.c:6117: qctl.qc_cmd==8388615 is false
      An event which alters the program's state, leading to the defect
      lfs.c:6156: 'name' is dereferenced by passing argument 2 to function 'get_print_quota'.
      An event which alters the program's state, leading to the defect
      lfs.c:5794: 'name' is passed to function 'get_print_quota'.
      An event which alters the program's state, leading to the defect
      lfs.c:5823: 'name' is dereferenced by passing argument 1 to function 'print_quota_title'.
      An event which alters the program's state, leading to the defect
      lfs.c:5615: 'name' is passed to function 'print_quota_title'.
      An event which alters the program's state, leading to the defect
      lfs.c:5619: 'name' is dereferenced by formatted printing function 'printf'.

      The list of commits since the previous build v2_10_58_0-135-g9ba637b:

      108333a New tag 2.10.59
      630cd49 LU-10794 lfs: make quota work for grace time
      2a5f8ce LU-10783 kernel: kernel update RHEL7.4 [3.10.0-693.21.1.el7]
      e8bafb1 LU-10750 mdd: declare changelogs only when enabled
      b231e3a LU-10465 lov: decrease default stripe size to 1MB

      Attachments

        Issue Links

          Activity

            [LU-10811] New static analysis issues in v2_10_59_0-0-g108333a
            adilger Andreas Dilger made changes -
            Link New: This issue is related to LU-4629 [ LU-4629 ]
            adilger Andreas Dilger made changes -
            Resolution New: Not a Bug [ 6 ]
            Status Original: Open [ 1 ] New: Resolved [ 5 ]
            dmiter Dmitry Eremin (Inactive) made changes -
            Description Original: Found 1 new static analysis issues in v2_10_59_0-0-g108333a:
            # *Null pointer may be passed to function that may dereference it*
            #* {{lustre/utils/lfs.c}}: in _lfs_quota_, Null pointer 'name' that comes from line 6018 may be passed to function and can be dereferenced there by passing argument 2 to function 'get\_print\_quota' at line 6156.

            The list of commits since the previous build v2_10_58_0-135-g9ba637b:
            |[108333a|http://git.whamcloud.com/?p=fs/lustre-release.git;a=commit;h=108333aa389b9675588a825704f6bce7d7021786]|New tag 2.10.59|
            |[630cd49|http://git.whamcloud.com/?p=fs/lustre-release.git;a=commit;h=630cd4953daa876697f1aed4eb3819d60d0e2d17]|LU-10794 lfs: make quota work for grace time|
            |[2a5f8ce|http://git.whamcloud.com/?p=fs/lustre-release.git;a=commit;h=2a5f8ce1a4d46143368d9ec16a8db988f6154e68]|LU-10783 kernel: kernel update RHEL7.4 [3.10.0-693.21.1.el7]|
            |[e8bafb1|http://git.whamcloud.com/?p=fs/lustre-release.git;a=commit;h=e8bafb17ed102626adc6ae0806279f95c8cdd2b3]|LU-10750 mdd: declare changelogs only when enabled|
            |[b231e3a|http://git.whamcloud.com/?p=fs/lustre-release.git;a=commit;h=b231e3ae5cf7df6abf1fbd683589f0fcff2de03f]|LU-10465 lov: decrease default stripe size to 1MB|
            New: Found 1 new static analysis issues in v2_10_59_0-0-g108333a:
            # *Null pointer may be passed to function that may dereference it*
            #* {{lustre/utils/lfs.c}}: in _lfs_quota_, Null pointer 'name' that comes from line 6018 may be passed to function and can be dereferenced there by passing argument 2 to function 'get\_print\_quota' at line 6156.

            TRACEBACK:
            An event which alters the program's state, leading to the defect
            lfs.c:6018: 'name' has been assigned a NULL value.
            A condition which occurs, leading to the defect
            lfs.c:6028: Entering loop, because (c=getopt(argc, argv, "gi:I:o:pqtuvh" ) ) != -1 is true
            An event which alters the program's state, leading to the defect
            lfs.c:6032: Goto statement causes loop exit.
            An event which alters the program's state, leading to the defect
            lfs.c:6035: Goto statement causes loop exit.
            A condition which occurs, leading to the defect
            lfs.c:6117: qctl.qc_cmd==8388615 is false
            An event which alters the program's state, leading to the defect
            lfs.c:6156: 'name' is dereferenced by passing argument 2 to function 'get_print_quota'.
            An event which alters the program's state, leading to the defect
            lfs.c:5794: 'name' is passed to function 'get_print_quota'.
            An event which alters the program's state, leading to the defect
            lfs.c:5823: 'name' is dereferenced by passing argument 1 to function 'print_quota_title'.
            An event which alters the program's state, leading to the defect
            lfs.c:5615: 'name' is passed to function 'print_quota_title'.
            An event which alters the program's state, leading to the defect
            lfs.c:5619: 'name' is dereferenced by formatted printing function 'printf'.

            The list of commits since the previous build v2_10_58_0-135-g9ba637b:
            |[108333a|http://git.whamcloud.com/?p=fs/lustre-release.git;a=commit;h=108333aa389b9675588a825704f6bce7d7021786]|New tag 2.10.59|
            |[630cd49|http://git.whamcloud.com/?p=fs/lustre-release.git;a=commit;h=630cd4953daa876697f1aed4eb3819d60d0e2d17]|LU-10794 lfs: make quota work for grace time|
            |[2a5f8ce|http://git.whamcloud.com/?p=fs/lustre-release.git;a=commit;h=2a5f8ce1a4d46143368d9ec16a8db988f6154e68]|LU-10783 kernel: kernel update RHEL7.4 [3.10.0-693.21.1.el7]|
            |[e8bafb1|http://git.whamcloud.com/?p=fs/lustre-release.git;a=commit;h=e8bafb17ed102626adc6ae0806279f95c8cdd2b3]|LU-10750 mdd: declare changelogs only when enabled|
            |[b231e3a|http://git.whamcloud.com/?p=fs/lustre-release.git;a=commit;h=b231e3ae5cf7df6abf1fbd683589f0fcff2de03f]|LU-10465 lov: decrease default stripe size to 1MB|
            dmiter Dmitry Eremin (Inactive) created issue -

            People

              wc-triage WC Triage
              dmiter Dmitry Eremin (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: