SERVER-31516 Fix improper initialization of empty NamespaceString.

(cherry picked from commit e7390933e152d8e6f00c90ac341f691780c261fd)

2 files changed, 13 insertions, 1 deletions

diff --git a/src/mongo/db/namespace_string-inl.h b/src/mongo/db/namespace_string-inl.h
index 0ff45fca39e..7ad830f2481 100644
--- a/src/mongo/db/namespace_string-inl.h
+++ b/src/mongo/db/namespace_string-inl.h
@@ -118,7 +118,7 @@ inline bool NamespaceString::validCollectionName(StringData coll) {
     return true;
 }
 
-inline NamespaceString::NamespaceString() : _ns(), _dotIndex(0) {}
+inline NamespaceString::NamespaceString() : _ns(), _dotIndex(std::string::npos) {}
 inline NamespaceString::NamespaceString(StringData nsIn) {
     _ns = nsIn.toString();  // copy to our buffer
     _dotIndex = _ns.find('.');
diff --git a/src/mongo/db/namespace_string_test.cpp b/src/mongo/db/namespace_string_test.cpp
index 69884bd5ca9..69742c797bf 100644
--- a/src/mongo/db/namespace_string_test.cpp
+++ b/src/mongo/db/namespace_string_test.cpp
@@ -272,4 +272,16 @@ TEST(NamespaceStringTest, makeListIndexesNSIsCorrect) {
     ASSERT(ns.isListIndexesCursorNS());
     ASSERT_EQUALS(NamespaceString("DB.COLL"), ns.getTargetNSForListIndexes());
 }
+
+TEST(NamespaceStringTest, EmptyNSStringReturnsEmptyColl) {
+    NamespaceString nss{};
+    ASSERT_TRUE(nss.toString().empty());
+    ASSERT_EQ(nss.coll(), StringData{});
+}
+
+TEST(NamespaceStringTest, EmptyNSStringReturnsEmptyDb) {
+    NamespaceString nss{};
+    ASSERT_TRUE(nss.toString().empty());
+    ASSERT_EQ(nss.db(), StringData{});
+}
 }