From: Sergey Matveev <stargrave@stargrave.org>
Date: Thu, 23 Mar 2023 10:59:04 +0000 (+0300)
Subject: Do not omit errors
X-Git-Tag: v0.27.3~1
X-Git-Url: http://www.git.stargrave.org/?p=sgblog.git;a=commitdiff_plain;h=b5008a81433fad137b0f867e90da4d902c17f1a5

Do not omit errors
---

diff --git a/cmd/sgblog-comment-add/mail.go b/cmd/sgblog-comment-add/mail.go
index 2a8bf45..98ee162 100644
--- a/cmd/sgblog-comment-add/mail.go
+++ b/cmd/sgblog-comment-add/mail.go
@@ -21,6 +21,7 @@ import (
 	"bytes"
 	"encoding/base64"
 	"errors"
+	"fmt"
 	"io"
 	"mime"
 	"mime/multipart"
@@ -74,6 +75,10 @@ func parseEmail(msg *mail.Message) (subj string, body io.Reader, err error) {
 		return
 	}
 	ct, params, err := mime.ParseMediaType(ct)
+	if err != nil {
+		err = fmt.Errorf("can not ParseMediaType: %w", err)
+		return
+	}
 	if ct != "multipart/signed" {
 		err = errors.New("only text/plain and multipart/signed+text/plain Content-Type supported")
 		return
@@ -108,7 +113,11 @@ func parseEmail(msg *mail.Message) (subj string, body io.Reader, err error) {
 			return
 		}
 		if strings.HasPrefix(ct, "multipart/mixed") {
-			ct, params, err = mime.ParseMediaType(ct)
+			_, params, err = mime.ParseMediaType(ct)
+			if err != nil {
+				err = fmt.Errorf("can not ParseMediaType: %w", err)
+				return
+			}
 			boundary = params["boundary"]
 			if len(boundary) == 0 {
 				err = errors.New("no boundary string")