Report a coding bug about unused result.

[guodongli-google]

Confirm by changing [ ] to [x] below to ensure that it's a bug:

• I've gone though Developer Guide and API reference
• I've checked AWS Forums and StackOverflow for answers
• I've searched for previous similar issues and didn't find any solution

Describe the bug
At https://github.com/aws/aws-sdk-go/blob/main/private/protocol/rest/unmarshal.go#L143, a new error is created but not returned, and function awserr.New has no side effect:

	if err != nil {
		awserr.New(request.ErrCodeSerialization, "failed to decode REST response", err)
	}

Version of AWS SDK for Go?
The current version

Expected behavior
The correct code should be:

	if err != nil {
		return awserr.New(request.ErrCodeSerialization, "failed to decode REST response", err)
	}

Found by static analyzer #deepgo

[KaibaLopez]

Hi @guodongli-google ,
Thanks for pointing this out to us

[github-actions]

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.