[Tcl-bugs] [ tcl-Bugs-513385 ] error in timeout destroys token

[<no...@so...>]

Bugs item #513385, was opened at 2002-02-05 12:36
You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=110894&aid=513385&group_id=10894

Category: 28. http Package
Group: 8.4a4
Status: Open
Resolution: None
Priority: 5
Submitted By: Don Porter (dgp)
Assigned to: Jeffrey Hobbs (hobbs)
Summary: error in timeout destroys token

Initial Comment:

The timeout option for http::geturl is
implemented by a timer event that calls

	http::reset $token timeout

If there's an error returned by the command
specified by the -command option, then
http::reset will unset the state array and
raise an error that will be processed as
a background error.

All this processing takes place in an event
loop, probably within [http::wait $token].
Given the above, it appears that callers
of [http::wait $token] need to be able to
handle the non-existence of $token upon
its return.  Also, [http::wait] itself needs to
be written to deal with the non-existence
of $token when its call to [vwait] returns.

Something needs fixing around lines 358 and 840
of the current http.tcl.


----------------------------------------------------------------------

>Comment By: Don Porter (dgp)
Date: 2002-02-27 23:56

Message:
Logged In: YES 
user_id=80530

Would be good to fix this for the release of http 2.4.2
with Tcl 8.4a4.  Is there an historical reason for the
unset of the state array in [reset] ?  It seems to be
the cause of the trouble.  Wouldn't we rather keep the
state array around with a status of "error"?

The log indicates that "sandeep" added the unset to
[reset] in Revision 1.28 of http.tcl to "fix two
potential memory leaks".  Anyone have more detailed
recollection about this?

----------------------------------------------------------------------

You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=110894&aid=513385&group_id=10894