diff options
author | Michael Niedermayer <[email protected]> | 2012-10-25 00:39:33 +0200 |
---|---|---|
committer | Reinhard Tartler <[email protected]> | 2014-03-02 11:42:36 -0500 |
commit | b3f106cb1f0036ce54ead5b59120fed7d7aa11d7 (patch) | |
tree | e404991cc39f9aa13bf38b9e3c652f837ea8ee68 | |
parent | 298d66c8de1d4faa152f4ef557566aa0519223d8 (diff) |
qt-faststart: Check the ftello() return codes
This silences a warning in the coverity static analyzer.
Signed-off-by: Martin Storsjö <[email protected]>
(cherry picked from commit 63848854256a024a19435e87d6bc76fffa65e81e)
-rw-r--r-- | tools/qt-faststart.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/tools/qt-faststart.c b/tools/qt-faststart.c index 2b2e00ca57..5c511a0154 100644 --- a/tools/qt-faststart.c +++ b/tools/qt-faststart.c @@ -137,11 +137,11 @@ int main(int argc, char *argv[]) goto error_out; } if (fseeko(infile, -ATOM_PREAMBLE_SIZE, SEEK_CUR) || - fread(ftyp_atom, atom_size, 1, infile) != 1) { + fread(ftyp_atom, atom_size, 1, infile) != 1 || + (start_offset = ftello(infile)) < 0) { perror(argv[1]); goto error_out; } - start_offset = ftello(infile); } else { int ret; /* 64-bit special case */ @@ -202,6 +202,10 @@ int main(int argc, char *argv[]) goto error_out; } last_offset = ftello(infile); + if (last_offset < 0) { + perror(argv[1]); + goto error_out; + } moov_atom_size = atom_size; moov_atom = malloc(moov_atom_size); if (!moov_atom) { |