Du gibst z.B. IdSMTP nicht frei.
Generell solltest Du Dir angewöhnen IMMER Ressourcenschutzblöcke zu verwenden, wenn Du Instanzen von Klassen erstellst.
Delphi-Quellcode:
Instanz := TMyClass.Create;
Try
... mach was mit der Instanz
Finally
Instanz.free;
End