Zugriffsverletzung in System.Classes.MakeObjectInstance

[Scurra]

Hallo zusammen,

nach einem Update einer Software, die wir in unserem Unternehmen entwickeln erhalten wir regelmäßig Fehlerberichte wegen einer Zugriffsverletzung in der Funktion System.Classes.MakeObjectInstance.

Eine Gemeinsamkeit in allen Fehlerberichten ist, dass die Benutzer zuerst eine Aktion durchführen, bei der wir Thumbnails von Bildern mit Hilfe der Komponente ImageEn erzeugen. Wenn später an einer anderen Stelle im Code ein Formular/Dialog erzeugt wird (das können verschiedene Formulare sein), dann tritt gelegentlich die Zugriffsverletzung laut Callstack (siehe unten) im Konstruktor von TCustomForm auf. In dem Update haben wir bei der Thumbnail-Generierung größere Änderungen vorgenommen. Beispielsweise haben wir die Generierung der Thumbnails mit Hilfe von Threads parallelisiert, so dass die Thumbnails von mehreren Bildern gleichzeitig erstellt werden können. Das legt natürlich den Verdacht nahe, dass die Threads irgendwie zu dem Problem führen. Zu dem Zeitpunkt, an dem der Fehler auftritt, sind die Threads aber schon beendet und freigegeben.

Im Internet habe ich leider keineInformationen über die Funktion MakeObjectInstance gefunden, die mir weiter helfen könnten. Hat jemand einen Anhaltspunkt für mich, welche Umstände zu der Zugriffsverletzung führen können? Ich tappe im Moment im Dunkeln und weiß nicht so recht, wie ich den Fehler beheben kann. Dass ich den Fehler nicht reproduzieren kann, macht die Sache noch schwerer.

Hier noch der Callstack ausgehend von der Stelle, an der wir ganz einfach mit TMyForm.Create(Owner) ein Formular erzeugen.

[004FDDC8] System.Classes.MakeObjectInstance
[00648F15] Vcl.Controls.TWinControl.Create
[00651FDF] Vcl.Controls.TCustomControl.Create
[006DD6AE] Vcl.ExtCtrls.TCustomPanel.Create
[004F0C41] System.Classes.CreateComponent
[004F0E6C] System.Classes.TReader.ReadComponent
[00407474] System._FreeMem
[0040C3A8] System._UStrClr
[004F1141] System.Classes.TReader.ReadDataInner
[004F1080] System.Classes.TReader.ReadData
[004FBCB9] System.Classes.TComponent.ReadState
[00644B66] Vcl.Controls.TControl.ReadState
[006492BD] Vcl.Controls.TWinControl.ReadState
[00755271] Vcl.Forms.TCustomForm.ReadState
[00477EFA] System.Generics.Collections.TListHelper.InternalAd d4
[004F2031] System.Classes.TReader.ReadRootComponent
[004EC1AA] System.Classes.TStream.ReadComponent
[004E3D83] System.Classes.InternalReadComponentRes
[004E5493] System.Classes.InitComponent
[004E5521] System.Classes.InitInheritedComponent
[00754A7E] Vcl.Forms.TCustomForm.Create

[Delphi.Narium]

Welches Format haben die Bilder und Thumbnails?

Zufällig Gif?

Da hab' ich mit ImageEn auch ab und an Zugriffsverletzungen.

Grund weiß ich nicht. Aber die Bilder, die bei ImageEn zu 'ner Zugriffsverletzung führen, lassen sich größtenteils auch nicht mit IrfanView, FireFox, ... anzeigen.

Mein Vermutung ist, dass es bei Gif-Bildern wohl Fehlermöglichkeiten gibt, die, mal mehr, mal weniger gut, abgefangen werden.

[Scurra]

Zitat:

Welches Format haben die Bilder und Thumbnails?

Es handelt sich um jpg-Dateien.

Ich habe auch mal versucht herauszufinden, ob ImageEn threadsicher ist, aber da ich einzelne Instanzen immer nur in einem Thread erzeuge, dort verwende und dann wieder freigebe, würde ich eher ausschließen, dass das zu Problemen führt. Natürlich könnte es sein, dass die Instanzen selbst auf globale Variablen zugreifen oder so, aber das fände ich sehr unüblich und selbst wenn es so wäre, wie könnte es dann sein, dass das zu Zugriffsverletzungen an einer ganz anderen Stelle führt?

[Scurra]

Zur Info: Ich habe die Implementierung nun so geändert, dass ich die TImageEn-Instanz nicht in den Threads erzeuge, sondern außerhalb in der Klasse, die die Threads verwaltet. Die Instanzen werden dann im Konstruktor der Threads übergeben. Das finde ich zwar vom Codedesign her nicht schön, aber seitdem habe ich den Fehler nicht mehr gesehen.
Scheint also auf jeden Fall so zu sein, dass TImageEn-Instanzen nicht in Threads erzeugt/freigegeben werden dürfen.

[Bernhard Geyer]

Kein Wunder das es kracht.
Deine erzeugten Instanzen werden Windows-Handles/Ressourcen haben. Und diese sind nur im erstellenden Thread gültig.

Erstellst du diese Instanzen also im Thread und später in der Anwendung (im Hauptthread) anzuzeigen, dann sind dir zufällige Zugriffsverletzungen gewiss.

[Scurra]

Zitat:

Erstellst du diese Instanzen also im Thread und später in der Anwendung (im Hauptthread) anzuzeigen, dann sind dir zufällige Zugriffsverletzungen gewiss.

Ich habe die Instanzen im Thread erzeugt, gegebene Bilddateien geladen, verkleinerte Bilder davon wieder auf die Festplatte gespeichert und anschließend freigegeben. Der Hauptthread wusste nichts von den TImageEn-Instanzen.