🚀 go-pugleaf

> TClientSocket *sock = new TClientSocket(MainForm);

I always use

TClientSocket *sock = new TClientSocket(NULL);

to create client sockets inside a thread. You yourself noticed 
this is the right way.

> My theory as to why I would get AVs is that maybe all three 
> threads are calling new TClientSocket(MainForm) almost 
> simultaneously and basically,

I don't care about that. Why should MainForm be the owner of an 
object created inside a different thread?

> The biggest hole in my theory that I can see is that it 
> doesn't really explain why it would work on my PC and not my 
> clients.

That should be deeply checked, but again, you shouldn't care too 
much IMO.

> It still could be a timing issue,

Or another thousands of issues...

> Also, the only advantage that I can see to having MainForm 
> being the owner of the TClientSocket is that if something 
> unexpected occurs and the TClientSocket doesn't get deleted 
> somehow, the MainForm will free up the memory when the form 
> gets destroyed.

Not a big advantage. Almost a disadvantage under certain points 
of view.

> This advantage is greatly decreased by the fact that the form 
> is never destroyed until the program exits, in which case 
> Windows is going to free up the memory anyway.

Anyway, your thread should be strong enough to handle any 
possible problem and free the TClientSocket pointer by itself.

Did you find anything more in the meantime?

HTH,

Stefano.