cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: bug in curl_easy_perform

From: Oleg Bartunov <oleg_at_sai.msu.su>
Date: Thu, 23 Aug 2001 17:10:19 +0300 (GMT)

Daniel,

do you think this problem could be related with my 'weird timings' one ?

        Oleg
On Thu, 23 Aug 2001, Daniel Stenberg wrote:

> On Thu, 23 Aug 2001, T. Bharath wrote:
>
> > Now the solution is to reset it at the begginning or the end of
> > Curl_perform
>
> Thanks a lot for digging, reporting and for providing a solution. I would
> however not go with that exact solution...
>
> If we would reset the flag in the beginning of the _perform function, then
> we'd leak the memory that is possibly still allocated and pointed to by the
> data->url pointer.
>
> If we would do it in the end of the function, well then we would risk that
> _perform() is re-invoked with a data->url pointer that is illegal (or NULL).
>
> I think it should be done where data->url is (re-)set and of course in the
> curl_easy_cleanup(). It is already done in the later.
>
> I'll add the fix right now.
>
> > But my question here is are there any other bits,variables that need to
> > be reset
>
> Well, if there are, they too are mistakes and have fallen through my checks.
>
>

        Regards,
                Oleg
_____________________________________________________________
Oleg Bartunov, sci.researcher, hostmaster of AstroNet,
Sternberg Astronomical Institute, Moscow University (Russia)
Internet: oleg@sai.msu.su, http://www.sai.msu.su/~megera/
phone: +007(095)939-16-83, +007(095)939-23-83
Received on 2001-08-23