Bug #3639
shotwell crashed with SIGABRT in resources_add_stock_icon()
| Status: | Fixed | Start date: | 05/21/2011 | |
|---|---|---|---|---|
| Priority: | High | Due date: | ||
| Assignee: | % Done: | 0% |
||
| Category: | - | |||
| Target version: | 0.12 | |||
| Keywords: |
Description
I believe it was a random crash for the reporter on launchpad though the description said
“Crashed while an update in packages was being performed.”
origin: https://bugs.launchpad.net/ubuntu/+source/shotwell/+bug/746772
Associated revisions
Closes #3639, prevents the app from crashing if it fails to load a stock icon (warnings about the missing icon will appear in the console; this is by design).
History
Updated by Omer Akram about 2 years ago
Version: shotwell 0.9.0-0ubuntu2
Updated by Lucas Beeler over 1 year ago
- Description updated (diff)
In general, adding stock icons while a system update is going on (and potentially rebuilding the system stock items cache) will lead to undefined behavior. I don't know how much we can do about this. Chime in if you'd like this ticket to survive, otherwise I'll mark it as wontfix on October 1.
Updated by Jim Nelson over 1 year ago
- Target version set to 0.12
Further examination indicates that the shotwell-crop stock icon file was being loaded and an I/O error occurred, possibly because of the update removing the old file and installing a new one (possible, since we've renamed icon files in the past). Either way, looking at the code, there's two things we can do here:
1. Issue a warning rather than an error. Shotwell crashed because the icon file was not found, but an icon is not critical to Shotwell's operation.
2. Use a better warning message. The current message simply dumped the GLib error message out, which we didn't see because the log file wasn't included, making it more difficult to read the stack trace. Let's add some descriptive text here as well as print the GLib error.
This is a simple fix we can do in 0.12 to get this one off the books.
Updated by Lucas Beeler over 1 year ago
- Description updated (diff)
- Assignee set to Clinton Rogers
Updated by Clinton Rogers over 1 year ago
- Status changed from Open to 5
- Resolution set to fixed
f348dd8711a86de5fe7f6bb7b4f72fbbd54d37cd
We now complain in the console if we can't load the stock icon we wanted, but we continue running.
Updated by Charles Lindsay about 1 month ago
- Status changed from 5 to Fixed