From 59a537514fcc8c69daacba66f0baa120c4317319 Mon Sep 17 00:00:00 2001 From: Shivam Sandbhor Date: Wed, 19 Jan 2022 20:04:09 +0530 Subject: [PATCH] Check for errors before modifying proc attrs (#1181) Signed-off-by: Shivam Sandbhor --- pkg/csplugin/broker.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/csplugin/broker.go b/pkg/csplugin/broker.go index c8cc94e78..4da994522 100644 --- a/pkg/csplugin/broker.go +++ b/pkg/csplugin/broker.go @@ -241,10 +241,10 @@ func (pb *PluginBroker) loadNotificationPlugin(name string, binaryPath string) ( } cmd := exec.Command(binaryPath) cmd.SysProcAttr, err = getProcessAtr(pb.pluginProcConfig.User, pb.pluginProcConfig.Group) - cmd.SysProcAttr.Credential.NoSetGroups = true if err != nil { return nil, errors.Wrap(err, "while getting process attributes") } + cmd.SysProcAttr.Credential.NoSetGroups = true pb.pluginMap[name] = &NotifierPlugin{} l := log.New() err = types.ConfigureLogger(l)