From 5637569f74f3e90bdc585609735d04231e9fc403 Mon Sep 17 00:00:00 2001 From: Rodrigo Campos Date: Sun, 7 Aug 2016 13:10:05 -0300 Subject: [PATCH] Check return value from volume.SetVolumeOwnership() in downwardAPI The function can fail, so we must check the return code. --- pkg/volume/downwardapi/downwardapi.go | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/pkg/volume/downwardapi/downwardapi.go b/pkg/volume/downwardapi/downwardapi.go index db8ded6ccbb..679c73114c3 100644 --- a/pkg/volume/downwardapi/downwardapi.go +++ b/pkg/volume/downwardapi/downwardapi.go @@ -187,7 +187,11 @@ func (b *downwardAPIVolumeMounter) SetUpAt(dir string, fsGroup *int64) error { return err } - volume.SetVolumeOwnership(b, fsGroup) + err = volume.SetVolumeOwnership(b, fsGroup) + if err != nil { + glog.Errorf("Error applying volume ownership settings for group: %v", fsGroup) + return err + } return nil }