Hi Jacek,

On Mon, Sep 5, 2011 at 3:33 AM, Jacek Caban <ja...@codeweavers.com> wrote:

> AFAICS secur_url wasn't allocated with CoTaskMemAlloc until this patch. Why
> are you changing it?

map_url_to_zone gets "secur_url" allocated in one of two ways. The
first way is through CoInternetGetSecurityUrl, which allocates the
resulting URL with CoTaskMemAlloc. The second way is if
CoInternetGetSecurityUrl fails, in which case it was allocating with
HeapAlloc. Since we can't change how CoInternetGetSecurityUrl
allocates memory, I switched everything to use the CoTaskMem*
functions, so that we don't mix CoTaskMemAlloc functions with HeapFree
functions.

-- 
Thomas Mullaly
thomas.mull...@gmail.com


Reply via email to