Passing null to Repository.Discover(...) causes AccessViolationException / process termination

Abierto

#1,592 abierto el 5/7/2018 2 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Lenguaje dominante: C#
Estrellas: 3.5k
Forks: 925
Métricas de merge de PR: Sin PR fusionados en 30 d

Descripción

Reproduction steps

Pass null to Repository.Discover(...)

Expected behavior

A friendly exception. 😉

Actual behavior

Application: devenv.exe
Framework Version: v4.0.30319
Description: The process was terminated due to an unhandled exception.
Exception Info: System.AccessViolationException
   at LibGit2Sharp.Core.NativeMethods.git_repository_discover(LibGit2Sharp.Core.Handles.GitBuf, LibGit2Sharp.Core.FilePath, Boolean, LibGit2Sharp.Core.FilePath)
   at LibGit2Sharp.Core.Proxy+<>c__DisplayClass3e.<git_repository_discover>b__3d(LibGit2Sharp.Core.Handles.GitBuf)
   at LibGit2Sharp.Core.Proxy.ConvertPath(System.Func`2<LibGit2Sharp.Core.Handles.GitBuf,Int32>)
   at LibGit2Sharp.Core.Proxy.git_repository_discover(LibGit2Sharp.Core.FilePath)
   at LibGit2Sharp.Repository.Discover(System.String)
   at GitHub.Services.GitService.GetRepository(System.String)

Version of LibGit2Sharp (release number or SHA1)

0.23.1.0

Operating system(s) tested; .NET runtime tested

Windows 10 + .NET v4.0.30319

Guía de contribución

Abrir la guía de contribución

Primeros pasos

Lee el issue completo y luego la guía de contribución del proyecto.
Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
Haz un fork del repositorio y trabaja en una rama.
Abre un pull request que haga referencia al número del issue.

Línea de trabajo

El punto de entrada que falla es Repository.Discover(String), que llega a Core.Proxy.git_repository_discover y NativeMethods.git_repository_discover en el stack trace. Empieza por ahí, siguiendo cómo se pasa una ruta null a la llamada nativa; después añade una prueba de regresión y verifica que null produzca una excepción administrada en lugar de terminar el proceso.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico: csharp, git
Área: api, backend
Tipo de issue: Error
Dificultad: 3/5
Tiempo estimado: 1-2 días
Estado de actividad: Estancado
Claridad: Bien especificado
Aptitud para principiantes: 38/100

Un resumen breve de issues de GitHub para principiantes.