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

filename_is_volatile() uses strtoul(..., ..., 0) to parse MDT index

Details

    • Bug
    • Resolution: Fixed
    • Major
    • Lustre 2.6.0
    • Lustre 2.5.0
    • 3
    • 10033

    Description

      If llapi_create_volatile_idx() is passed an MDT index other than -1 then it uses '%.4X' to encode the index into the volatile file name. But it uses an index of 0 rather than the one provided.

      filename_is_volatile() uses strtoul(..., ..., 0) to parse the MDT index out of the volatile file name.

      So llapi_create_volatile_idx() ignores MDT index when specified and filename_is_volatile() doesn't properly understand MDT indices greater than 9.

      Attachments

        Activity

          [LU-3867] filename_is_volatile() uses strtoul(..., ..., 0) to parse MDT index

          Landed to master.

          jamesanunez James Nunez (Inactive) added a comment - Landed to master.
          pjones Peter Jones added a comment -

          James

          Bobbie has a fix for this in gerrit already. Could you oversee that it completes testing and reviews.

          Thanks

          Peter

          pjones Peter Jones added a comment - James Bobbie has a fix for this in gerrit already. Could you oversee that it completes testing and reviews. Thanks Peter
          bobbielind Bobbie Lind (Inactive) added a comment - Patch submitted here http://review.whamcloud.com/#/c/7613/

          People

            jamesanunez James Nunez (Inactive)
            jhammond John Hammond
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: