fixes #3738 (CrashDumper: ошибочный показ активного журнала)

5 files changed, 23 insertions, 32 deletions

diff --git a/src/mir_app/src/chat.h b/src/mir_app/src/chat.h
index 6ba50be865..3f4fcc8dda 100644
--- a/src/mir_app/src/chat.h
+++ b/src/mir_app/src/chat.h
@@ -129,8 +129,6 @@ BOOL          DoTrayIcon(SESSION_INFO *si, GCEVENT *gce);
 BOOL          DoPopup(SESSION_INFO *si, GCEVENT *gce);

 int           ShowPopup(MCONTACT hContact, SESSION_INFO *si, HICON hIcon, char* pszProtoName, wchar_t* pszRoomName, COLORREF crBkg, const wchar_t* fmt, ...);

 

-CSrmmLogWindow *Srmm_GetLogWindow(CMsgDialog *pDlg);

-

 void          Chat_EventToGC(SESSION_INFO *si, MEVENT hDbEvent);

 void          Chat_RemoveContact(MCONTACT hContact);

 

diff --git a/src/mir_app/src/mir_app.def b/src/mir_app/src/mir_app.def
index 8359cb841b..fb83d97f00 100644
--- a/src/mir_app/src/mir_app.def
+++ b/src/mir_app/src/mir_app.def
@@ -907,3 +907,4 @@ Clist_GroupSaveExpanded @1003 NONAME
 ?GetInput@CSrmmBaseDialog@@QBEPAUHWND__@@XZ @1024 NONAME

 ?Srmm_ApplyOptions@@YGXXZ @1025 NONAME

 ?OnOptionsApplied@CSrmmBaseDialog@@UAEXXZ @1026 NONAME

+?Srmm_GetWindowClass@@YGPAUSrmmLogWindowClass@@PAVCMsgDialog@@@Z @1027 NONAME

diff --git a/src/mir_app/src/mir_app64.def b/src/mir_app/src/mir_app64.def
index 7cedefa3d5..d977ff1183 100644
--- a/src/mir_app/src/mir_app64.def
+++ b/src/mir_app/src/mir_app64.def
@@ -907,3 +907,4 @@ Clist_GroupSaveExpanded @1003 NONAME
 ?GetInput@CSrmmBaseDialog@@QEBAPEAUHWND__@@XZ @1024 NONAME

 ?Srmm_ApplyOptions@@YAXXZ @1025 NONAME

 ?OnOptionsApplied@CSrmmBaseDialog@@UEAAXXZ @1026 NONAME

+?Srmm_GetWindowClass@@YAPEAUSrmmLogWindowClass@@PEAVCMsgDialog@@@Z @1027 NONAME

diff --git a/src/mir_app/src/srmm_base.cpp b/src/mir_app/src/srmm_base.cpp
index 6176b753d1..d8572c2700 100644
--- a/src/mir_app/src/srmm_base.cpp
+++ b/src/mir_app/src/srmm_base.cpp
@@ -519,11 +519,14 @@ bool CSrmmBaseDialog::OnInitDialog()
 	WindowList_Add(g_hWindowList, m_hwnd, m_hContact);

 	SetWindowLongPtr(m_hwnd, GWLP_USERDATA, (LONG_PTR)this);

 

-	m_pLog = Srmm_GetLogWindow((CMsgDialog*)this);

-	m_pLog->Attach();

