Fix the behaviour of FileGroup when seeking too far.

[dcpomatic.git] / src / lib / file_group.cc

diff --git a/src/lib/file_group.cc b/src/lib/file_group.cc
index 3e8a7b79c43177a8d775889ab91d08855bb0e77e..f06ca300718acd727c74b39fa09981d1185ffb55 100644 (file)
--- a/src/lib/file_group.cc
+++ b/src/lib/file_group.cc
@@ -132,12 +132,18 @@ FileGroup::seek (int64_t pos, int whence) const
                if (sub_pos < int64_t (len)) {
                        break;
                }
-               sub_pos -= len;
                ++i;
+               if (i < _paths.size()) {
+                       /* If we've run out of files we need to seek off the end of the last file */
+                       sub_pos -= len;
+               }
        }
 
        if (i == _paths.size ()) {
-               return -1;
+               /* Seeking too far isn't an error; we'll seek too far in the last file which
+                * will "pass on" fseek()'s behaviour to our caller.
+                */
+               i--;
        }
 
        ensure_open_path (i);