summaryrefslogtreecommitdiff
path: root/libavformat/ty.c
Commit message (Collapse)AuthorAge
* avformat/ty: Remove write-only array and variableAndreas Rheinhardt2020-08-11
| | | | | | | | | | | | | | | | Up until now, the TiVo demuxer parse an array of SEQ entries, yet it has never ever made any use of them. In fact, parse_master, the function parsing said table, only influenced the outside world in three ways: Via an excessive amount of error message in case a certain parameter is not what it expected; via an allocation (the aforementioned write-only array); and by setting a certain parameter (ty->cur_chunk_pos), but that parameter is always overwritten before it is used (it is overwritten in get_chunk() on success and if get_chunk() fails, the error is returned to the caller anyway). So remove the array and the function used to parse it. Reviewed-by: Paul B Mahol <onemda@gmail.com> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
* avformat: Forward errors where possibleAndreas Rheinhardt2019-12-12
| | | | | | | | | | It is not uncommon to find code where the caller thinks to know better what the return value should be than the callee. E.g. something like "if (av_new_packet(pkt, size) < 0) return AVERROR(ENOMEM);". This commit changes several instances of this to instead forward the actual error. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* lavf: Constify the probe function argument.Carl Eugen Hoyos2019-03-21
| | | | | Reviewed-by: Lauri Kasanen Reviewed-by: Tomas Härdin
* libavformat/ty: use decimal constant for shiftMichael Niedermayer2018-10-07
| | | | | | | This was the only case in the source that uses a hexadecimal shift value. The change removed a special case in respect to greping Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* avformat/ty: check cur_chunk_pos before using itPaul B Mahol2017-11-12
| | | | | | Fixes #6831. Signed-off-by: Paul B Mahol <onemda@gmail.com>
* avformat/ty: fix memory leaksPaul B Mahol2017-11-12
| | | | Signed-off-by: Paul B Mahol <onemda@gmail.com>
* avformat/ty: check if header is correctly parsed and is still presentPaul B Mahol2017-11-12
| | | | | | Fixes #6829. Signed-off-by: Paul B Mahol <onemda@gmail.com>
* avformat/ty: fix format specifiers in debug log messagesJames Almer2017-11-08
| | | | Signed-off-by: James Almer <jamrial@gmail.com>
* avformat/ty: do not overread chunkPaul B Mahol2017-11-06
| | | | Signed-off-by: Paul B Mahol <onemda@gmail.com>
* avformat/ty: try harder to find stream signaturePaul B Mahol2017-11-06
| | | | | | Search all chunks, not just first one. Signed-off-by: Paul B Mahol <onemda@gmail.com>
* avformat: add TiVo ty demuxerPaul B Mahol2017-11-06
Signed-off-by: Paul B Mahol <onemda@gmail.com>
generated by cgit v1.2.3 (git 2.39.1) at 2024-07-10 23:31:30 +0000