Open-Harbor/osrm-backend
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 3 Wiki Activity

fix memory leak of char[] initialization

Browse Source
This commit is contained in:
Huyen Chau Nguyen 2016-10-19 11:55:35 -07:00
parent 316ef305de
commit 2d13116487
2 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
include/engine/datafacade/internal_datafacade.hpp
View File

@ -149,9 +149,8 @@ class InternalDataFacade final : public BaseDataFacade
}
auto timestamp_size = storage::io::readTimestampSize(timestamp_stream);
char *timestamp_ptr = new char[timestamp_size]();
storage::io::readTimestamp(timestamp_stream, timestamp_ptr, timestamp_size);
m_timestamp = std::string(timestamp_ptr);
m_timestamp.resize(timestamp_size);
storage::io::readTimestamp(timestamp_stream, &m_timestamp.front(), timestamp_size);
}
void LoadGraph(const boost::filesystem::path &hsgr_path)

4
include/storage/io.hpp
View File

@ -70,7 +70,9 @@ inline std::uint32_t readTimestampSize(boost::filesystem::ifstream &timestamp_in
}
// Reads the timestamp in a file
inline void readTimestamp(boost::filesystem::ifstream &timestamp_input_stream, char *timestamp, std::size_t timestamp_length)
inline void readTimestamp(boost::filesystem::ifstream &timestamp_input_stream,
char *timestamp,
std::size_t timestamp_length)
{
timestamp_input_stream.read(timestamp, timestamp_length * sizeof(char));
}