cppcheck: Passing the result of c_str() to a function that takes std::string as argument

cppcheck: Passing the result of c_str() to a function that takes std::string as argument is slow and redundant.

Tag @sroe