update: remove unnecessary initialization on the heap

This commit is contained in:
2023-03-19 22:20:24 +01:00
parent d877f2cc6c
commit 860e777104

View File

@@ -422,8 +422,8 @@ bool CFlashUpdate::checkVersion4Update()
showGlobalStatus(20); showGlobalStatus(20);
hide(); hide();
versionInfo = new CFlashVersionInfo(newVersion); // Memory leak: versionInfo CFlashVersionInfo versionInfo(newVersion);
sprintf(msg, g_Locale->getText(LOCALE_FLASHUPDATE_MSGBOX), versionInfo->getType(true), versionInfo->getVersionString(), versionInfo->getDate(), versionInfo->getTime()); sprintf(msg, g_Locale->getText(LOCALE_FLASHUPDATE_MSGBOX), versionInfo.getType(true), versionInfo.getVersionString(), versionInfo.getDate(), versionInfo.getTime());
if (gotImage) if (gotImage)
{ {
@@ -431,7 +431,6 @@ bool CFlashUpdate::checkVersion4Update()
if ((strncmp(PACKAGE_RELEASE_CYCLE, versionInfo->getReleaseCycle(), 2) != 0) && if ((strncmp(PACKAGE_RELEASE_CYCLE, versionInfo->getReleaseCycle(), 2) != 0) &&
(ShowMsg(LOCALE_MESSAGEBOX_INFO, LOCALE_FLASHUPDATE_WRONGBASE, CMsgBox::mbrYes, CMsgBox::mbYes | CMsgBox::mbNo, NEUTRINO_ICON_UPDATE) != CMsgBox::mbrYes)) (ShowMsg(LOCALE_MESSAGEBOX_INFO, LOCALE_FLASHUPDATE_WRONGBASE, CMsgBox::mbrYes, CMsgBox::mbYes | CMsgBox::mbNo, NEUTRINO_ICON_UPDATE) != CMsgBox::mbrYes))
{ {
delete versionInfo;
return false; return false;
} }
#endif #endif
@@ -439,12 +438,9 @@ bool CFlashUpdate::checkVersion4Update()
if ((fileType != '0' /*Release*/) && if ((fileType != '0' /*Release*/) &&
(ShowMsg(LOCALE_MESSAGEBOX_INFO, LOCALE_FLASHUPDATE_EXPERIMENTALIMAGE, CMsgBox::mbrYes, CMsgBox::mbYes | CMsgBox::mbNo, NEUTRINO_ICON_UPDATE) != CMsgBox::mbrYes)) (ShowMsg(LOCALE_MESSAGEBOX_INFO, LOCALE_FLASHUPDATE_EXPERIMENTALIMAGE, CMsgBox::mbrYes, CMsgBox::mbYes | CMsgBox::mbNo, NEUTRINO_ICON_UPDATE) != CMsgBox::mbrYes))
{ {
delete versionInfo;
return false; return false;
} }
} }
delete versionInfo;
} }
else else
{ {