Patches item #1885530, was opened at 2008-02-02 22:20
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=637566&aid=1885530&group_id=104305

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: sipp
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Kenneth Cox (kenstir)
Assigned to: Nobody/Anonymous (nobody)
Summary: fix crash in pcap thread

Initial Comment:
Sipp 3 and 2 occassionally crash dereferencing a play_args_t structure that has 
been freed by the control thread aborting the call.  There is not currently any 
signalling used to tear down the pcap thread when the call aborts, and so the 
play_args_t is freed out from under the running thread.

This patch avoids the crash by changing the play_args_t struct inside the call 
class to a pointer allocated by malloc().  If the thread is successfully 
started, then  the thread routine (send_wrapper()) is responsible for freeing 
the play_args_t.  If the thread is not successfully started, then the freeing 
is done by the caller.

Problem verified, and fix validated with valgrind on Linux.


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

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=637566&aid=1885530&group_id=104305

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Sipp-users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/sipp-users

Reply via email to