From fc6eaf36388841b16ff004e1d48e887d3f9613dc Mon Sep 17 00:00:00 2001
From: michael boulton <61595820+mbfr@users.noreply.github.com>
Date: Fri, 14 Aug 2020 19:19:21 +0100
Subject: Fix null pointer dereference in status.ByIndex (#628)

`git_status_byindex` can return a null pointer if there is no statuses.
---
 status_test.go | 28 ++++++++++++++++++++++++++++
 1 file changed, 28 insertions(+)

(limited to 'status_test.go')

diff --git a/status_test.go b/status_test.go
index 17ed94f..d5cd530 100644
--- a/status_test.go
+++ b/status_test.go
@@ -61,3 +61,31 @@ func TestStatusList(t *testing.T) {
 		t.Fatal("Incorrect entry path: ", entry.IndexToWorkdir.NewFile.Path)
 	}
 }
+
+func TestStatusNothing(t *testing.T) {
+	t.Parallel()
+	repo := createTestRepo(t)
+	defer cleanupTestRepo(t, repo)
+
+	seedTestRepo(t, repo)
+
+	opts := &StatusOptions{
+		Show:  StatusShowIndexAndWorkdir,
+		Flags: StatusOptIncludeUntracked | StatusOptRenamesHeadToIndex | StatusOptSortCaseSensitively,
+	}
+
+	statusList, err := repo.StatusList(opts)
+	checkFatal(t, err)
+
+	entryCount, err := statusList.EntryCount()
+	checkFatal(t, err)
+
+	if entryCount != 0 {
+		t.Fatal("expected no statuses in empty repo")
+	}
+
+	_, err = statusList.ByIndex(0)
+	if err == nil {
+		t.Error("expected error getting status by index")
+	}
+}
-- 
cgit v1.2.3