handle file close error

This commit is contained in:
JieJhih Jhang 2019-04-21 16:28:46 +08:00
parent 1377108c08
commit 84d29664c4

View File

@ -334,7 +334,7 @@ func (o *Options) runLoop() error {
} }
} }
func (o *Options) writeConfigFile() error { func (o *Options) writeConfigFile() (err error) {
const mediaType = runtime.ContentTypeYAML const mediaType = runtime.ContentTypeYAML
info, ok := runtime.SerializerInfoForMediaType(o.codecs.SupportedMediaTypes(), mediaType) info, ok := runtime.SerializerInfoForMediaType(o.codecs.SupportedMediaTypes(), mediaType)
if !ok { if !ok {
@ -347,10 +347,15 @@ func (o *Options) writeConfigFile() error {
if err != nil { if err != nil {
return err return err
} }
// TODO handle error
defer configFile.Close()
if err := encoder.Encode(o.config, configFile); err != nil { defer func() {
ferr := configFile.Close()
if ferr != nil && err == nil {
err = ferr
}
}()
if err = encoder.Encode(o.config, configFile); err != nil {
return err return err
} }