C# RDP not connecting after forloop

Question

I am currently using Microsoft Terminal Services Client what works amazingly and does what i want it to do. But i have ran into a small issue what is proving to be difficult to fix. I am trying to connect to the rdp and then constantly scan if a file exist. But when i do the for loop (After it call the rdp to connect) it doesn't connect. Here is my code:

                axMsTscAxNotSafeForScripting1.Server = "0.0.0.0";
                axMsTscAxNotSafeForScripting1.UserName = "Test";
                IMsTscNonScriptable secured = (IMsTscNonScriptable)axMsTscAxNotSafeForScripting1.GetOcx();
                secured.ClearTextPassword = "c";
                axMsTscAxNotSafeForScripting1.Connect();
                Thread.Sleep(2000);
                for(; ; )
                {
                    if (File.Exists(b + "t.txt"))
                    {
                        MessageBox.Show("File Exists");
                    }
                }

score 0 · Answer 1 · answered Jan 31 '19 at 15:21

0

I don't think that the for loops affect connection. But, you are creating an infinite loop which does not terminates in any circumstance. Which means you are blocking the Executing thread.

answered Jan 31 '19 at 15:21

Derviş Kayımbaşıoğlu

28,492
4
50
72

When i remove the forloop it works perfectly, Thats what i dont understand... And the forloop is after i connect what is strange – Best Exploits Jan 31 '19 at 15:23
I explained to you, you are blocking thread with infinite loop. – Derviş Kayımbaşıoğlu Jan 31 '19 at 16:05
But the thread is before the infinite loop so shouldn't it execute that first? Also is there a way i can prevent the thread from blocking? – Best Exploits Jan 31 '19 at 16:10
You are not opening thread. You are in the same thread. So the code starts executing rdp then you get into infinite loop – Derviş Kayımbaşıoğlu Jan 31 '19 at 16:17
I tried adding a thread.sleep(2000) That didnt combat the issue. Do you have any suggestions? – Best Exploits Jan 31 '19 at 16:21

Doctor Jones · Accepted Answer · 2019-01-31T16:47:59.230

0

You shouldn't use a loop to poll for the existence of the file, that's what FileSystemWatcher is for. You'll need to do something like this.

var watcher= new FileSystemWatcher(b);
watcher.Filter = "t.txt";
watcher.Created += (sender, eventArgs) => MessageBox.Show("File Exists");

I've made some assumptions about what's in your b variable, but the point is, the watcher will watch a directory for changes, set the filter for the types of files you're looking for (it will accept wildcards).

If you want to prevent the program from exiting, don't use an infinite loop, just use Console.ReadKey or Console.ReadLine so you're not doing a busy wait.

Console.WriteLine("Press any key to exit.");
Console.ReadKey();

edited Jan 31 '19 at 16:47

answered Jan 31 '19 at 16:34

Doctor Jones

21,196
13
77
99

Your sir saved me! Thank you very much. I didnt know this was a method.. Thanks! – Best Exploits Jan 31 '19 at 16:43
Line 3 is saying isnt working, "A local parameter named "e" cannot be declared" – Best Exploits Jan 31 '19 at 16:45
Hmmm, I'd guess you might already have a variable named `e`, I've renamed it in the example code, but any other name will do, as long as it doesn't clash with an existing variable. – Doctor Jones Jan 31 '19 at 16:48
Now im getting sender cannot be declared. Im sorry for bugging you so much but really appreciate the help – Best Exploits Jan 31 '19 at 16:53
You'll need to rename that variable too, they're not used, so any unique name will do. – Doctor Jones Jan 31 '19 at 17:02
If your version of C# supports it, you can use discard parameters and just declare them as `(_, _)` – Doctor Jones Jan 31 '19 at 17:03
Ive updated my question because when using filesystemwatcher i run into a new error. Please re-check my question above. Thanks – Best Exploits Jan 31 '19 at 18:15

C# RDP not connecting after forloop

2 Answers2