git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* HTTP push
@ 2005-11-07  2:35 Johannes Schindelin
  2005-11-07  4:31 ` Nick Hengeveld
  0 siblings, 1 reply; 2+ messages in thread
From: Johannes Schindelin @ 2005-11-07  2:35 UTC (permalink / raw)
  To: git

Hi,

I played a little with git-http-push. There are some strange things going 
on: whenever I commented out the calls to curl_multi_remove_handle() and 
curl_multi_cleanup(), all was well. Whenever I did not, there was either a 
segfault, or an endless loop.

Also, I had to update my curl, because http-push *absolutely* needs 
CURLOPT_HTTPGET. I tried to fake it with any combination of 
CURLOPT_{NOBODY,POST,PUT,UPLOAD}, but failed. (Yes, that means that you 
won't have to deal any longer with my patches to support older curl 
versions...)

Note: all my tests are from an iBook (client) to an Intel Linux (server).

I will run some tests on another client with valgrind (that will wait 
until tomorrow; it's 3.30am local time, so I head to bed).

Ciao,
Dscho

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: HTTP push
  2005-11-07  2:35 HTTP push Johannes Schindelin
@ 2005-11-07  4:31 ` Nick Hengeveld
  0 siblings, 0 replies; 2+ messages in thread
From: Nick Hengeveld @ 2005-11-07  4:31 UTC (permalink / raw)
  To: Johannes Schindelin; +Cc: git

On Mon, Nov 07, 2005 at 03:35:33AM +0100, Johannes Schindelin wrote:

> I played a little with git-http-push. There are some strange things going 
> on: whenever I commented out the calls to curl_multi_remove_handle() and 
> curl_multi_cleanup(), all was well. Whenever I did not, there was either a 
> segfault, or an endless loop.

http-push is a bit more aggressive than http-fetch about freeing up memory,
perhaps too aggressive...  I ran into similar issues when memory was
freed more than once.

> I will run some tests on another client with valgrind (that will wait 
> until tomorrow; it's 3.30am local time, so I head to bed).

Thanks, I'll work on memory cleanup some more.

-- 
For a successful technology, reality must take precedence over public
relations, for nature cannot be fooled.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2005-11-07  4:32 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-11-07  2:35 HTTP push Johannes Schindelin
2005-11-07  4:31 ` Nick Hengeveld

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).