qt-faststart: Fix the signedness of variables keeping the ftello return values
authorMichael Niedermayer <michaelni@gmx.at>
Mon, 29 Oct 2012 21:05:33 +0000 (22:05 +0100)
committerMartin Storsjö <martin@martin.st>
Fri, 28 Feb 2014 22:24:55 +0000 (00:24 +0200)
These variables are assigned the return values of ftello, which
returns an off_t, which is a signed type. On errors, ftello returns
-1, thus make sure this error return value can be stored properly.

Signed-off-by: Martin Storsjö <martin@martin.st>
tools/qt-faststart.c

index 0abcd2b..2b2e00c 100644 (file)
@@ -89,7 +89,7 @@ int main(int argc, char *argv[])
     uint32_t atom_type   = 0;
     uint64_t atom_size   = 0;
     uint64_t atom_offset = 0;
-    uint64_t last_offset;
+    int64_t last_offset;
     unsigned char *moov_atom = NULL;
     unsigned char *ftyp_atom = NULL;
     uint64_t moov_atom_size;
@@ -97,7 +97,7 @@ int main(int argc, char *argv[])
     uint64_t i, j;
     uint32_t offset_count;
     uint64_t current_offset;
-    uint64_t start_offset = 0;
+    int64_t start_offset = 0;
     unsigned char copy_buffer[COPY_BUFFER_SIZE];
     int bytes_to_copy;