# segfault in http-push

5 messages from 2008-02-17 to 2008-02-18. Participants: Bob Copeland, Johannes Schindelin, Junio C Hamano.
Thread: https://gitlist.dev/t/12164

## Bob Copeland, 2008-02-17 22:00

Subject: segfault in http-push
Message-ID: <20080217220013.GA12419@hash.localnet>
URL: https://gitlist.dev/e/20080217220013.GA12419%40hash.localnet

```
Hi,

With 1.4.5.2, I see this whenever I try to push a new branch over http:

$ git-push hash long_ids
Fetching remote heads...
  refs/
  refs/heads/
  refs/tags/
  refs/remotes/
updating 'refs/heads/long_ids'
  from 0000000000000000000000000000000000000000
  to   b24938890f04f3ba42b88f56f3ff2bbe0fda6629
    sending 10200 objects
    done
Updating remote server info
error: http-push died with strange error
error: failed to push to 'https://user@machine/foo.git/'

Here's a backtrace:

#0  0xb7cbd583 in strlen () from /lib/tls/i686/cmov/libc.so.6
#1  0x08053b5a in start_put (request=0xdce9ad8) at http-push.c:520
#2  0x080550d7 in fill_active_slot (unused=0x0) at http-push.c:810
#3  0x0804da9d in fill_active_slots () at http.c:428
#4  0x0804dc8b in step_active_slots () at http.c:454
#5  0x0804dd0b in run_active_slot (slot=0x8e70148) at http.c:474
#6  0x080502c9 in remote_ls (path=0x808bdbd "refs/", flags=5, 
    userFunc=0x804f2b0 <add_remote_info_ref>, userData=0xbfa04434)
    at http-push.c:1467
#7  0x08050552 in update_remote_info_refs (lock=0x80e66c0) at http-push.c:1942
#8  0x080534ee in main (argc=3, argv=Cannot access memory at address 0x4
) at http-push.c:2421

Error is at:
520             request->url = xmalloc(strlen(remote->url) +
521                                    strlen(request->lock->token) + 51);

(gdb) p request->lock
$1 = (struct remote_lock *) 0x80e7d40
(gdb) p request->lock->token
$2 = 0x65707954 <Address 0x65707954 out of bounds>

If I go fix up the remote repository I can usually get it to work.  
Any thoughts?

-- 
Bob Copeland %% www.bobcopeland.com

```

## Johannes Schindelin, 2008-02-17 23:23

Subject: Re: segfault in http-push
Message-ID: <alpine.LSU.1.00.0802172321551.30505@racer.site>
URL: https://gitlist.dev/e/alpine.LSU.1.00.0802172321551.30505%40racer.site
In-Reply-To: <20080217220013.GA12419@hash.localnet>

```
Hi,

On Sun, 17 Feb 2008, Bob Copeland wrote:

> With 1.4.5.2, I see this whenever I try to push a new branch over http:

I can only suspect that you mean 1.5.4.2, not 1.4.5.2 (which I could not 
find).

Alas, a fix I provided for exactly this case did not make it into master 
yet, let alone into any released version.  Can you please test the current 
"next"?

FWIW, the fix is add8e8ce(http-push: avoid invalid memory accesses).

Thanks,
Dscho

```

## Bob Copeland, 2008-02-18 02:05

Subject: Re: segfault in http-push
Message-ID: <20080218020555.GA12814@hash.localnet>
URL: https://gitlist.dev/e/20080218020555.GA12814%40hash.localnet
In-Reply-To: <alpine.LSU.1.00.0802172321551.30505@racer.site>

```
On Sun, Feb 17, 2008 at 11:23:38PM +0000, Johannes Schindelin wrote:
> I can only suspect that you mean 1.5.4.2, not 1.4.5.2 (which I could not 
> find).

Just so...

> Alas, a fix I provided for exactly this case did not make it into master 
> yet, let alone into any released version.  Can you please test the current 
> "next"?

Yep, next fixes it for me, thanks!

-- 
Bob Copeland %% www.bobcopeland.com 

```

## Johannes Schindelin, 2008-02-18 02:08

Subject: Re: segfault in http-push
Message-ID: <alpine.LSU.1.00.0802180207360.30505@racer.site>
URL: https://gitlist.dev/e/alpine.LSU.1.00.0802180207360.30505%40racer.site
In-Reply-To: <20080218020555.GA12814@hash.localnet>

```
Hi,

On Sun, 17 Feb 2008, Bob Copeland wrote:

> On Sun, Feb 17, 2008 at 11:23:38PM +0000, Johannes Schindelin wrote:
> 
> > Alas, a fix I provided for exactly this case did not make it into 
> > master yet, let alone into any released version.  Can you please test 
> > the current "next"?
> 
> Yep, next fixes it for me, thanks!

Thanks for testing!

Ciao,
Dscho "who is pleased that we fix some bugs before they are reported"

```

## Junio C Hamano, 2008-02-18 03:06

Subject: Re: segfault in http-push
Message-ID: <7v7ih3vu80.fsf@gitster.siamese.dyndns.org>
URL: https://gitlist.dev/e/7v7ih3vu80.fsf%40gitster.siamese.dyndns.org
In-Reply-To: <alpine.LSU.1.00.0802180207360.30505@racer.site>

```
Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:

> Hi,
>
> On Sun, 17 Feb 2008, Bob Copeland wrote:
>
>> On Sun, Feb 17, 2008 at 11:23:38PM +0000, Johannes Schindelin wrote:
>> 
>> > Alas, a fix I provided for exactly this case did not make it into 
>> > master yet, let alone into any released version.  Can you please test 
>> > the current "next"?
>> 
>> Yep, next fixes it for me, thanks!
>
> Thanks for testing!
>
> Ciao,
> Dscho "who is pleased that we fix some bugs before they are reported"

Wonderful.

I take this as a go ahead to merge js/maint-http-push topic to 'maint'
before we cut 1.5.4.3.

Thanks both of you, especially Dscho.

```