fix: unchecked error on ReadAll (#851)

Drew Smirnoff and gittihub-jpg created 2 months ago

Co-authored-by: gittihub-jpg <[e-mail]>

Change summary

fetcher/fetcher.go | 12 ++++++++++--
1 file changed, 10 insertions(+), 2 deletions(-)

Detailed changes

fetcher/fetcher.go

@@ -185,9 +185,17 @@ func decodeAttachmentData(rawBytes []byte, encoding string) ([]byte, error) {
 	switch strings.ToLower(encoding) {
 	case "base64":
 		decoder := base64.NewDecoder(base64.StdEncoding, bytes.NewReader(rawBytes))
-		return ioutil.ReadAll(decoder)
+		data, err := ioutil.ReadAll(decoder)
+		if err != nil {
+			return nil, err
+		}
+		return data, nil
 	case "quoted-printable":
-		return ioutil.ReadAll(quotedprintable.NewReader(bytes.NewReader(rawBytes)))
+		data, err := ioutil.ReadAll(quotedprintable.NewReader(bytes.NewReader(rawBytes)))
+		if err != nil {
+			return nil, err
+		}
+		return data, nil
 	default:
 		return rawBytes, nil
 	}