[FFmpeg-devel] [PATCH 2/2] avformat: add protocol_whitelist

Michael Niedermayer michael at niedermayer.cc
Sun Jan 24 16:12:53 CET 2016


On Sun, Jan 24, 2016 at 04:03:40PM +0100, Michael Niedermayer wrote:
> On Sun, Jan 24, 2016 at 03:37:24PM +0100, Andreas Cadhalpun wrote:
> > On 24.01.2016 03:42, Michael Niedermayer wrote:
[...]
> > >      if ((ret = av_opt_set_dict(*puc, options)) < 0)
> > >          goto fail;
> > > +
> > > +    whitelist = (*puc)->protocol_whitelist;
> > > +    if (whitelist && av_match_list((*puc)->prot->name, whitelist, ',') <= 0) {
> > > +        av_log(*puc, AV_LOG_ERROR, "Protocol not on whitelist!\n");
> > 
> > It would be nice to mention the protocol and the whitelist in the error message.

the protocol is already printed as part of the av_log() prefix, ill
add printing the whitelist

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Awnsering whenever a program halts or runs forever is
On a turing machine, in general impossible (turings halting problem).
On any real computer, always possible as a real computer has a finite number
of states N, and will either halt in less than N cycles or never halt.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160124/53376877/attachment.sig>


More information about the ffmpeg-devel mailing list