Summary: | Movie in pdf is not played if movie file path contains ./ | ||
---|---|---|---|
Product: | [Applications] okular | Reporter: | Oliver Sander <oliver.sander> |
Component: | PDF backend | Assignee: | Okular developers <okular-devel> |
Status: | RESOLVED FIXED | ||
Severity: | normal | ||
Priority: | NOR | ||
Version: | 0.14.95 | ||
Target Milestone: | --- | ||
Platform: | Compiled Sources | ||
OS: | Linux | ||
Latest Commit: | http://commits.kde.org/okular/06b0cf86169a92802ff4e4df0e9d22bc67b387fb | Version Fixed In: | |
Sentry Crash Report: | |||
Attachments: |
pdf containing the movie that won't play
Corresponding tex sources Example movie file Patch to use KTemporaryFile |
Description
Oliver Sander
2012-07-09 09:13:10 UTC
Created attachment 72395 [details]
pdf containing the movie that won't play
Created attachment 72396 [details]
Corresponding tex sources
Created attachment 72397 [details]
Example movie file
Git commit 06b0cf86169a92802ff4e4df0e9d22bc67b387fb by Tobias Koenig. Committed on 11/07/2012 at 12:24. Pushed by tokoe into branch 'master'. Always create temporary movie files with a sane name M +2 -1 core/movie.cpp http://commits.kde.org/okular/06b0cf86169a92802ff4e4df0e9d22bc67b387fb Thanks, the movie plays now. However, looking at the patch suggests a few ways to create other non-playing situations. For example, having the filename start with ././ makes the movie not play again (admittedly, this is a bit contrived). But less contrived: when I specify the movie file by absolute path okular won't play it either. Created attachment 72456 [details]
Patch to use KTemporaryFile
What about a plain KTemporaryFile?
Note that, with the attached diff, the extension is not preserved any more. However, it seems to work even without it.
For the record: Tobias fixed this properly in commit 4c8f0bc5def7c755ab810adec78826fd51f713b1 Thanks! |