SHChangeNotify[A|W]: free only items we allocated.

260cab42 · Uwe Bonnes · Alexandre Julliard · e2905ea4 · 260cab42
Commit 260cab42 authored Mar 26, 2000 by Uwe Bonnes Committed by Alexandre Julliard Mar 26, 2000
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 4 deletions

changenotify.c dlls/shell32/changenotify.c +4 -4

No files found.
--- a/dlls/shell32/changenotify.c
+++ b/dlls/shell32/changenotify.c
@@ -222,8 +222,8 @@ void WINAPI SHChangeNotifyW (LONG wEventId, UINT  uFlags, LPCVOID dwItem1, LPCVO

 	if(uFlags & SHCNF_PATHA)
 	{
-	  SHFree(pidl1);
-	  SHFree(pidl2);
+            if (pidl1) SHFree(pidl1);
+            if (pidl2) SHFree(pidl2);
 	}
 }

@@ -269,8 +269,8 @@ void WINAPI SHChangeNotifyA (LONG wEventId, UINT  uFlags, LPCVOID dwItem1, LPCVO
 	/* if we allocated it, free it */
 	if(uFlags & SHCNF_PATHA)
 	{
-	  SHFree(Pidls[0]);
-	  SHFree(Pidls[1]);
+            if (Pidls[0]) SHFree(Pidls[0]);
+            if (Pidls[1]) SHFree(Pidls[1]);
 	}
 }