Close response bodies properly

Why exactly didn't this blow up yet?

go/favicon/favicon.go

	if err != nil {
		return "", err
	}
	defer resp.Body.Close()
	tree, err := html.Parse(resp.Body)
	fmt.Println("parse html", tree, err)
	if err != nil {
	if err != nil {
		return "", err
	}
	defer resp.Body.Close()
	if resp.StatusCode >= 400 || resp.Header.Get("Content-Length") == "0" {
		return "", errors.New("no /favicon.ico")
	}