Marian Klymov authored Jun 09, 2018 and Brad King committed Jun 13, 2018

`ParseValueFromKStat` was constructing local `std::string` instances in
a loop and passing their `.c_str()` to `RunProcess` after destruction.
Preserve the strings in a vector to avoid dangling pointers.

While at it, simplify some logic by using standard algorithms.