Consider using SafeHandle in Libgit2Object instead of void*

Abierto
#2,111 8 comentarios 2 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
35/100
Tipo de issue
Refactorización
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
csharp, git
Área
devtools

Línea de trabajo

Comienza en LibGit2Sharp/Core/Handles/Libgit2Object.cs alrededor de la línea 88 y, después, sigue el finalizer y las sobrecargas de liberación nativa en LibGit2Sharp/Core/Handles/Objects.cs alrededor de las líneas 502-505. Compara esas rutas de ownership con las directrices de SafeHandle y los informes de runtime enlazados. El trabajo estará terminado cuando las rutas relevantes del ciclo de vida de los handles nativos estén representadas y liberadas de forma segura, sin la condición de carrera del finalizer indicada.

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

Descripción

We've had a couple of bug reports against dotnet/runtime with rarely-reproduceable crashes coming from libgit2sharp. We believe that one possible reason could be an interop anti-pattern in Libgit2Object - void*/IntPtr representation of a native handle that can be freed in finalizer. E.g. in this case the finalizer in Libgit2Object may end up calling native free here (and other overloads).

The reason why it's called an anti-pattern can be explained by a short repro in this issue: https://github.com/dotnet/runtime/issues/103522 and a general solution is to use SafeHandle for such handles. Also, see https://learn.microsoft.com/en-us/dotnet/standard/native-interop/best-practices

Reproduction steps
Expected behavior
Actual behavior
Version of LibGit2Sharp (release number or SHA1)
Operating system(s) tested; .NET runtime tested
Lenguaje dominante
C#
Estrellas
3.5k
Forks
925
Sin PR fusionados en 30 d

Guía de contribución

Abrir la guía de contribución

Primeros pasos

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

Más de libgit2/libgit2sharp

Todos los issues de libgit2/libgit2sharp

Issues similares

Más issues de C#

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.