Re: SHELL32: minor fix in debughlp.c (crash in file dialogs)

[Mike McCormack]

This following patch breaks the file open dialog, so if your file open 
dialog box crashes in the CVS, this is the thing to revert.

_dbg_ILGetNext doesn't seem to return NULL if pidl->mkid.cb is zero...

Mike

Filip Navara wrote:
> _dbg_ILGetNext returns NULL if pidl->mkid.cb is zero, fix the
> loops accordingly.
>
> --- dlls/shell32/debughlp.c
> +++ dlls/shell32/debughlp.c
> @@ -215,7 +215,7 @@
>      pidltemp = _dbg_ILGetNext(pidltemp);
>
> -
>    } while (pidltemp->mkid.cb);
> -
>    } while (pidltemp);
> }
>    else
>    {
>
> @@ -275,7 +275,7 @@
>          }
>        }
>        pidltemp = _dbg_ILGetNext(pidltemp);
> -
>  } while (pidltemp->mkid.cb);
> -
>  } while (pidltemp);
> }
>  return ret;
>
> }