Attached gif downloaded from telegram takes out the phone as thumbnailer hogs the memory and cpu
Bug #1527315 reported by
Dave Morley
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Canonical System Image |
Fix Released
|
High
|
Alejandro J. Cura | ||
thumbnailer (Ubuntu) |
Fix Released
|
Critical
|
Michi Henning |
Bug Description
STEPS:
1. drop the attached gif into the Photo's folder
2. Open gallery
3. Refresh the photo's scope
ISSUE:
The gif I think causes thumbnailer to loop, Memory usage jump 4% each loop until oom takes out the app but thumbnailer is still stuck in it's loop so eventually the scopes are all taken out, At that point thumbnailer is stopped and the system returns to normal with the exception that there is now a black space where the scopes used to be.
IMAGE:
Krillin ota 8.5
Related branches
lp:~michihenning/thumbnailer/animated-gifs
- PS Jenkins bot (community): Approve (continuous-integration)
- Michi Henning (community): Approve
-
Diff: 117 lines (+45/-0)3 files modifieddebian/changelog (+2/-0)
src/thumbnailer.cpp (+24/-0)
tests/thumbnailer/thumbnailer_test.cpp (+19/-0)
lp:~michihenning/thumbnailer/fix-1527315
- Michi Henning (community): Approve
- PS Jenkins bot (community): Approve (continuous-integration)
-
Diff: 161 lines (+55/-1)5 files modifieddebian/changelog (+8/-0)
src/thumbnailer-admin/show_stats.cpp (+1/-1)
src/thumbnailer.cpp (+24/-0)
tests/thumbnailer-admin/thumbnailer-admin_test.cpp (+3/-0)
tests/thumbnailer/thumbnailer_test.cpp (+19/-0)
lp:~jamesh/thumbnailer/gif-first-frame
- Michi Henning (community): Approve
- PS Jenkins bot (community): Approve (continuous-integration)
-
Diff: 195 lines (+68/-40)4 files modifiedsrc/image.cpp (+25/-4)
src/thumbnailer.cpp (+0/-24)
tests/image/image_test.cpp (+37/-0)
tests/thumbnailer/thumbnailer_test.cpp (+6/-12)
tags: | added: qa-dogfood |
Changed in canonical-devices-system-image: | |
status: | New → Confirmed |
importance: | Undecided → High |
milestone: | none → ww02-2016 |
Changed in thumbnailer (Ubuntu): | |
importance: | Undecided → High |
assignee: | nobody → Michi Henning (michihenning) |
Changed in canonical-devices-system-image: | |
assignee: | nobody → Michi Henning (michihenning) |
Changed in canonical-devices-system-image: | |
assignee: | Michi Henning (michihenning) → Alejandro J. Cura (alecu) |
Changed in thumbnailer (Ubuntu): | |
status: | New → In Progress |
Changed in canonical-devices-system-image: | |
status: | Confirmed → Fix Committed |
Changed in canonical-devices-system-image: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
That's a pretty stupid way to make a video. A real video would probably be a lot small and have better quality :-)
But I agree that we shouldn't fall over just because someone feeds us a file like this. I'll have a look at what's going on.