Resource may leak when ReadAll meets error

[rleungx]

Bug Report

1. Minimal reproduce step (Required)

tidb/domain/infosync/info.go

Lines 347 to 363 in b2ea0e6

	res, err = doRequestWithFailpoint(req)
	metrics.PDApiExecutionHistogram.WithLabelValues("placement").Observe(time.Since(start).Seconds())
	if err == nil {
	bodyBytes, err := io.ReadAll(res.Body)
	if err != nil {
	return nil, err
	}
	if res.StatusCode != http.StatusOK {
	err = errors.Errorf("%s", bodyBytes)
	if res.StatusCode == http.StatusNotFound || res.StatusCode == http.StatusPreconditionFailed {
	err = nil
	bodyBytes = nil
	}
	}
	terror.Log(res.Body.Close())
	return bodyBytes, err
	}

2. What did you expect to see? (Required)

The body should be closed when the error happens.

3. What did you see instead (Required)

The resource may leak.

4. What is your TiDB version? (Required)

master

[github-actions]

Please check whether the issue should be labeled with 'affects-x.y' or 'fixes-x.y.z', and then remove 'needs-more-info' label.