Crash calling Repository.IsValid with corrupt .git/config

Offen

#1,851 erstellt am 14.11.2020 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 3/5
Geschätzter Aufwand: 1-2 Tage
Anfängerfreundlichkeit: 45/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: csharp, git
Bereich: tooling

Rechercherichtung

Beginnen Sie bei Repository.IsValid und verfolgen Sie den Aufruf über Proxy.git_repository_open_ext bis zu NativeMethods.git_repository_open_ext. Untersuchen Sie anschließend den nativen Fehlerpfad, der git_repository_free umfasst. Reproduzieren Sie den Fall einer beschädigten .git/config unter Windows und verifizieren Sie, dass er statt einer Zugriffsverletzung LibGit2SharpException auslöst; fügen Sie für dieses Szenario eine Testabdeckung hinzu oder aktualisieren Sie diese, falls die bestehende Teststruktur einen geeigneten Ort dafür erkennen lässt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Reproduction steps

Add this to .git/config
[diff "mytool"] command = c:\path\to\my\tool\MyToolComparer.exe
This will cause the following effect for git

c:\my\git\repo> git status
fatal: bad config line 8 in file .git/config

Then call Repository.IsValid(@"c:\my\git\repo\.git") which is turn calls Proxy.git_repository_open_ext(path, RepositoryOpenFlags.NoSearch, null);

This fails because the return value from NativeMethods.git_repository_open_ext(out repo, path, flags, ceilingDirs) is not checked for error prior to freeing the out parameter repo (which is already free'd when there is an error condition in int git_repository_open_ext(git_repository **repo_ptr, const char *start_path, unsigned int flags, const char *ceiling_dirs))

Expected behavior

LibGit2Sharp.LibGit2SharpException

Actual behavior

Access violation (0xC0000005) in void git_repository_free(git_repository *repo) when dereferencing the null repo

Version of LibGit2Sharp (release number or SHA1)

0.27.0.34

Operating system(s) tested; .NET runtime tested

Windows 10 Pro 1909 - .NET CLR Version v4.0.30319

Vorherrschende Sprache: C#
Sterne: 3.5k
Forks: 925
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus libgit2/libgit2sharp

Potential AccessViolation at process exit on .NET Framework - SafeHandle finalizers now race `git_libgit2_shutdown` Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 52/100

libgit2/libgit2sharp#2193 · 2 Kommentare · Erstellt am 01.9.2026
AccessViolationException accessing Commit.Parents concurrently on Linux .NET 10 Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 38/100

libgit2/libgit2sharp#2192 · 1 Kommentar · Erstellt am 01.9.2026
Website is down Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 20/100

libgit2/libgit2sharp#2191 · 2 Reaktionen · Erstellt am 20.8.2026
Indexing a tree by path yields different result with trailing slash Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 68/100

libgit2/libgit2sharp#2189 · 1 Reaktion · Erstellt am 04.8.2026
Running from Native AOT app fails to build signature Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 35/100

libgit2/libgit2sharp#2187 · 2 Kommentare · Erstellt am 22.7.2026

Alle Issues in libgit2/libgit2sharp