Update: UltraVNC 1.4.3.6 and UltraVNC SC 1.4.3.6: viewtopic.php?t=37885
Important: Please update to latest version before to create a reply, a topic or an issue: viewtopic.php?t=37864

Join us on social networks and share our announcements:
- Website: https://uvnc.com/
- GitHub: https://github.com/ultravnc
- Mastodon: https://mastodon.social/@ultravnc
- Facebook: https://www.facebook.com/ultravnc1
- X/Twitter: https://twitter.com/ultravnc1
- Reddit community: https://www.reddit.com/r/ultravnc
- OpenHub: https://openhub.net/p/ultravnc

V1.23.1 problems

Post Reply
thegaffer14
Posts: 5
Joined: 2019-09-06 14:07

V1.23.1 problems

Post by thegaffer14 »

V1.23.1 seems to have several problems. MRU and the SessionDialog combo box doesn't function correctly.

1) MRU::AddItem needs move the trailing 0 as well
for (int j = _tcslen(m_index)+1; j >= 0; j--) instead of for (int j = _tcslen(m_index); j >= 0; j--)
then
m_index[MRU_MAX_ITEM_LENGTH] = _T('\0');
after the loop

2) The SessionDialog combo box message handler needs to change in order to execute a user selection and display it
case IDC_HOSTNAME_EDIT:
if (HIWORD(wParam) == CBN_SELCHANGE) {
TCHAR hostname[256];
int ItemIndex = SendMessage((HWND) lParam, CB_GETCURSEL, 0, 0);
SendMessage((HWND) lParam, (UINT) CB_GETLBTEXT, (WPARAM) ItemIndex, (LPARAM) hostname);
_this->m_pMRU->RemoveItem(hostname);
_this->m_pMRU->AddItem(hostname);
_this->IfHostExistLoadSettings(hostname);
_this->SettingsToUI();
}
if (HIWORD(wParam) == CBN_SELENDOK) {
TCHAR hostname[256];
int ItemIndex = SendMessage((HWND)lParam, CB_GETCURSEL, 0, 0);
SendMessage((HWND)lParam, (UINT)CB_SETCURSEL, (WPARAM)ItemIndex, (LPARAM)hostname);
}
break;


3) Saved settings with file names like this sprintf_s(fname, "%.10s-%d.vnc", tmphost, (disp > 0 && disp < 100) ? disp : port);
will cause problems as different servers on the same network will be saved to the same file.

4) Running viewer debugging (either x86 or x64) with encryption causes exceptions due to heap corruption.

P.S also
SessionDialog::InitDlgProcSecurity() needs

// Update the use encryption check box
HWND hfUseEncryption = GetDlgItem(hwnd, IDC_PLUGIN_CHECK);
SendMessage(hfUseEncryption, BM_SETCHECK, fUseDSMPlugin, 0);
Last edited by thegaffer14 on 2019-09-11 12:58, edited 1 time in total.
User avatar
Rudi De Vos
Admin & Developer
Admin & Developer
Posts: 6831
Joined: 2004-04-23 10:21
Contact:

Re: V1.23.1 problems

Post by Rudi De Vos »

missed this post gonna check it
User avatar
Rudi De Vos
Admin & Developer
Admin & Developer
Posts: 6831
Joined: 2004-04-23 10:21
Contact:

Re: V1.23.1 problems

Post by Rudi De Vos »

heap corruption
https://github.com/veyon/ultravnc/blob/ ... .cpp#L3602
Was set to small using the plugin we add the plugin name to the titlebar
thegaffer14
Posts: 5
Joined: 2019-09-06 14:07

Re: V1.23.1 problems

Post by thegaffer14 »

Thanks that seems to have fixed it
Post Reply