Crashes when drag drop object has certain path pattern

[8LWXpg]

Windows Terminal version

1.22.11141.0

Windows build number

10.0.26100.0

Other Software

Wrote a basic Winforms app that performs file/folder drag drop

private void DragSource_MouseMove(object sender, MouseEventArgs e)
{
	if (e.Button != MouseButtons.Left)
	{
		return;
	}

	var dataObject = new DataObject(DataFormats.FileDrop, _path);
	DragDropEffects result = DoDragDrop(dataObject, Globals.ArgParser.Move ? DragDropEffects.Move : DragDropEffects.Copy);
	if (result is DragDropEffects.Move or DragDropEffects.Copy)
	{
		Application.Exit();
	}
}

https://github.com/8LWXpg/dwag/blob/02f21dca322850c3fb2786f0e07ed2c630dcab4f/dwag/DrageSourceForm.cs#L65-L78

Steps to reproduce

Certain path pattern cause wt to crash

• Forward slash: E:/test.txt
• Relative path: E:\test\..\test.txt, ., .\test.txt

2025-06-08.154321_h264.mp4

Expected Behavior

Handles path properly.

Actual Behavior

Crashes

[DHowett]

Well, that's not good.

[DHowett]

I first thought that it would be a problem in our path translation code, but you're pasting into an untranslated terminal control... so it's probably in the general drop handler here:

terminal/src/cascadia/TerminalControl/TermControl.cpp

Lines 3174 to 3257 in 557a193

	else if (e.DataView().Contains(StandardDataFormats::StorageItems()))
	{
	Windows::Foundation::Collections::IVectorView<Windows::Storage::IStorageItem> items;
	try
	{
	items = co_await e.DataView().GetStorageItemsAsync();
	}
	CATCH_LOG();
	if (items.Size() > 0)
	{
	std::vector<std::wstring> fullPaths;
	// GH#14628: Workaround for GetStorageItemsAsync() only returning 16 items
	// at most when dragging and dropping from archives (zip, 7z, rar, etc.)
	if (items.Size() == 16 && e.DataView().Contains(winrt::hstring{ L"FileDrop" }))
	{
	auto fileDropData = co_await e.DataView().GetDataAsync(winrt::hstring{ L"FileDrop" });
	if (fileDropData != nullptr)
	{
	auto stream = fileDropData.as<IRandomAccessStream>();
	stream.Seek(0);
	const uint32_t streamSize = gsl::narrow_cast<uint32_t>(stream.Size());
	const Buffer buf(streamSize);
	const auto buffer = co_await stream.ReadAsync(buf, streamSize, InputStreamOptions::None);
	const HGLOBAL hGlobal = buffer.data();
	const auto count = DragQueryFileW(static_cast<HDROP>(hGlobal), 0xFFFFFFFF, nullptr, 0);
	fullPaths.reserve(count);
	for (unsigned int i = 0; i < count; i++)
	{
	std::wstring path;
	path.resize(wil::max_path_length);
	const auto charsCopied = DragQueryFileW(static_cast<HDROP>(hGlobal), i, path.data(), wil::max_path_length);
	if (charsCopied > 0)
	{
	path.resize(charsCopied);
	fullPaths.emplace_back(std::move(path));
	}
	}
	}
	}
	else
	{
	fullPaths.reserve(items.Size());
	for (const auto& item : items)
	{
	fullPaths.emplace_back(item.Path());
	}
	}
	std::wstring allPathsString;
	for (auto& fullPath : fullPaths)
	{
	// Join the paths with spaces
	if (!allPathsString.empty())
	{
	allPathsString += L" ";
	}
	const auto translationStyle{ _core.Settings().PathTranslationStyle() };
	_translatePathInPlace(fullPath, translationStyle);
	// All translated paths get quotes, and all strings spaces get quotes; all translated paths get single quotes
	const auto quotesNeeded = translationStyle != PathTranslationStyle::None || fullPath.find(L' ') != std::wstring::npos;
	const auto quotesChar = translationStyle != PathTranslationStyle::None ? L'\'' : L'"';
	// Append fullPath and also wrap it in quotes if needed
	if (quotesNeeded)
	{
	allPathsString.push_back(quotesChar);
	}
	allPathsString.append(fullPath);
	if (quotesNeeded)
	{
	allPathsString.push_back(quotesChar);
	}
	}
	_pasteTextWithBroadcast(winrt::hstring{ allPathsString });
	}

[jamespack]

What is the FullName ?. Powershell wouldn't give the RelativePath for FullName. It would give the fully rooted path. So im betting there is something in the path that Terminal doesnt like. Any change you could comment that? @8LWXpg

[jamespack]

Nevermind. It is the relative path. My bad.