Daniel Stenberg wrote: >> I just fixed both problems I've reported recently: > You rock Patrick, awesome!
Thank you but... well, this particular dangling pointer problem is resolved, however I think there's potentially a risk of having more troubles in the future: If you reuse a connection, conn->dns_entry is null (so conn->ip_addr may be dangling). I've tried to nullify conn->ip_addr where conn->dns_entry in cleared, but it reappears later as non-null in ftp_state_pasv_resp() while conn->dns_entry is still NULL :-( I suppose it is set from (dangling) tempaddr[i] or from the FTP data channel connection... I do not really master this part of the library, but I think there is some uncontrolled mess here. Maybe a resolver+connect+happy_eyeball+proxy guru can audit this. He'll probably be much more efficient than me. Cheers, Patrick ------------------------------------------------------------------- List admin: http://cool.haxx.se/list/listinfo/curl-library Etiquette: http://curl.haxx.se/mail/etiquette.html
