diff options
Diffstat (limited to 'src/mongo/db/update/storage_validation.cpp')
| -rw-r--r-- | src/mongo/db/update/storage_validation.cpp | 33 |
1 files changed, 11 insertions, 22 deletions
diff --git a/src/mongo/db/update/storage_validation.cpp b/src/mongo/db/update/storage_validation.cpp index 69967c14aef..d94c645a588 100644 --- a/src/mongo/db/update/storage_validation.cpp +++ b/src/mongo/db/update/storage_validation.cpp @@ -160,30 +160,19 @@ void scanDocument(const mutablebson::Document& doc, auto currElem = doc.root().leftChild(); while (currElem.ok()) { if (currElem.getFieldName() == idFieldName && shouldValidate) { - if (currElem.getType() == BSONType::Object) { - // We need to recursively validate the _id field while ensuring we disallow - // top-level $-prefix fields in the _id object. - scanDocument(currElem, - true /* deep */, - 0 /* recursionLevel - forces _id fields to be treated as top-level. */, - false /* Top-level _id fields cannot be $-prefixed. */, - shouldValidate, - containsDotsAndDollarsField); - } else { - uassertStatusOK(storageValidIdField(currElem.getValue())); - } - } else { - // Validate this child element. - const auto deep = true; - const uint32_t recursionLevel = 1; - scanDocument(currElem, - deep, - recursionLevel, - allowTopLevelDollarPrefixes, - shouldValidate, - containsDotsAndDollarsField); + uassertStatusOK(storageValidIdField(currElem.getValue())); } + // Validate this child element. + const auto deep = true; + const uint32_t recursionLevel = 1; + scanDocument(currElem, + deep, + recursionLevel, + allowTopLevelDollarPrefixes, + shouldValidate, + containsDotsAndDollarsField); + currElem = currElem.rightSibling(); } } |
