SpritePropertyEditorDialog deletes the originalSpritePixmap pointer which makes it necessary to dynamically reinitialize spritePixmap for each object.

[keshav2010]

This is a poor implementation which makes it hard to keep track of references as they are passed to various objects as function arguments, and might lead to potential memory leak or segmentation fault issues. The one way to fix the issue is to keep the dialog alive for the entire lifetime of the program, i.e in memory instead of freeing up the used memory.