Oops I shouldn't have erased names directly.

We always need to remove the index map when removing a name.

sc/inc/rangenam.hxx

@@ -230,6 +230,7 @@ public:
     SC_DLLPUBLIC bool insert(ScRangeData* p);
 
     void erase(const ScRangeData& r);
+    void erase(const rtl::OUString& rName);
 
     /**
      * Erase by iterator position.  Note that this method doesn't check for

sc/source/core/tool/rangenam.cxx

@@ -894,7 +894,7 @@ bool ScRangeName::insert(ScRangeData* p)
     }
 
     rtl::OUString aName(p->GetUpperName());
-    maData.erase(aName); // ptr_map won't insert it if a duplicate name exists.
+    erase(aName); // ptr_map won't insert it if a duplicate name exists.
     pair<DataType::iterator, bool> r = maData.insert(aName, p);
     if (r.second)
     {
@@ -909,7 +909,12 @@ bool ScRangeName::insert(ScRangeData* p)
 
 void ScRangeName::erase(const ScRangeData& r)
 {
-    DataType::iterator itr = maData.find(r.GetUpperName());
+    erase(r.GetUpperName());
+}
+
+void ScRangeName::erase(const rtl::OUString& rName)
+{
+    DataType::iterator itr = maData.find(rName);
     if (itr != maData.end())
         erase(itr);
 }