Re: Problem with cookies
Status: Alpha
Brought to you by:
coroberti
From: web l. a. p. t. t. <cur...@li...> - 2007-07-26 05:26:21
|
Robert, Just because it may work once or twice doesn't mean it will always work, the function just may use the exactly same stack space in memory once or twice. The bottom line: it wasn't correct, one should never point to local function variables after that function returns. It is correct now though with static declaration of that string array. Also, it would be nice to document the precedence of separator characters used in this code. Currently it's not clear from documentation. Artur On 7/25/07, web loading and performance testing tool < cur...@li...> wrote: > > Hi Artur, > > On 7/25/07, web loading and performance testing tool > > 1) You are welcome, and thank you for creating such a usefull tool. My > full > > name is Artur Barashev. > > Thanks file corrected. > > 2) Did you test your fix? :) > > Yes and the test was corrected as well. > > > It looks that the way you fixed it isn't > > correct, you are saving the separator > > pointer by pointing it to locally allocated string (allocated on the > > function stack). > > No, this is correct. You can test. > Still, added the word static (only for clarity). > > Thanks. It looks like we need to release a new stable version. > > Sincerely, > Robert Iakobashvili, > coroberti %x40 gmail %x2e com > ........................................................... > http://curl-loader.sourceforge.net > A web testing and traffic generation tool. > > ------------------------------------------------------------------------- > This SF.net email is sponsored by: Splunk Inc. > Still grepping through log files to find problems? Stop. > Now Search log events and configuration files using AJAX and a browser. > Download your FREE copy of Splunk now >> http://get.splunk.com/ > _______________________________________________ > curl-loader-devel mailing list > cur...@li... > https://lists.sourceforge.net/lists/listinfo/curl-loader-devel > |