Может ли присвоение shared_ptr мусора указателем `this`

Давайте возьмем следующий пример структуры данных (Node), которая представляет дерево дочерних узлов. Набор дочерних узлов для каждого объекта хранится в карте>

class Node;
typedef std::shared_ptr<Node> NodePtr;

class Node
{
std::map<const std::string, NodePtr> _childNodes;
void SomeOtherMethod();

public:
bool GetChildByKeyName(/*In*/ const std::string& key, /*Out*/ NodePtr& spChild)
{
bool result = false;
auto itor = _childNodes.find(key);
if (itor != _childNodes.end())
{
spChild = itor->second;
result = true;
SomeOtherMethod();
}
return result;
}
};

И следующий пример кода, как обычно называют его пользователи.

NodePtr spNode, spChildNode;
bool result;
...
result = spNode->GetChildByKeyName(strChildKeyName, spChildNode);

Все идет нормально.

Мне пришло в голову, что вызывающие стороны могут проходить по дереву без необходимости иметь дело с дополнительными переменными для каждой глубины в дереве.

NodePtr spNode;
bool result;

result = spNode->GetChildItem(strChildKeyName, spNode);
if (result)
spNode->GetChildItem(strSubKeyName, spNode);

В приведенном выше случае, если spNode является окончательный оставшаяся ссылка на объект, то меня беспокоит этот блок кода в методе GetChildItem:

            spChild = itor->second;
result = true;
SomeOtherMethod();

Не нарушает ли назначение spChild (который на самом деле является экземпляром spNode вызывающего) случайный «этот» узел, так как последняя ссылка только что исчезла? (И, следовательно, вызывать другие методы после присваивания spChild опасно). У меня есть потенциальная ошибка здесь?

Я думаю, что решение состоит в том, чтобы просто добавить эту строку в верхней части вызова метода:

NodePtr spChildRef = spChild; // maintain a reference to the caller's original node during the scope of the method

Мысли?