diff options
author | 2017-12-10 19:44:54 -0600 | |
---|---|---|
committer | 2017-12-10 20:44:54 -0500 | |
commit | e17ae378063b46c894a8c193823f029d7d87de81 (patch) | |
tree | 58ba9d0c2377b4f1894a8676245896f727f6324d /tensorflow/core/platform/profile_utils/android_armv7a_cpu_utils_helper.cc | |
parent | 76a9354d2d90ef8f2fa6562d3d8ef6bc973d56a3 (diff) |
Fix several potential memory leaks (#14816)
* Fix several potential memory leaks
This fix fixes several potential memory leaks, mostly
caused by error return without proper deleting.
Note: The original issue was raised by @orpillar, thanks!
This fix fixes 14800.
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
Diffstat (limited to 'tensorflow/core/platform/profile_utils/android_armv7a_cpu_utils_helper.cc')
-rw-r--r-- | tensorflow/core/platform/profile_utils/android_armv7a_cpu_utils_helper.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tensorflow/core/platform/profile_utils/android_armv7a_cpu_utils_helper.cc b/tensorflow/core/platform/profile_utils/android_armv7a_cpu_utils_helper.cc index fb1955edde..12dc9c58b3 100644 --- a/tensorflow/core/platform/profile_utils/android_armv7a_cpu_utils_helper.cc +++ b/tensorflow/core/platform/profile_utils/android_armv7a_cpu_utils_helper.cc @@ -118,9 +118,10 @@ int64 AndroidArmV7ACpuUtilsHelper::ReadCpuFrequencyFile( const int retval = fscanf(fp, "%lld", &freq_in_khz); if (retval < 0) { LOG(WARNING) << "Failed to \"" << file_path << "\""; + fclose(fp); return INVALID_CPU_FREQUENCY; } - pclose(fp); + fclose(fp); return freq_in_khz * 1000; // The file contains cpu frequency in khz } |