summaryrefslogtreecommitdiffstats
path: root/libavformat
Commit message (Collapse)AuthorAgeFilesLines
* Drop unreachable break and return statementsDiego Biurrun2016-11-033-3/+0
|
* rtmpproto: Restructure zlib code to avoid unreachable code warningDiego Biurrun2016-11-021-4/+5
| | | | libavformat\rtmpproto.c(1165) : warning C4702: unreachable code
* mov: Evaluate the movie display matrixVittorio Giovara2016-11-012-14/+37
| | | | | | | | | | | This matrix needs to be applied after all others have (currently only display matrix from trak), but cannot be handled in movie box, since streams are not allocated yet. So store it in main context, and apply it when appropriate, that is after parsing the tkhd one. Fate tests are updated accordingly. Signed-off-by: Vittorio Giovara <[email protected]>
* openssl: Allow newer TLS versions than TLSv1Mark Thompson2016-10-311-1/+6
| | | | | | The use of TLSv1_*_method() disallows newer protocol versions; instead use SSLv23_*_method() and then explicitly disable the deprecated protocol versions which should not be supported.
* Revert "rtmpproto: Don't include a client version in the unencrypted C1 ↵Luca Barbato2016-10-301-1/+4
| | | | | | handshake" This reverts commit 7d8d726be7dc46343ab1c98c339c1ed44bcb07c1.
* dash: Use correct ISO C scanf conversion specifierDiego Biurrun2016-10-281-1/+1
|
* srt: Adjust signedness of sscanf format stringsDiego Biurrun2016-10-281-1/+1
| | | | Fixes several warnings from -Wformat.
* Use ISO C printf conversion specifiers where appropriateDiego Biurrun2016-10-285-23/+34
|
* Adjust printf conversion specifiers to match variable signednessDiego Biurrun2016-10-284-25/+25
|
* Remove some pointless TRACE level debug codeDiego Biurrun2016-10-274-41/+0
| | | | This also kills some warnings with certain compiler options.
* nut: Drop pointless TRACE level debug codeDiego Biurrun2016-10-272-46/+0
| | | | The code has little usefulness and uses the __PRETTY_FUNCTION__ GNU extension.
* nsv: Drop unnecessary TRACE level debug codeDiego Biurrun2016-10-271-13/+0
| | | | The output is rather silly and the code uses non-standard __FUNCTION__.
* openssl: Avoid double semicolons after the GET_BIO_DATA macroMartin Storsjö2016-10-251-2/+2
| | | | | | | | | | | When the macro is expanded with a semicolon following it and the macro itself contains a semicolon, we ended up in double semicolons, which is treated as a statement that disallows further declarations. This avoids errors about mixed declarations and statements on gcc, after ee050797664c. Signed-off-by: Martin Storsjö <[email protected]>
* openssl: Support version 1.1.0.Matt Oliver2016-10-231-2/+35
| | | | | | | Further simplifications by Martin Storsjö, to minimize the diff. Signed-off-by: Martin Storsjö <[email protected]>
* rtmpdh: Don't use the OpenSSL DH structMartin Storsjö2016-10-232-57/+52
| | | | | | | | | Instead use our own struct, which we already use when using gcrypt and gnutls. In OpenSSL 1.1, the DH struct has been made opaque. Signed-off-by: Martin Storsjö <[email protected]>
* mov: Update colr valuesVittorio Giovara2016-10-211-7/+11
| | | | | | | | | | For 'nclx', the latest edition of the standard switched from JPEG XR to 23001-8, which matches the current order of our entries. Bounds are preserved as a sanity check. For 'nclc', qtff edition 2016-09-13 introduced a few new entries. Signed-off-by: Vittorio Giovara <[email protected]>
* lavf: check that the codec is supported by extract_extradataAnton Khirnov2016-10-211-0/+10
| | | | | Avoids superfluous error message spam after 8e2ea691351c5079cdab245ff7bfa5c0f3e3bfe4
* lavf: use the new bitstream filter for extracting extradataAnton Khirnov2016-10-162-13/+119
| | | | | | | | | This also fixes a minor bug introduced in the codecpar conversion, where the termination condition for extracting the extradata does not match the actual extradata setting code. As a result, the packet durations made up by lavf go back to their values before the codecpar conversion. That is of little consequence since that code should eventually be dropped completely.
* hevcdec: move decoder-independent declarations into a separate headerAnton Khirnov2016-10-162-39/+40
| | | | | | | This way they can be reused by other code without including the whole decoder-specific hevcdec.h Also, add the HEVC_ prefix to them, since similarly named values exist for H.264 as well and are sometimes used in the same code.
* hevc: rename hevc.[ch] to hevcdec.[ch]Anton Khirnov2016-10-162-2/+2
| | | | | This is more consistent with the rest of libav and frees up the hevc.h name for decoder-independent shared declarations.
* avformat/matroska: fix MatroskaVideoFieldOrder enum valuesJames Almer2016-10-162-3/+3
| | | | | | | | | | | | | | | The spec says 9: Interlaced with bottom field displayed first and top field stored first 14: Interlaced with top field displayed first and bottom field stored first And avcodec.h states AV_FIELD_TB, //< Top coded first, bottom displayed first AV_FIELD_BT, //< Bottom coded first, top displayed first Signed-off-by: James Almer <[email protected]> Signed-off-by: Anton Khirnov <[email protected]>
* file protocol: handle the file: protocol string in file_checkAnton Khirnov2016-10-161-1/+6
| | | | | | This is consistent with what file_open() does. CC: [email protected]
* rtmpproto: Don't include a client version in the unencrypted C1 handshakeMartin Storsjö2016-10-141-4/+1
| | | | | | | | | | According to the public RTMP specification, these 4 bytes should be zero. librtmp in server mode assumes that the RTMPE (FP9) handshake is used if these bytes are nonzero. Signed-off-by: Martin Storsjö <[email protected]>
* rtmpproto: Don't include the libavformat version as "clientid"Martin Storsjö2016-10-141-3/+0
| | | | | | | | | | When acting as server, the server can include a "clientid" property in some status messages. But this should be a unique number identifying the client session, not identifying the server itself. In practice, omitting it works just as well as including this incorrect field. Signed-off-by: Martin Storsjö <[email protected]>
* rtmpproto: Send chunk size on the network channelMartin Storsjö2016-10-141-1/+1
| | | | | | | | This makes sure that e.g. Adobe FME actually reacts to it. As long as the value we've been sending is the default one (128), the bug hasn't been noticed. Signed-off-by: Martin Storsjö <[email protected]>
* rtmpproto: Lengthen the filename buffer when receiving streamsMartin Storsjö2016-10-141-1/+1
| | | | | | | Some applications such as Adobe FME append lots of parameters here, making it easily overflow the current limit. Signed-off-by: Martin Storsjö <[email protected]>
* rtmpproto: Check the return from ff_amf_read_stringMartin Storsjö2016-10-141-0/+7
| | | | | | | | If this failed, we used to continue with an uninitialized filename buffer. CC: [email protected] Signed-off-by: Martin Storsjö <[email protected]>
* avidec: Set palette alpha as fully opaqueVittorio Giovara2016-10-021-1/+1
| | | | Palette format is always in RGBA.
* avidec: Do not special case palette on big-endianVittorio Giovara2016-10-021-5/+1
| | | | This simplifies the code a bit, does not change output data in any way.
* avio: add a new flag for marking streams seekable by timestampAnton Khirnov2016-09-303-2/+10
|
* avio: cosmetics, prettify AVIO_SEEKABLE_NORMALAnton Khirnov2016-09-301-2/+4
| | | | | Move the doxy above the definition, change the value itself to the (1 << n) pattern, which is more readable for flags.
* lavf: fix usage of AVIOContext.seekableAnton Khirnov2016-09-3048-78/+84
| | | | | | | | It is supposed to be a flag. The only currently defined value is AVIO_SEEKABLE_NORMAL, but other ones may be added in the future. However all the current lavf code treats this field as a bool (mainly for historical reasons). Change all those cases to properly check for AVIO_SEEKABLE_NORMAL.
* rtsp: Fix a crash with the RTSP muxerMartin Storsjö2016-09-151-1/+2
| | | | | | | | | | | | | This was introduced in bc2a32969e. The whole block that the statement was added to is only relevant when used as a demuxer, but the other statements there have had other if statements guarding them. Make sure to only run this whole block if being used as a demuxer. CC: [email protected] Signed-off-by: Martin Storsjö <[email protected]>
* mov: Remove old b-frame/video delay heuristicDerek Buitenhuis2016-08-291-3/+0
| | | | | | | | This was added before edts support existed, and is no longer valid. Signed-off-by: Derek Buitenhuis <[email protected]> Signed-off-by: Luca Barbato <[email protected]>
* mov: Remove ancient heuristic hackDerek Buitenhuis2016-08-292-8/+1
| | | | | | | | | This breaks files with legitimate single-entry edit lists, and the hack, introduced in f03a081df09f9c4798a17d7e24446ed47924b11b, has no link to any known sample in its commit message. Signed-off-by: Derek Buitenhuis <[email protected]> Signed-off-by: Luca Barbato <[email protected]>
* av1: Add codec_id and basic demuxing supportLuca Barbato2016-08-242-0/+2
| | | | Signed-off-by: Diego Biurrun <[email protected]>
* rtpdec_mpeg4: validate fmtp fieldsLuca Barbato2016-08-231-3/+14
|
* nsv: Drop disabled cruftDiego Biurrun2016-08-171-4/+0
|
* wc3movie: Drop unused cruftDiego Biurrun2016-08-171-4/+0
|
* rm: Drop broken disabled cruftDiego Biurrun2016-08-172-21/+0
|
* aiff: Skip padding byte for odd-sized chunksDiego Biurrun2016-08-101-2/+6
| | | | Bug-Id: 660
* m4vdec: Check for non-startcode 00 00 00 sequences in probeMichael Niedermayer2016-08-031-3/+5
| | | | | | | This makes the m4v detection less trigger-happy. Bug-Id: 949 Signed-off-by: Diego Biurrun <[email protected]>
* Revert "Don't use expressions with side effects in macro parameters"Martin Storsjö2016-08-021-4/+2
| | | | | | | | | This reverts commit 25bacd0a0c32ae682e6f411b1ac9020aeaabca72. Since 230b1c070, the bytewise AV_W*() macros only expand their argument once, so revert to the more readable version of these. Signed-off-by: Martin Storsjö <[email protected]>
* Don't use expressions with side effects in macro parametersMartin Storsjö2016-07-311-2/+4
| | | | | | | | | AV_WB32 can be implemented as a macro that expands its parameters multiple times (in case AV_HAVE_FAST_UNALIGNED isn't set and the compiler doesn't support GCC attributes); make sure not to read multiple times from the source in this case. Signed-off-by: Martin Storsjö <[email protected]>
* flac demuxer: improve probingAnton Khirnov2016-07-311-3/+18
| | | | | Extend the probe function to validate the STREAMINFO block that must follow the fLaC ID tag.
* Fix instances of broken indentation found by gcc 6Anton Khirnov2016-07-311-2/+2
|
* mov: Validate the ID numberLuca Barbato2016-07-291-2/+2
| | | | | | IDs in MOV start from 1. Signed-off-by: Diego Biurrun <[email protected]>
* riff: don't overwrite bps from WAVEFORMATEX if EXTENSIBLE doesn't contain ↵Hendrik Leppkes2016-07-221-1/+5
| | | | | | | | | | | | | that data. According to the specification on the MSDN [1], 0 is valid for that particular field, and it should be ignored in that case. [1]: http://msdn.microsoft.com/en-us/library/windows/desktop/dd757714(v=vs.85).aspx Bug-Id: 950 Signed-off-by: Anton Khirnov <[email protected]>
* mov: Rework the check for invalid indexes in stscVittorio Giovara2016-07-201-2/+7
| | | | | | | | | | There are samples with invalid stsc that may work fine as is and do not need extradata change. So ignore any out of range index, and error out only when explode is set. Found-by: Matthieu Bouron <[email protected]> Signed-off-by: Vittorio Giovara <[email protected]>
* mov: Wrap stsc index and count compare in a separate functionVittorio Giovara2016-07-201-5/+10
|