highperfocused/bitcoin
1
0
Fork 0
mirror of https://github.com/bitcoin/bitcoin.git synced 2025-07-07 08:50:26 +02:00
Code Issues Packages Projects Releases Wiki Activity

Net: Fix resource leak in ReadBinaryFile(...)

Browse Source 
Introduced in 0b6f40d4ca via PR #10408.
This commit is contained in:
practicalswift
2017-06-14 17:32:45 +02:00
parent 228c319a94
commit f2fb132cb0
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/torcontrol.cpp
View File

@ -376,8 +376,10 @@ static std::pair<bool,std::string> ReadBinaryFile(const fs::path &filename, size
while ((n=fread(buffer, 1, sizeof(buffer), f)) > 0) { while ((n=fread(buffer, 1, sizeof(buffer), f)) > 0) {
// Check for reading errors so we don't return any data if we couldn't // Check for reading errors so we don't return any data if we couldn't
// read the entire file (or up to maxsize) // read the entire file (or up to maxsize)
if (ferror(f)) if (ferror(f)) {
fclose(f);
return std::make_pair(false,""); return std::make_pair(false,"");
}
retval.append(buffer, buffer+n); retval.append(buffer, buffer+n);
if (retval.size() > maxsize) if (retval.size() > maxsize)
break; break;