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

'read on open' breaks GSS integrity check

Details

    • Bug
    • Resolution: Fixed
    • Critical
    • Lustre 2.12.0
    • Lustre 2.12.0
    • 3
    • 9223372036854775807

    Description

      Feature 'read on open for DoM files' breaks GSS integrity check.

      Indeed, when ski or krb5i flavors are selected, GSS integrity mechanism signs requests on client side before they are sent, and then checks signature on server side upon receiving.
      So it is not possible to alter the request content once signature is calculated.

      However, with the patch implementing 'read on open for DoM files' (https://review.whamcloud.com/23011), the value of request->rq_reqmsg->lm_repsize is changed after the request has been wrapped. And this field is included in request->rq_reqbuf on which the signature is calculated.
      So the signature calculated on the received request on server side does not match the signature calculated on client side.

      Consequence is that it is not possible to use Kerberos or Shared Key with integrity protection flavors (and possible full encryption flavors as well).

      Attachments

        Activity

          [LU-11414] 'read on open' breaks GSS integrity check
          pjones Peter Jones made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: In Progress [ 3 ] New: Resolved [ 5 ]
          pjones Peter Jones made changes -
          Comment [ A comment with security level 'Employeees' was removed. ]
          pjones Peter Jones made changes -
          Comment [ A comment with security level 'Employeees' was removed. ]
          tappro Mikhail Pershin made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          jgmitter Joseph Gmitter (Inactive) made changes -
          Fix Version/s New: Lustre 2.12.0 [ 13495 ]
          jgmitter Joseph Gmitter (Inactive) made changes -
          Assignee Original: WC Triage [ wc-triage ] New: Mikhail Pershin [ tappro ]
          jgmitter Joseph Gmitter (Inactive) made changes -
          Labels Original: gss New: DoM2 gss
          sebastien Sebastien Buisson created issue -

          People

            tappro Mikhail Pershin
            sebastien Sebastien Buisson
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: