Fix hypotetical file leakage in cputemp.go
This commit is contained in:
@@ -36,28 +36,25 @@ func (c *MyConfig) UpdateCPUTemperature() {
|
|||||||
|
|
||||||
if err != nil {
|
if err != nil {
|
||||||
log.Printf("Unable to open %s: %s", filename, err)
|
log.Printf("Unable to open %s: %s", filename, err)
|
||||||
} else {
|
continue
|
||||||
|
}
|
||||||
|
defer file.Close()
|
||||||
|
|
||||||
reader := bufio.NewReader(file)
|
reader := bufio.NewReader(file)
|
||||||
line, _, err := reader.ReadLine()
|
line, _, err := reader.ReadLine()
|
||||||
|
|
||||||
if err != nil {
|
if err != nil {
|
||||||
log.Printf("Unable to read from %s: %s", filename, err)
|
log.Printf("Unable to read from %s: %s", filename, err)
|
||||||
err = file.Close()
|
continue
|
||||||
|
|
||||||
if err != nil {
|
|
||||||
log.Printf("Unable to close %s: %s", filename, err)
|
|
||||||
}
|
}
|
||||||
} else {
|
|
||||||
err = file.Close()
|
|
||||||
|
|
||||||
if err != nil {
|
|
||||||
log.Printf("Unable to close %s: %s", filename, err)
|
|
||||||
} else {
|
|
||||||
temp, err := strconv.ParseInt(string(line), 10, 32)
|
temp, err := strconv.ParseInt(string(line), 10, 32)
|
||||||
|
|
||||||
if err != nil {
|
if err != nil {
|
||||||
log.Printf("Unable to convert string to number from file %s: %s", filename, err)
|
log.Printf("Unable to convert string to number from file %s: %s", filename, err)
|
||||||
} else {
|
continue
|
||||||
|
}
|
||||||
|
|
||||||
if temp > 1000 {
|
if temp > 1000 {
|
||||||
temp /= 1000
|
temp /= 1000
|
||||||
}
|
}
|
||||||
@@ -65,10 +62,6 @@ func (c *MyConfig) UpdateCPUTemperature() {
|
|||||||
temperature[i] = temp
|
temperature[i] = temp
|
||||||
validCount++
|
validCount++
|
||||||
}
|
}
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
if validCount == 0 {
|
if validCount == 0 {
|
||||||
continue
|
continue
|
||||||
|
|||||||
Reference in New Issue
Block a user