Ohne das jetzt genau zu analysieren, fehlt -denke ich mal- einfach die richtige Synchronisation. TCriticalSections sind wirklich sehr leichtgewichtig und sperren nur unterschiedliche Threads gegenseitig aus. Ein und der selbe Thread kann so oft er will, ein 'Acquire' aufrufen.

Dessenungeachtet würde ich trotzdem den an sich falschen Code (weil keine 'offiziellen' Synchronisationsmechanismen verwendet werden) durch korrekten Code ersetzen. Sprich: Neuschreiben. Alles andere ist doch Quatsch. Das ist Frickelcode und wird einfach nicht korrekt funktionieren.