apps/go2rtc
1
0
Fork 0
mirror of https://github.com/AlexxIT/go2rtc.git synced 2025-10-13 03:53:53 +08:00
Code Issues Packages Projects Releases Wiki Activity

Fix resource leak in Nest source due to lack of closing HTTP response bodies

Browse Source
This commit is contained in:
Chris Thach
2024-08-06 22:25:55 +00:00
parent bd88695e59
commit 66de2f91b6
1 changed files with 5 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
pkg/nest/api.go
View File

@@ -53,6 +53,8 @@ func NewAPI(clientID, clientSecret, refreshToken string) (*API, error) {
if err != nil { if err != nil {
return nil, err return nil, err
} }
defer res.Body.Close()
if res.StatusCode != 200 { if res.StatusCode != 200 {
return nil, errors.New("nest: wrong status: " + res.Status) return nil, errors.New("nest: wrong status: " + res.Status)
} }
@@ -92,6 +94,7 @@ func (a *API) GetDevices(projectID string) (map[string]string, error) {
if err != nil { if err != nil {
return nil, err return nil, err
} }
defer res.Body.Close()
if res.StatusCode != 200 { if res.StatusCode != 200 {
return nil, errors.New("nest: wrong status: " + res.Status) return nil, errors.New("nest: wrong status: " + res.Status)
@@ -157,6 +160,7 @@ func (a *API) ExchangeSDP(projectID, deviceID, offer string) (string, error) {
if err != nil { if err != nil {
return "", err return "", err
} }
defer res.Body.Close()
if res.StatusCode != 200 { if res.StatusCode != 200 {
return "", errors.New("nest: wrong status: " + res.Status) return "", errors.New("nest: wrong status: " + res.Status)
@@ -211,6 +215,7 @@ func (a *API) ExtendStream() error {
if err != nil { if err != nil {
return err return err
} }
defer res.Body.Close()
if res.StatusCode != 200 { if res.StatusCode != 200 {
return errors.New("nest: wrong status: " + res.Status) return errors.New("nest: wrong status: " + res.Status)