-	if (m_pLog->GetType() != 0) { // custom log type

-		DestroyWindow(GetDlgItem(m_hwnd, IDC_SRMM_LOG));

-		SetWindowLong(m_pLog->GetHwnd(), GWLP_ID, IDC_SRMM_LOG);

+	auto *pDlg = (CMsgDialog *)this;

+	if (auto *pLogWindowClass = Srmm_GetWindowClass(pDlg)) {

+		m_pLog = pLogWindowClass->pfnBuilder(*pDlg);

+		m_pLog->Attach();

+		if (m_pLog->GetType() != 0) { // custom log type

+			DestroyWindow(GetDlgItem(m_hwnd, IDC_SRMM_LOG));

+			SetWindowLong(m_pLog->GetHwnd(), GWLP_ID, IDC_SRMM_LOG);

+		}

 	}

 

 	SetWindowLongPtr(m_message.GetHwnd(), GWLP_USERDATA, LPARAM(this));

diff --git a/src/mir_app/src/srmm_log.cpp b/src/mir_app/src/srmm_log.cpp
index b91204a59c..647304eb86 100644
--- a/src/mir_app/src/srmm_log.cpp
+++ b/src/mir_app/src/srmm_log.cpp
@@ -27,22 +27,7 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
 #include "stdafx.h"

 #include "chat.h"

 

-struct LoggerClass

-{

-	LoggerClass(CMPlugin *p1, const char *p2, const wchar_t *p3, pfnSrmmLogCreator p4) :

-		pPlugin(p1),

-		szShortName(mir_strdup(p2)),

-		wszScreenName(mir_wstrdup(p3)),

-		pfnBuilder(p4)

-	{}

-

-	CMPlugin *pPlugin;

-	ptrA szShortName;

-	ptrW wszScreenName;

-	pfnSrmmLogCreator pfnBuilder;

-};

-

-static OBJLIST<LoggerClass> g_arLogClasses(1, PtrKeySortT);

+static OBJLIST<SrmmLogWindowClass> g_arLogClasses(1, PtrKeySortT);

 

 static CMOption<char *> g_logger(SRMM_MODULE, "Logger", "built-in");

 

@@ -51,7 +36,7 @@ static CMOption<char *> g_logger(SRMM_MODULE, "Logger", "built-in");
 static bool sttEnableCustomLogs(CMsgDialog *pDlg)

 {

 	// always enable custom log viewers for private chats

-	if (!pDlg->isChat())

+	if (!pDlg || !pDlg->isChat())

 		return true;

 

 	// if custom log viewers are disable, use build-in one

@@ -79,21 +64,23 @@ static bool sttEnableCustomLogs(CMsgDialog *pDlg)
 	return true;

 }

 

-CSrmmLogWindow* Srmm_GetLogWindow(CMsgDialog *pDlg)

+MIR_APP_DLL(SrmmLogWindowClass *) Srmm_GetWindowClass(CMsgDialog *pDlg)

 {

 	if (sttEnableCustomLogs(pDlg)) {

-		CMStringA szViewerName(db_get_sm(pDlg->m_hContact, SRMSGMOD, "Logger"));

+		CMStringA szViewerName;

+		if (pDlg != nullptr)

+			szViewerName = db_get_sm(pDlg->m_hContact, SRMSGMOD, "Logger");

 		if (szViewerName.IsEmpty())

 			szViewerName = g_logger;

 

 		for (auto &it : g_arLogClasses)

 			if (szViewerName == it->szShortName)

-				return it->pfnBuilder(*pDlg);

+				return it;

 	}

 

 	for (auto &it : g_arLogClasses)

 		if (!mir_strcmp(it->szShortName, "built-in"))

-			return it->pfnBuilder(*pDlg);

+			return it;

 

 	return nullptr; // shall never happen

 }

@@ -141,9 +128,10 @@ public:
 	{

 		pDialog = this;

 

+		auto *pClass = Srmm_GetWindowClass(0);

 		for (auto &it : g_arLogClasses) {

 			int idx = m_list.AddString(TranslateW_LP(it->wszScreenName, it->pPlugin), LPARAM(it));

-			if (!mir_strcmp(g_logger, it->szShortName))

+			if (it == pClass)

 				m_list.SetCurSel(idx);

 		}

 

@@ -156,7 +144,7 @@ public:
 		if (idx == -1)

 			return false;

 

-		if (auto *pLogger = (LoggerClass *)m_list.GetItemData(idx))

+		if (auto *pLogger = (SrmmLogWindowClass *)m_list.GetItemData(idx))

 			g_logger = pLogger->szShortName;

 

 		PostMessage(m_hwndParent, WM_CLOSE, 1, 0);

@@ -199,7 +187,7 @@ MIR_APP_DLL(HANDLE) RegisterSrmmLog(CMPlugin *pPlugin, const char *pszShortName,
 	if (!pszShortName || !pwszScreenName || !fnBuilder)

 		return nullptr;

 

-	auto *p = new LoggerClass(pPlugin, pszShortName, pwszScreenName, fnBuilder);

+	auto *p = new SrmmLogWindowClass(pPlugin, pszShortName, pwszScreenName, fnBuilder);

 	g_arLogClasses.insert(p);

 

 	if (pDialog)

@@ -209,7 +197,7 @@ MIR_APP_DLL(HANDLE) RegisterSrmmLog(CMPlugin *pPlugin, const char *pszShortName,
 

 MIR_APP_DLL(void) UnregisterSrmmLog(HANDLE pLogger)

 {

-	g_arLogClasses.remove((LoggerClass *)pLogger);

+	g_arLogClasses.remove((SrmmLogWindowClass *)pLogger);

 

 	if (pDialog)

 		pDialog->Rebuild();