From 5fa7f288ce71815c3bcc5267448f50f5ebc1b0fe Mon Sep 17 00:00:00 2001 From: Mengjiao Liu Date: Fri, 2 Apr 2021 15:56:07 +0800 Subject: [PATCH] Fix nil pointer dereference by changing code order --- pkg/volume/quobyte/quobyte_test.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/volume/quobyte/quobyte_test.go b/pkg/volume/quobyte/quobyte_test.go index ed7100cb179..d32879345f4 100644 --- a/pkg/volume/quobyte/quobyte_test.go +++ b/pkg/volume/quobyte/quobyte_test.go @@ -91,14 +91,14 @@ func doTestPlugin(t *testing.T, spec *volume.Spec) { pod := &v1.Pod{ObjectMeta: metav1.ObjectMeta{UID: types.UID("poduid")}} mounter, err := plug.(*quobytePlugin).newMounterInternal(spec, pod, mount.NewFakeMounter(nil)) - volumePath := mounter.GetPath() if err != nil { t.Errorf("Failed to make a new Mounter: %v", err) } if mounter == nil { - t.Error("Got a nil Mounter") + t.Fatal("Got a nil Mounter") } + volumePath := mounter.GetPath() if volumePath != fmt.Sprintf("%s/plugins/kubernetes.io~quobyte/root#root@vol", tmpDir) { t.Errorf("Got unexpected path: %s expected: %s", volumePath, fmt.Sprintf("%s/plugins/kubernetes.io~quobyte/root#root@vol", tmpDir)) }