github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-07-21 10:51:29 +00:00
Code Issues Projects Releases Wiki Activity

kubeadm: fix the nil pointer dereference in testcase

Browse Source 
`genCSRConfig.kubeadmConfig` is possible to be nil if there any error
from the config loading, so access the field should only be done if
there is no error in the previous step.

Signed-off-by: Dave Chen <dave.chen@arm.com>
This commit is contained in:
Dave Chen 2023-01-05 16:15:44 +08:00
parent 575616cc72
commit 23f75bf40e
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
cmd/kubeadm/app/cmd/certs_test.go
View File

@ -464,11 +464,11 @@ kubernetesVersion: %s`,
err := config.load() err := config.load()
if test.expectErr { if test.expectErr {
assert.Error(t, err) assert.Error(t, err)
} else {
assert.NoError(t, err)
} }
for _, assertFunc := range test.assertions { if !test.expectErr && assert.NoError(t, err) {
assertFunc(t, config) for _, assertFunc := range test.assertions {
assertFunc(t, config)
}
} }
}) })
} }