Bug 756198

Created attachment 535215 [details]
File: dso_list

Created attachment 535216 [details]
File: environ

Created attachment 535217 [details]
File: maps

Created attachment 535218 [details]
File: backtrace

Backtrace analysis of bugs across components suggests the actual bug is in component perl, subversion or neon instead of component git, reassigning to perl.

Bugs which were found to be similar to this bug: bug #605287, bug #693915

This comment is automatically generated.

This NULL pointer dereference in glibc strlen(). The SVN extension for git is written as Perl module in C. I think the bug is in that code (git-svn) instead of in the perl. At least this warning backs it:

Username: Use of uninitialized value $username in chomp at
/usr/libexec/git-core/git-svn line 4322.

Reproducer:

(1) Install git-svn
(2) Run "git svn clone 'http://quick-settings.googlecode.com/svn' </dev/null"

Pure subversion-perl reproducer:

#!/usr/bin/perl
use strict;
use warnings;

use SVN::Client;

sub simple_prompt {
    my $cred = shift;
    $cred->username(undef);
    $cred->password(undef);
}

my $ctx = SVN::Client->new(
    auth => [SVN::Client::get_simple_provider(),
    SVN::Client::get_simple_prompt_provider(\&simple_prompt, 1),
    SVN::Client::get_username_provider()]
);

$ctx->ls('http://quick-settings.googlecode.com/svn/', 'HEAD', 0);

exit 0;
__END__

This is simplified version of an example from SVN::Client documentation. The important part is setting undef as the username. You can use any HTTP/SVN server requiring authentication. This segfaults on the same place in
subversion/bindings/swig/perl/native/core.c:

XS(_wrap_svn_auth_cred_simple_t_username_set) {
    [...]
    char *buf2 = 0 ;
    [...]
    arg1 = (svn_auth_cred_simple_t *)(argp1);
    res2 = SWIG_AsCharPtrAndSize(ST(1), &buf2, NULL, &alloc2);
    if (!SWIG_IsOK(res2)) {
      SWIG_exception_fail(SWIG_ArgError(res2), "in method '" "svn_auth_cred_simple_t_username_set" "', argument " "2"" of type '" "char const *""'");
    }
    arg2 = (char *)(buf2);
    {
→     apr_size_t len = strlen(arg2) + 1;
      char *copied;
      if (arg1->username) free((char *)arg1->username);
      copied = malloc(len);
      memcpy(copied, arg2, len);
      arg1->username = copied;
    }
    [...]
}

Full back-trace shows arg2 is NULL. The arg2 is buf2 which for unknown reason is not properly initialized by SWIG_AsCharPtrAndSize().

Reassigning to subversion.

This message is a reminder that Fedora 16 is nearing its end of life.
Approximately 4 (four) weeks from now Fedora will stop maintaining
and issuing updates for Fedora 16. It is Fedora's policy to close all
bug reports from releases that are no longer maintained. At that time
this bug will be closed as WONTFIX if it remains open with a Fedora 
'version' of '16'.

Package Maintainer: If you wish for this bug to remain open because you
plan to fix it in a currently maintained version, simply change the 'version' 
to a later Fedora version prior to Fedora 16's end of life.

Bug Reporter: Thank you for reporting this issue and we are sorry that 
we may not be able to fix it before Fedora 16 is end of life. If you 
would still like to see this bug fixed and are able to reproduce it 
against a later version of Fedora, you are encouraged to click on 
"Clone This Bug" and open it against that version of Fedora.

Although we aim to fix as many bugs as possible during every release's 
lifetime, sometimes those efforts are overtaken by events. Often a 
more recent Fedora release includes newer upstream software that fixes 
bugs or makes them obsolete.

The process we are following is described here: 
http://fedoraproject.org/wiki/BugZappers/HouseKeeping

Fedora 16 changed to end-of-life (EOL) status on 2013-02-12. Fedora 16 is 
no longer maintained, which means that it will not receive any further 
security or bug fix updates. As a result we are closing this bug.

If you can reproduce this bug against a currently maintained version of 
Fedora please feel free to reopen this bug against that version.

Thank you for reporting this bug and we are sorry it could not be fixed.