Check for errors before modifying proc attrs (#1181)

Signed-off-by: Shivam Sandbhor <shivam.sandbhor@gmail.com>
This commit is contained in:
Shivam Sandbhor 2022-01-19 20:04:09 +05:30 committed by GitHub
parent cc1ab8c50d
commit 59a537514f
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -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)