libgit2 / libgit2/libgit2sharp

Stash from StashCollection.Add is unsafe.

Offen
#865 11 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

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

Beschreibung

Let's say we create two stashes.

First stash operation:
Stash1 will be stash@{0}.

Second stash operation:
Stash1 will be stash@{1}
Stash2 will be stash@{0}

The managed reference to Stash1 will still be stash@{0}. We need a way to reflect these index changes, as unsafe in practice.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne damit, StashCollection.Add nachzuverfolgen, und reproduziere die im Issue beschriebene Sequenz mit zwei Stashes. Prüfe, wie die verwaltete Referenz auf Stash1 nach dem zweiten Hinzufügen dargestellt wird; die Aufgabe ist erledigt, wenn diese Referenz Stash1 weiterhin identifiziert, nachdem sich die Stash-Indizes verschoben haben. Es ist keine Test- oder Quelldatei genannt, daher erfordern das Auffinden der relevanten Implementierung und das Definieren des sicheren Aktualisierungsverhaltens Vertrautheit mit dem Projekt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
csharp, git
Bereich
devtools
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.