Git-Mirrors/monero
1
0
Fork 0
mirror of https://github.com/monero-project/monero.git synced 2025-08-09 07:42:16 -04:00
Code Issues Projects Releases Packages Wiki Activity

wipeable_string: call memwipe directly

Browse source 
since the original reason for the indirect call (that memwipe
was not in contrib) is now gone
This commit is contained in:
moneromooo-monero 2018-01-31 12:25:30 +00:00
parent ed67e5c001
commit fde4489e1d
No known key found for this signature in database
GPG key ID: 686F07454D6CEFC3
3 changed files with 6 additions and 15 deletions
Download patch file Download diff file Expand all files Collapse all files

16
contrib/epee/src/wipeable_string.cpp
View file

@ -27,14 +27,13 @@
// THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
#include <string.h>
#include "memwipe.h"
#include "misc_log_ex.h"
#include "wipeable_string.h"
namespace epee
{
void *(*wipeable_string::wipefunc)(void*, size_t) = NULL;
wipeable_string::wipeable_string(const wipeable_string &other):
buffer(other.buffer)
{
@ -55,12 +54,11 @@ wipeable_string::wipeable_string(const std::string &other)
wipeable_string::wipeable_string(std::string &&other)
{
CHECK_AND_ASSERT_THROW_MES(wipefunc, "wipefunc is not set");
grow(other.size());
memcpy(buffer.data(), other.c_str(), size());
if (!other.empty())
{
wipefunc(&other[0], other.size()); // we're kinda left with this again aren't we
memwipe(&other[0], other.size()); // we're kinda left with this again aren't we
other = std::string();
}
}
@ -78,30 +76,28 @@ wipeable_string::~wipeable_string()
void wipeable_string::wipe()
{
CHECK_AND_ASSERT_THROW_MES(wipefunc, "wipefunc is not set");
wipefunc(buffer.data(), buffer.size() * sizeof(char));
memwipe(buffer.data(), buffer.size() * sizeof(char));
}
void wipeable_string::grow(size_t sz, size_t reserved)
{
CHECK_AND_ASSERT_THROW_MES(wipefunc, "wipefunc is not set");
if (reserved < sz)
reserved = sz;
if (reserved <= buffer.capacity())
{
if (sz < buffer.size())
wipefunc(buffer.data() + sz, buffer.size() - sz);
memwipe(buffer.data() + sz, buffer.size() - sz);
buffer.resize(sz);
return;
}
size_t old_sz = buffer.size();
std::unique_ptr<char[]> tmp{new char[old_sz]};
memcpy(tmp.get(), buffer.data(), old_sz * sizeof(char));
wipefunc(buffer.data(), old_sz * sizeof(char));
memwipe(buffer.data(), old_sz * sizeof(char));
buffer.reserve(reserved);
buffer.resize(sz);
memcpy(buffer.data(), tmp.get(), old_sz * sizeof(char));
wipefunc(tmp.get(), old_sz * sizeof(char));
memwipe(tmp.get(), old_sz * sizeof(char));
}
void wipeable_string::push_back(char c)