GNOME Bugzilla – Bug 308662
Script-FU Section crashes on 'OldPhoto'-abbortion
Last modified: 2008-01-15 12:57:15 UTC
Start script-fu->deco->old photo and while processing close the script-window. The whole script-fu-section crashes -> no script-fu-menu available anymore
Closing the main dialog associated with a running script is probably not the nicest thing to do. Once the dialog has been closed, another dialog pops up with comments about the plug-in may have died and the number of times the message has been repeated. The warning message seems to be appearing for each attempt to update the progress bar (which is a lot of times). The script will run to completion and the updated image will appear but script-fu dies with a segmentation fault. I wouldn't be surprised to learn that this problem can be reproduced by closing most (all?) of the other scripts which use the progress bar. It shouldn't be too hard to track down what is causing the seg fault.
Closing the window is equivalent to pressing the Cancel button in the dialog. I might be wrong but I think the effect of pressing Cancel is to kill Script-Fu. So this would be a duplicate of bug #109605.
Eeek, actually it appears to be gimp that is crashing here, not Script-Fu: gimp-2.3: fatal error: Segmentation fault gimp-2.3 (pid:3151): [E]xit, [H]alt, show [S]tack trace or [P]roceed: s
+ Trace 61426
It's simple to prohibit further use of the progress pdb callback after it has failed once. I added two lines of code to app/pdb/gimppdbprogress.c and it stops the warning to be repeated over and over again. But since the script continues to run w/o any progress indication then, that is probably not the right fix or only part of it.
The cancel button is insensitive while the script is running, so the right thing to do would be to ignore closing the window as well.
Fixed in both branches: 2005-09-11 Michael Natterer <mitch@gimp.org> * plug-ins/script-fu/script-fu-interface.c (script_fu_response): bail out if the action_area is insensitive. Ignores attempts to close the window while the script is running. Fixes bug #308662.