]> git.sesse.net Git - ffmpeg/commit
avformat: make avformat_network_init() explicitly optional
authorwm4 <nfxjfg@googlemail.com>
Tue, 16 Jan 2018 11:57:04 +0000 (12:57 +0100)
committerwm4 <nfxjfg@googlemail.com>
Tue, 16 Jan 2018 11:57:04 +0000 (12:57 +0100)
commit631c56a8e46dea41585f3e7b3ef9c52b49faa385
tree12294edd1f95322624f5bc3a4cda392353da2e46
parent6512ff72f9cc058e27646604caeacc4ef0dbd93e
avformat: make avformat_network_init() explicitly optional

It was sort of optional before - if you didn't call it, networking was
initialized on demand, and an ugly warning was logged. Also, the doxygen
comments threatened that it would be made strictly required one day.

Make it explicitly optional. I would prefer to deprecate it fully, but
there might still be legitimate reasons to use this. But the average
user won't need it.

This is needed only for two reasons: to initialize TLS libraries like
OpenSSL and GnuTLS, and winsock.

OpenSSL and GnuTLS were already silently initialized on demand if the
global network init function was not called. They also have various
thread-safety acrobatics, which make concurrent initialization within
libavformat safe. In addition, the libraries are moving towards making
their global init functions safe, which removes all need for central
global init. In particular, GnuTLS 3.5.16 and OpenSSL 1.1.0g have been
found to have safe init functions. In all cases, they use internal
reference counters to avoid that the global uninit functions interfere
with concurrent uses of the library by other API users who called global
init.

winsock should be thread-safe as well, and maintains an internal
reference counter as well.

Since we still support ancient TLS libraries, which do not have this
fixed, and since it's unknown whether winsock and GnuTLS
reinitialization is costly in any way, don't deprecate the libavformat
functions yet.
doc/APIchanges
libavformat/avformat.h
libavformat/network.c
libavformat/network.h
libavformat/utils.c
libavformat/version.h