GenericIndexIterator: Fix NameIndex dependency

Next() was still copying the name. Add a GetNodeValue() method to the
policy that fetches the value instead.

src/add-ons/kernel/file_systems/packagefs/IndexImpl.h

@@ -93,10 +93,8 @@ GenericIndexIterator<Policy>::Next(void* buffer, size_t* _keyLength)
 
 	Node* node = _ToNode();
 	if (node != NULL) {
-		if (buffer != NULL) {
+		if (buffer != NULL)
-			strlcpy((char*)buffer, node->Name(), kMaxIndexKeyLength);
+			Policy::GetNodeValue(node, buffer, _keyLength);
-			*_keyLength = strlen(node->Name());
-		}
 
 		fNextTreeNode = Policy::GetNodeTree(fIndex)->Next(fNextTreeNode);
 	}

src/add-ons/kernel/file_systems/packagefs/NameIndex.cpp

@@ -94,6 +94,12 @@ struct NameIndex::IteratorPolicy {
 	{
 		return index->fEntries;
 	}
+
+	static void GetNodeValue(Node* node, void* buffer, size_t* _keyLength)
+	{
+		strlcpy((char*)buffer, node->Name(), kMaxIndexKeyLength);
+		*_keyLength = strlen(node->Name());
+	}
 };