Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

INI: Add support for new builds (fix #611) #612

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

INI: Add support for new builds (fix #611) #612

wants to merge 2 commits into from

Conversation

warezio
Copy link

@warezio warezio commented Dec 14, 2018

  • 6.1.7601.24308 x86

fre4kyC0de
fre4kyC0de suggested changes Dec 14, 2018
View reviewed changes
README.md
@@ -297,6 +297,7 @@ Visit [issues](https://github.com/stascorp/rdpwrap/issues) section, and check wh
- 6.1.7601.22843 (Windows 7 SP1 with KB3003743 LDR)
- 6.1.7601.23403 (Windows 7 SP1 with KB3125574)
- 6.1.7601.24234 (Windows 7 SP1 with KB4462923)
- 6.1.7601.24308 (Windows 7 SP1 with KB4471328)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This build does not exist...

You should add 10.0.17763.165/168 instead

technical.txt
@@ -44,6 +44,7 @@ Terminal Services supported versions
6.1.7601.22843 (Windows 7 SP1 with KB3003743 LDR) [policy hook + extended patch]
6.1.7601.23403 (Windows 7 SP1 with KB3125574) [policy hook + extended patch]
6.1.7601.24234 (Windows 7 SP1 with KB4462923) [policy hook + extended patch]
6.1.7601.24308 (Windows 7 SP1 with KB4471328) [policy hook + extended patch]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This build does not exist...

You should add 10.0.17763.165/168 instead

fre4kyC0de
fre4kyC0de reviewed Dec 14, 2018
View reviewed changes
technical.txt
@@ -167,6 +167,9 @@ Terminal Services supported versions

Source code changelog (rdpwrap library):

2018.12.13 :
- added support for termsrv.dll 6.1.7601.24308 x86
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This termsrv.dll version 6.1.7601.24308 does not exist...

You should add 10.0.17763.165/168 instead

@jaxjexjox
Copy link

I tried your ini file, my entire RDP subsystem died on reboot, suggest this is well tested before pushing out.

Unless I somehow managed to copy and paste text from an ini file incorrectly.

@mickael9 mickael9 mentioned this pull request Dec 27, 2018
Open
@Macrisss
Copy link

Macrisss commented Jan 2, 2019

[*] Terminal Services version: 10.0.17763.168
[-] This version of Terminal Services is not supported.
Please add!

@perlestius
Copy link

perlestius commented Jan 13, 2019 •
edited
Loading

rdpwrap.zip, based on original ini-file, which was edited according to very useful comment by @hajubu
It works for me! (Win10 x64 1809, build 17763.253, 17763.168 version of termsrv.dll)

@RostyslavBaidak
Copy link

Hello. Help me please. I'm used RDPWrap 1.6.2
rdpwraperror2
windows ver

@perlestius
Copy link

@RostyslavBaidak, look here

@RostyslavBaidak
Copy link

@perlestius

  1. versions do not match
  2. I do not quite understand the way to solve the problem.

@perlestius
Copy link

perlestius commented Mar 9, 2019 •
edited
Loading

  1. versions do not match
    Какие именно версии не совпадают? Судя по вашему скриншоту, версия termsrv.dll.dll у Вас 10.0.17763.292. По ссылке, которую я Вам дал выше, даны рекомендации именно для этой версии.
  2. I do not quite understand the way to solve the problem.
    Логинитесь на машину консольно (т.е. НЕ по RDP). Правите файл C:\Program Files\RDP Wrapper\rdpwrap.ini согласно инструкциям по ссылке выше, после чего перезапускаете службы удаленных рабочих столов (TermService). Или можете взять уже исправленный файл из вложения к этому сообщению.

@RostyslavBaidak
Copy link

@perlestius спасибо большое, помог и сохранил кучу моего времени. Проблема была в том, что я подключался к машине не консольно, а по RDP и + я не перезапускал службу удаленного доступа. Еще раз спасибо.

@RostyslavBaidak
Copy link

Доброго времени суток! Wrapper отлично работал, пока не обновился Windows.
10 0 17763 437
Возможно решить данную проблему с версией RDP Wrapper 1.6.2? Если да, подскажите пожалуйста.

@perlestius
Copy link

@RostyslavBaidak
Тут гляньте.
И на будущее: перед тем, как задавать вопрос, изучите внимательнее раздел "Issues" - там добрые люди выкладывают актуальные ini-файлы.

@RostyslavBaidak
Copy link

@perlestius спасибо за помощь. В следующий раз буду более внимательней к поиску решения проблемы.

@764763903a
Copy link

image
希望添加对这个版本的更新

@hajubu
Copy link

hajubu commented Apr 16, 2019 via email

@764763903a
Copy link

Hi @764763903a You should have a look to issue #735 for x64 #721 for x86 There the guys are discussing the Version 17134.706 Von: 764763903a Gesendet: Dienstag, 16. April 2019 14:44 An: stascorp/rdpwrap Cc: hajubu; Mention Betreff: Re: [stascorp/rdpwrap] INI: Add support for new builds (fix #611)(#612) 希望添加对这个版本的更新 — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

谢谢我看到了我已经成功修改了

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fre4kyC0de fre4kyC0de fre4kyC0de requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@warezio @jaxjexjox @Macrisss @perlestius @RostyslavBaidak @764763903a @hajubu @fre4kyC0de