[PATCH] comdlg32: Avoid passing NULL to FILEDLG95_LOOKIN_InsertItemAfterParent.

[Lei Zhang]

Hi,

This is my second attempt to fix bug 16332. We can make sure
FILEDLG95_LOOKIN_InsertItemAfterParent() doesn't infinite loop by
always giving it a non-null pidl. The Desktop folder seems like a
sensible default.
-------------- next part --------------
From a6c3c0549d6f35da8fd7f2a7ed0c17a17390e695 Mon Sep 17 00:00:00 2001
From: Lei Zhang <thestig at google.com>
Date: Thu, 4 Dec 2008 14:52:18 -0800
Subject: [PATCH] comdlg32: Avoid passing NULL to FILEDLG95_LOOKIN_InsertItemAfterParent.

---
 dlls/comdlg32/filedlg.c |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/dlls/comdlg32/filedlg.c b/dlls/comdlg32/filedlg.c
index e1eedd1..0ea5284 100644
--- a/dlls/comdlg32/filedlg.c
+++ b/dlls/comdlg32/filedlg.c
@@ -2920,6 +2920,9 @@ int FILEDLG95_LOOKIN_SelectItem(HWND hwnd,LPITEMIDLIST pidl)
 
   TRACE("\n");
 
+  if (!pidl)
+      SHGetSpecialFolderLocation(0,CSIDL_DESKTOP,&pidl);
+
   iItemPos = FILEDLG95_LOOKIN_SearchItem(hwnd,(WPARAM)pidl,SEARCH_PIDL);
 
   liInfos = GetPropA(hwnd,LookInInfosStr);
-- 
1.5.4.5