Commands which accept UUIDs do not correctly validate that the collection belongs to the database over which the command was issued

XMLWordPrintableJSON

    • Type: Bug
    • Resolution: Fixed
    • Priority: Major - P3
    • 3.7.3
    • Affects Version/s: 3.7.2
    • Component/s: Catalog
    • None
    • Fully Compatible
    • ALL
    • Query 2018-03-12
    • 0
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None

      Various commands accept UUIDs to uniquely identify a collection, instead of a a collection name. However, the RPC protocol for issuing commands also requires clients to include a particular database name. If the UUID identifies a collection within a different database from the one named in the command request, the intended behavior is that the command should fail.

      This behavior was correctly implemented in 3.6, but appears to have regressed during 3.7 development. I believe the regression was caused by SERVER-32367, due to commit e5e8dde676.

      This appears to affect several commands, so I think we should audit and test all of the commands that accept UUIDs as part of the fix. Here's a repro for the find command in particular:

      (function() {
          "use strict";
      
          assert.writeOK(db.c.insert({}));
      
          let collectionInfos = db.getCollectionInfos({});
          let uuid = collectionInfos[0].info.uuid;
      
          let otherDb = db.getSisterDB("other");
          assert.commandFailed(otherDb.runCommand({find: uuid}));
      }());
      

            Assignee:
            David Storch
            Reporter:
            David Storch
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: