mirrors/keepassxc
1
0
Fork 0
mirror of https://github.com/keepassxreboot/keepassxc.git synced 2025-04-05 05:27:39 +03:00
Code Issues Projects Releases Packages Wiki Activity

Make Entry::m_tmpHistoryItem a QScopedPointer (#2524)

Browse source 
* Make m_tmpHistoryItem a QScopedPointer

Most of the time, `m_tmpHistoryItem` should be null by the time an
`Entry` is destroyed. However, if a caller ever calls `beginUpdate()`
without later calling `endUpdate()` -- perhaps because an exception was
throw in the meantime -- it may not be null. This change avoids a memory 
leak in that case.

Found via https://lgtm.com/projects/g/keepassxreboot/keepassxc/alerts
This commit is contained in:
Carlo Teubner 2018-12-01 21:08:55 +00:00 • committed by Jonathan White
parent 1af293847c
commit bdee748a6a
2 changed files with 6 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

13
src/core/Entry.cpp
View file

@ -40,7 +40,6 @@ Entry::Entry()
, m_attachments(new EntryAttachments(this))
, m_autoTypeAssociations(new AutoTypeAssociations(this))
, m_customData(new CustomData(this))
, m_tmpHistoryItem(nullptr)
, m_modifiedSinceBegin(false)
, m_updateTimeinfo(true)
{
@ -706,9 +705,9 @@ void Entry::copyDataFrom(const Entry* other)
void Entry::beginUpdate()
{
Q_ASSERT(!m_tmpHistoryItem);
Q_ASSERT(m_tmpHistoryItem.isNull());
m_tmpHistoryItem = new Entry();
m_tmpHistoryItem.reset(new Entry());
m_tmpHistoryItem->setUpdateTimeinfo(false);
m_tmpHistoryItem->m_uuid = m_uuid;
m_tmpHistoryItem->m_data = m_data;
@ -721,16 +720,14 @@ void Entry::beginUpdate()
bool Entry::endUpdate()
{
Q_ASSERT(m_tmpHistoryItem);
Q_ASSERT(!m_tmpHistoryItem.isNull());
if (m_modifiedSinceBegin) {
m_tmpHistoryItem->setUpdateTimeinfo(true);
addHistoryItem(m_tmpHistoryItem);
addHistoryItem(m_tmpHistoryItem.take());
truncateHistory();
} else {
delete m_tmpHistoryItem;
}
m_tmpHistoryItem = nullptr;
m_tmpHistoryItem.reset();
return m_modifiedSinceBegin;
}