diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2023-09-22 13:42:59 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-09-22 13:42:59 +0200 |
commit | fe9a0962e81b17ad49449f4a8cca38bbd5d0d7d6 (patch) | |
tree | ad4f705cd619f87d1fdb1c94be908af2a482c2be | |
parent | e8f307505d01449b77c16b3de4a28bb017de90dd (diff) | |
parent | 71125e36f538b3048729ebb97d68ba75948a1d6b (diff) |
Merge pull request #28614 from vespa-engine/toregge/only-read-when-file-pointer-is-valid
Don't read from invalid file pointer (e.g. when
-rw-r--r-- | eval/src/apps/analyze_onnx_model/analyze_onnx_model.cpp | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/eval/src/apps/analyze_onnx_model/analyze_onnx_model.cpp b/eval/src/apps/analyze_onnx_model/analyze_onnx_model.cpp index 03db333d582..31cb1d6b385 100644 --- a/eval/src/apps/analyze_onnx_model/analyze_onnx_model.cpp +++ b/eval/src/apps/analyze_onnx_model/analyze_onnx_model.cpp @@ -78,10 +78,12 @@ MemoryUsage extract_memory_usage() { vespalib::string vm_size = UNKNOWN; vespalib::string vm_rss = UNKNOWN; FilePointer file(fopen("/proc/self/status", "r")); - vespalib::string line; - while (read_line(file, line)) { - extract(line, "VmSize:", vm_size); - extract(line, "VmRSS:", vm_rss); + if (file.valid()) { + vespalib::string line; + while (read_line(file, line)) { + extract(line, "VmSize:", vm_size); + extract(line, "VmRSS:", vm_rss); + } } return {convert(vm_size), convert(vm_rss)}; } |