[PATCH try2 1/2] wininet: Track child handles, free all child handles on WININET_FreeHandle as native.

Misha Koshelev mk144210 at bcm.edu
Thu Sep 20 20:59:40 CDT 2007


Fixes bug #9479 while keeping out all the casts that I brought in my first version
(see Jacek's comments here:
http://www.winehq.org/pipermail/wine-devel/2007-September/059124.html )

As a reminder, this bug depends on freeing a wininet handle also freeing child handles as
occurs on native.

I believe doing hte list_init( &info->children ) in WININET_AllocHandle makes sense as prior to
a handle being created there can't be any children. In theory, all the list_add_head's can be
placed into a separate function (say a WININET_AddChild that does the WININET_AddRef and list_add_head),
but I didn't think this was worthwhile for a two-line function.
---
 dlls/wininet/cookie.c   |    2 --
 dlls/wininet/ftp.c      |    3 +++
 dlls/wininet/http.c     |    2 ++
 dlls/wininet/internet.c |   15 ++++++++++++++-
 dlls/wininet/internet.h |    3 +++
 dlls/wininet/urlcache.c |    1 -
 6 files changed, 22 insertions(+), 4 deletions(-)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 89f5a7e50dc26f330f7e43b9c8e938cbbd394157.diff
Type: text/x-patch
Size: 4676 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-patches/attachments/20070920/ffd4e1e5/attachment.bin 


More information about the wine-patches mailing list