Mirrors/PowerToys
1
0
Fork 0
mirror of https://github.com/microsoft/PowerToys.git synced 2026-04-03 01:36:31 +02:00
Code Issues Packages Projects Releases Wiki Activity

[ImageResizer]Fix use without initializing warning (#29236)

Browse Source 
[ImageResizer]Fix use without initializing warn
This commit is contained in:
Jaime Bernardo
2025-03-18 16:24:34 +00:00
committed by GitHub
parent 4377de260f
commit 8862b22c45
2 changed files with 21 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
src/modules/imageresizer/ImageResizerContextMenu/dllmain.cpp
View File

@@ -258,14 +258,20 @@ private:
for (DWORD i = 0; i < fileCount; i++)
{
IShellItem* shellItem;
psiItemArray->GetItemAt(i, &shellItem);
LPWSTR itemName;
// Retrieves the entire file system path of the file from its shell item
shellItem->GetDisplayName(SIGDN_FILESYSPATH, &itemName);
CString fileName(itemName);
fileName.Append(_T("\r\n"));
// Write the file path into the input stream for image resizer
writePipe.Write(fileName, fileName.GetLength() * sizeof(TCHAR));
HRESULT getItemAtResult = psiItemArray->GetItemAt(i, &shellItem);
if (SUCCEEDED(getItemAtResult))
{
LPWSTR itemName;
// Retrieves the entire file system path of the file from its shell item
HRESULT getDisplayResult = shellItem->GetDisplayName(SIGDN_FILESYSPATH, &itemName);
if (SUCCEEDED(getDisplayResult))
{
CString fileName(itemName);
fileName.Append(_T("\r\n"));
// Write the file path into the input stream for image resizer
writePipe.Write(fileName, fileName.GetLength() * sizeof(TCHAR));
}
}
}
writePipe.Close();
}

8
src/modules/imageresizer/dll/ContextMenuHandler.cpp
View File

@@ -392,8 +392,14 @@ HRESULT __stdcall CContextMenuHandler::GetState(IShellItemArray* psiItemArray, B
PERCEIVED type;
PERCEIVEDFLAG flag;
IShellItem* shellItem;
//Check extension of first item in the list (the item which is right-clicked on)
psiItemArray->GetItemAt(0, &shellItem);
HRESULT getItemAtResult = psiItemArray->GetItemAt(0, &shellItem);
if(!SUCCEEDED(getItemAtResult)) {
// Avoid crashes in the following code.
return E_FAIL;
}
LPTSTR pszPath;
// Retrieves the entire file system path of the file from its shell item
HRESULT getDisplayResult = shellItem->GetDisplayName(SIGDN_FILESYSPATH, &pszPath);