aboutsummaryrefslogtreecommitdiffstats
path: root/libavformat/avio.c
Commit message (Collapse)AuthorAgeFilesLines
* avformat/whip: Add WHIP muxer support for subsecond latency streamingJack Lau2025-06-041-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 0. WHIP Version 3. 1. The WHIP muxer has been renamed and refined, with improved logging context and error messages for SSL, DTLS, and RTC. 2. Magic numbers have been replaced with macros and extracted to functions, and log levels have been altered for better clarity. 3. DTLS curve list has been updated, and SRTP profile names have been refined for FFmpeg and OpenSSL. 4. ICE STUN magic number has been refined, and RTP payload types have been updated based on Chrome's definition. 5. Fixed frame size has been refined to rtc->audio_par->frame_size, and h264_mp4toannexb is now used to convert MP4/ISOM to annexb. 6. OPUS timestamp issue has been addressed, and marker setting has been corrected after utilizing BSF. 7. DTLS handshake and ICE handling have been optimized for improved performance, with a single handshake timeout and server role to prevent ARQ. 8. Consolidated ICE request/response handling and DTLS handshake into a single function, and fixed OpenSSL build errors to work with Pion. 9. Merge TLS & DTLS implementation, shared BIO callbacks, read, write, print_ssl_error, openssl_init_ca_key_cert, init_bio_method function and shared same data structure 10. Modify configure that whip is enabled only dtls is enabled(just support openssl for now) to fix build error Co-authored-by: winlin <winlinvip@gmail.com> Co-authored-by: yangrtc <yangrtc@aliyun.com> Co-authored-by: cloudwebrtc <duanweiwei1982@gmail.com> Co-authored-by: Haibo Chen <495810242@qq.com> Co-authored-by: Steven Liu <lq@chinaffmpeg.org> Co-authored-by: Jun Zhao <barryjzhao@tencent.com> Signed-off-by: Jack Lau <jacklau1222@qq.com> Signed-off-by: Steven Liu <lq@chinaffmpeg.org>
* avutil/common: Don't auto-include mem.hAndreas Rheinhardt2024-03-311-0/+1
| | | | | | | | | | | There are lots of files that don't need it: The number of object files that actually need it went down from 2011 to 884 here. Keep it for external users in order to not cause breakages. Also improve the other headers a bit while just at it. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
* avformat: remove deprecated FF_API_AVIO_WRITE_NONCONSTJames Almer2024-03-071-4/+0
| | | | Signed-off-by: James Almer <jamrial@gmail.com>
* avformat: remove deprecated FF_API_AVIODIRCONTEXTJames Almer2024-03-071-2/+0
| | | | Signed-off-by: James Almer <jamrial@gmail.com>
* avformat/avio: Avoid av_strdup(NULL)Andreas Rheinhardt2024-03-051-8/+12
| | | | Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
* avformat/avio: Avoid indirection in ffio_fdopen()Andreas Rheinhardt2024-03-051-19/+21
| | | | Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
* avformat/avio: Avoid function pointer castsAndreas Rheinhardt2024-03-051-3/+2
| | | | | | | It is undefined behaviour to use a different type for a call than the actual type of the function. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
* avformat/aviobuf: Move code specific to URLContexts to avio.cAndreas Rheinhardt2024-03-051-2/+189
| | | | | | | This separates the URL-layer adjacent parts of the code from the parts that are also usable with custom IO. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
* avformat/avio: Don't use incompatible function pointer type for callAndreas Rheinhardt2023-09-121-9/+7
| | | | | | | | | | | | | | It is undefined behaviour even in cases where it works (it works because it is only a const uint8_t* vs. uint8_t* difference). Instead add a cbuf parameter to pass a const buffer (for writing) as well as a parameter indicating whether we are reading or writing; retry_transfer_wrapper() itself then uses the correct function based upon this information. Reviewed-by: Marton Balint <cus@passwd.hu> Reviewed-by: Tomas Härdin <git@haerdin.se> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
* avformat/avio: Constify data pointees of write callbacksAndreas Rheinhardt2023-09-101-0/+4
| | | | | | | | They are currently non-const for reasons unknown, although avio_write() accepts a const buffer. Reviewed-by: Anton Khirnov <anton@khirnov.net> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
* avformat/aviobuf: Don't use incompatible function pointer type for callAndreas Rheinhardt2023-09-101-4/+11
| | | | | | | | | | It is undefined behaviour even in cases where it works (it works because both are pointers). Instead change the functions involved to use the type expected by the AVIO-API and add inline wrappers for our internal callers. Reviewed-by: Tomas Härdin <git@haerdin.se> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
* avformat/avio: Remove redundant checksAndreas Rheinhardt2023-09-101-4/+1
| | | | | | | Checking the return value of av_opt_set() is equivalent to the current checks. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
* avformat/avio: Remove duplicated freeing codeAndreas Rheinhardt2023-09-101-2/+0
| | | | | | The target of the jump frees this stuff, too. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
* avformat: Remove avformat and avio headers from protocolsAndreas Rheinhardt2023-09-101-1/+0
| | | | Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
* avformat/avio: Schedule AVIODirContext to become an opaque typeAndreas Rheinhardt2022-10-031-0/+6
| | | | | | | Users can't make anything with its content. Making it opaque might allow us to avoid one level of indirection. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
* avformat/avio: Check av_opt_copy() for failureMichael Niedermayer2021-05-271-2/+5
| | | | | | Fixes: CID1477416 Unchecked return value Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* avio: do not export avpriv_io_{move,delete}Anton Khirnov2021-05-221-3/+3
| | | | | They are private and not used by anything outside of lavf. There is no reason for them to be exported.
* avutil: remove deprecated AVClass.child_class_nextJames Almer2021-04-271-3/+0
| | | | Signed-off-by: James Almer <jamrial@gmail.com>
* avformat/avio: Use av_strstart instead of strncmpAndreas Rheinhardt2021-02-281-3/+2
| | | | | | | It makes the intent clearer and avoids calculating the length in advance. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
* avformat/avio: Remove ffurl_openAndreas Rheinhardt2021-02-021-7/+0
| | | | | | It is only used in commented-out (and nonworking) code in async.c. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
* URLContext: switch to child_class_iterate()Anton Khirnov2020-06-101-0/+3
|
* remove CHAR_MIN/CHAR_MAX usagePaul B Mahol2020-03-171-2/+2
| | | | It is not needed at all.
* avformat/avio: fix ff_rename to respect used protocolMarton Balint2020-01-191-10/+4
| | | | | | | | | | | Also simplify it and make it always log the error. This fixes for example the image2 muxer when used with an URL which also contains the protocol: ffmpeg -f lavfi -i testsrc -vframes 10 -atomic_writing 1 file:out%d.png Signed-off-by: Marton Balint <cus@passwd.hu>
* avformat/avio: move ff_rename implementation from internal.h to avio.cMarton Balint2020-01-191-0/+15
| | | | Signed-off-by: Marton Balint <cus@passwd.hu>
* avformat/avio: Use ffurl_closepAndreas Rheinhardt2019-11-291-2/+1
| | | | | Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* lavf/avio: Print https warning also for avio_find_protocol_name().Carl Eugen Hoyos2019-10-011-4/+3
| | | | Helps to fix ticket #8197.
* avformat/avio: make the logic simpleJun Zhao2018-05-061-3/+2
| | | | | | remove the "ret" to make the code simple and generic. Signed-off-by: Jun Zhao <mypopydev@gmail.com>
* avformat/avio: check input URLContext value NULLSteven Liu2017-12-281-4/+6
| | | | | Reviewed-by: Michael Niedermayer <michael@niedermayer.cc> Reviewed-by: Karthick Jeyapal <kjeyapal@akamai.com>
* lavf/avio: Print the https warning also for missing tls protocol.Carl Eugen Hoyos2017-10-231-1/+1
|
* libavformat: not treat 0 as EOFDaniel Kucera2017-10-191-2/+4
| | | | | | | | | | transfer_func variable passed to retry_transfer_wrapper are h->prot->url_read and h->prot->url_write functions. These need to return EOF or other error properly. In case of returning >= 0, url_read/url_write is retried until error is returned. Signed-off-by: Daniel Kucera <daniel.kucera@gmail.com>
* avformat/avio: Remove no-op code in url_find_protocol().Wan-Teh Chang2017-07-101-2/+0
| | | | | | | | | | | | | In url_find_protocol(), proto_str is either "file" or a string consisting of only the characters in URL_SCHEME_CHARS, which does not include ','. Therefore the strchr(proto_str, ',') call always returns NULL. Note: The code was added in commit 6161c41817f6e53abb3021d67ca0f19def682718. Signed-off-by: Wan-Teh Chang <wtc@google.com> Signed-off-by: Muhammad Faiz <mfcc64@gmail.com>
* lavf/avio: Be more explicit in logging white/black list matchesAlexander Strasser2017-03-141-2/+2
| | | | | | | | | The current form of the messages indicating matches in the white or black lists seems to be a bit too much relying on context. Make the messages more explicit. Signed-off-by: Alexander Strasser <eclipse7@gmx.net>
* lavf/avio: Remove unnecessary escaping of ' in string literalsAlexander Strasser2017-03-141-2/+2
| | | | Signed-off-by: Alexander Strasser <eclipse7@gmx.net>
* HTTP: improve performance by reducing forward seeksJoel Cunningham2017-02-141-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit optimizes HTTP performance by reducing forward seeks, instead favoring a read-ahead and discard on the current connection (referred to as a short seek) for seeks that are within a TCP window's worth of data. This improves performance because with TCP flow control, a window's worth of data will be in the local socket buffer already or in-flight from the sender once congestion control on the sender is fully utilizing the window. Note: this approach doesn't attempt to differentiate from a newly opened connection which may not be fully utilizing the window due to congestion control vs one that is. The receiver can't get at this information, so we assume worst case; that full window is in use (we did advertise it after all) and that data could be in-flight The previous behavior of closing the connection, then opening a new with a new HTTP range value results in a massive amounts of discarded and re-sent data when large TCP windows are used. This has been observed on MacOS/iOS which starts with an initial window of 256KB and grows up to 1MB depending on the bandwidth-product delay. When seeking within a window's worth of data and we close the connection, then open a new one within the same window's worth of data, we discard from the current offset till the end of the window. Then on the new connection the server ends up re-sending the previous data from new offset till the end of old window. Example (assumes full window utilization): TCP window size: 64KB Position: 32KB Forward seek position: 40KB * (Next window) 32KB |--------------| 96KB |---------------| 160KB * 40KB |---------------| 104KB Re-sent amount: 96KB - 40KB = 56KB For a real world test example, I have MP4 file of ~25MB, which ffplay only reads ~16MB and performs 177 seeks. With current ffmpeg, this results in 177 HTTP GETs and ~73MB worth of TCP data communication. With this patch, ffmpeg issues 4 HTTP GETs and 3 seeks for a total of ~22MB of TCP data communication. To support this feature, the short seek logic in avio_seek() has been extended to call a function to get the short seek threshold value. This callback has been plumbed to the URLProtocol structure, which now has infrastructure in HTTP and TCP to get the underlying receiver window size via SO_RCVBUF. If the underlying URL and protocol don't support returning a short seek threshold, the default s->short_seek_threshold is used This feature has been tested on Windows 7 and MacOS/iOS. Windows support is slightly complicated by the fact that when TCP window auto-tuning is enabled, SO_RCVBUF doesn't report the real window size, but it does if SO_RCVBUF was manually set (disabling auto-tuning). So we can only use this optimization on Windows in the later case Signed-off-by: Joel Cunningham <joel.cunningham@me.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* avformat/avio: Fix null pointer dereference in case of memleakMichael Niedermayer2016-05-181-0/+2
| | | | Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* avformat/avio: fix memory leak in url_find_protocolYong Lei2016-05-181-0/+1
| | | | Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* Merge commit 'fab8156b2f30666adabe227b3d7712fd193873b1'Derek Buitenhuis2016-04-211-2/+5
|\ | | | | | | | | | | | | * commit 'fab8156b2f30666adabe227b3d7712fd193873b1': avio: Copy URLContext generic options into child URLContexts Merged-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
| * avio: Copy URLContext generic options into child URLContextsMartin Storsjö2016-03-241-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since all URLContexts have the same AVOptions, such AVOptions will be applied on the outermost context only and removed from the dict, while they probably make sense on all contexts. This makes sure that rw_timeout gets propagated to the innermost URLContext (to make sure it gets passed to the tcp protocol, when opening a http connection for instance). Alternatively, such matching options would be kept in the dict and only removed after the ffurl_connect call. Signed-off-by: Martin Storsjö <martin@martin.st>
* | Merge commit 'ccea588f831906084b8c8235222920e6984beb72'Derek Buitenhuis2016-04-181-1/+4
|\| | | | | | | | | | | | | * commit 'ccea588f831906084b8c8235222920e6984beb72': avio: Add an option 'rw_timeout' Merged-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
| * avio: Add an option 'rw_timeout'Andrey Utkin2016-03-241-4/+17
| | | | | | | | | | | | | | | | | | | | | | If set non-zero, this limits duration of the retry_transfer_wrapper() loop, thus affecting ffurl_read*(), ffurl_write(). As soon as one single byte is successfully received/transmitted, the timer restarts. This has further changes by Michael Niedermayer and Martin Storsjö. Signed-off-by: Martin Storsjö <martin@martin.st>
* | Merge commit 'd44f3e4059506a182f59218b1e967d42b01e097c'Derek Buitenhuis2016-04-181-0/+3
|\| | | | | | | | | | | | | * commit 'd44f3e4059506a182f59218b1e967d42b01e097c': avio: Apply avoptions on the URLContext itself as well Merged-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
| * avio: Apply avoptions on the URLContext itself as wellMartin Storsjö2016-03-241-0/+3
| | | | | | | | | | | | | | Currently the list of avoptions for URLContext is empty though, but such options will be added. Signed-off-by: Martin Storsjö <martin@martin.st>
| * urlprotocol: receive a list of protocols from the callerAnton Khirnov2016-02-221-16/+26
| | | | | | | | | | This way, the decisions about which protocols are available for use in any given situations can be delegated to the caller.
* | lavf/avio: Remove linebreak from https warning.Carl Eugen Hoyos2016-04-101-1/+1
| |
* | avformat/avio: Fix unknown protocol handlingMichael Niedermayer2016-03-171-8/+10
| | | | | | | | | | | | Fixes regression since bb8cc89b2986df6f60831b67cd250da312cce1d0 Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* | avformat: Add a protocol blacklisting APIDerek Buitenhuis2016-03-041-2/+20
| | | | | | | | Signed-off-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
* | Merge commit '832a202c47a246ed15e3edc6b05dfcfa7d82c4b2'Derek Buitenhuis2016-02-291-2/+5
|\| | | | | | | | | | | | | * commit '832a202c47a246ed15e3edc6b05dfcfa7d82c4b2': protocols: make the list of protocols static Merged-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
| * protocols: make the list of protocols staticAnton Khirnov2016-02-221-4/+10
| | | | | | | | | | Disallow other code to touch it directly, now it's only accessible through a blacklisting/whitelisting function.
* | Merge commit '7d61dc95d741ca134d59b1f34c4e10c4c4e36f56'Derek Buitenhuis2016-02-291-20/+2
|\| | | | | | | | | | | | | * commit '7d61dc95d741ca134d59b1f34c4e10c4c4e36f56': lavf: move urlcontext_child_class_next() to protocols.c Merged-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
| * lavf: move urlcontext_child_class_next() to protocols.cAnton Khirnov2016-02-221-20/+1
| | | | | | | | | | It needs to access the list of protocols directly, so it more properly belongs there.
generated by cgit v1.2.3 (git 2.25.1) at 2025-08-04 13:19:26 +0000