mcxxx / cassia

Automatically exported from code.google.com/p/cassia
0 stars 0 forks source link

WTSSendMessage does not support a length of 0 for the title of the text box. #35

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1.Connect to a server
2.use ITerminalServicesSession.MessageBox(string text)
or
2. use ITerminalServicesSession.MessageBox(string text, string caption) and
have caption = to "" or null;

What is the expected output? What do you see instead?
Received a error code of -2147467259 ("The stub received bad data") from
line 132 in NativeMethodsHelper.cs

What version of the product are you using? On what operating system?
Cassia 2.0.50727.1433 Server is Server 2003 x64

Can replicate issue by using the send message feature in task manager on
the server, this issue appears to be a windows issue not a cassia issue.

Attached is a edited version of the source to correct issue. uses " "
instead of "" for a null title, also checks for empty titles.

Please provide any additional information below.

Original issue reported on code.google.com by left...@gmail.com on 5 May 2010 at 7:54

Attachments:

GoogleCodeExporter commented 9 years ago
I grabbed the wrong version number. I am using cassia 2.0.0.60

Original comment by left...@gmail.com on 5 May 2010 at 8:04

GoogleCodeExporter commented 9 years ago
Thanks for reporting this, tracking down the problem, and providing a patch, 
leftler! 
Much appreciated. I'll try to reproduce this, and if I can, I'll apply your 
patch.

Original comment by danports on 5 May 2010 at 11:16

GoogleCodeExporter commented 9 years ago
Reproduced on Server 2003 (32-bit) with Cassia trunk and with Task Manager. No 
problems 
on 7 or Server 2008 R2.

Original comment by danports on 5 May 2010 at 11:31

GoogleCodeExporter commented 9 years ago
Committed in r96. In a few minutes, you should be able to grab the latest build 
from 
the CI server 
(http://teamcity.codebetter.com/project.html?projectId=project51), or you 
can svn update and compile the changes yourself. Thanks again for reporting 
this.

Original comment by danports on 5 May 2010 at 11:53