diff options
Diffstat (limited to 'internal/media')
-rw-r--r-- | internal/media/processingemoji.go | 12 | ||||
-rw-r--r-- | internal/media/processingmedia.go | 14 |
2 files changed, 12 insertions, 14 deletions
diff --git a/internal/media/processingemoji.go b/internal/media/processingemoji.go index 121f54ddc..e3ed5ce0a 100644 --- a/internal/media/processingemoji.go +++ b/internal/media/processingemoji.go @@ -121,6 +121,12 @@ func (p *ProcessingEmoji) loadStatic(ctx context.Context) error { return p.err } + defer func() { + if err := stored.Close(); err != nil { + log.Errorf("loadStatic: error closing stored full size: %s", err) + } + }() + // we haven't processed a static version of this emoji yet so do it now static, err := deriveStaticEmoji(stored, p.emoji.ImageContentType) if err != nil { @@ -129,12 +135,6 @@ func (p *ProcessingEmoji) loadStatic(ctx context.Context) error { return p.err } - if err := stored.Close(); err != nil { - p.err = fmt.Errorf("loadStatic: error closing stored full size: %s", err) - atomic.StoreInt32(&p.staticState, int32(errored)) - return p.err - } - // put the static in storage if err := p.storage.Put(ctx, p.emoji.ImageStaticPath, static.small); err != nil { p.err = fmt.Errorf("loadStatic: error storing static: %s", err) diff --git a/internal/media/processingmedia.go b/internal/media/processingmedia.go index 26fbc0cea..3f3a68f3f 100644 --- a/internal/media/processingmedia.go +++ b/internal/media/processingmedia.go @@ -145,9 +145,7 @@ func (p *ProcessingMedia) loadThumb(ctx context.Context) error { return p.err } - // whatever happens, close the stream when we're done defer func() { - log.Tracef("loadThumb: closing stored stream %s", p.attachment.URL) if err := stored.Close(); err != nil { log.Errorf("loadThumb: error closing stored full size: %s", err) } @@ -210,6 +208,12 @@ func (p *ProcessingMedia) loadFullSize(ctx context.Context) error { return p.err } + defer func() { + if err := stored.Close(); err != nil { + log.Errorf("loadFullSize: error closing stored full size: %s", err) + } + }() + // decode the image ct := p.attachment.File.ContentType switch ct { @@ -227,12 +231,6 @@ func (p *ProcessingMedia) loadFullSize(ctx context.Context) error { return p.err } - if err := stored.Close(); err != nil { - p.err = fmt.Errorf("loadFullSize: error closing stored full size: %s", err) - atomic.StoreInt32(&p.fullSizeState, int32(errored)) - return p.err - } - // set appropriate fields on the attachment based on the image we derived p.attachment.FileMeta.Original = gtsmodel.Original{ Width: decoded.width, |