[FFmpeg-devel] [PATCH 6/9] lavf/http: add http_accept
Nicolas George
george at nsup.org
Thu Jul 9 15:41:00 CEST 2015
Le decadi 20 messidor, an CCXXIII, Stephan Holljes a écrit :
> Signed-off-by: Stephan Holljes <klaxa1337 at googlemail.com>
> ---
> libavformat/http.c | 19 +++++++++++++++++++
> 1 file changed, 19 insertions(+)
> Changes since first version:
> - Add av_assert0() check for sc->listen
> - Add bitmask for s->flags in ffurl_alloc
Thanks.
>
> diff --git a/libavformat/http.c b/libavformat/http.c
> index 676bfd5..3c1ec35 100644
> --- a/libavformat/http.c
> +++ b/libavformat/http.c
> @@ -25,6 +25,7 @@
> #include <zlib.h>
> #endif /* CONFIG_ZLIB */
>
> +#include "libavutil/avassert.h"
> #include "libavutil/avstring.h"
> #include "libavutil/opt.h"
>
> @@ -382,6 +383,24 @@ static int http_open(URLContext *h, const char *uri, int flags,
> return ret;
> }
>
> +static int http_accept(URLContext *s, URLContext **c)
> +{
> + int ret;
> + HTTPContext *sc = s->priv_data;
> + HTTPContext *cc;
> + URLContext *sl = sc->hd;
> + URLContext *cl;
> + av_assert0(sc->listen);
> + if ((ret = ffurl_alloc(c, s->filename, s->flags & AVIO_FLAG_READ_WRITE, &sl->interrupt_callback)) < 0)
> + goto fail;
Same remark as in my previous mail.
> + cc = (*c)->priv_data;
> + if ((ret = ffurl_accept(sl, &cl)) < 0)
> + goto fail;
> + cc->hd = cl;
Does it make sense to set "cc->listen = 1" here, to indicate that the client
is not yet a normal client? It may make the tests for trailers simpler.
> +fail:
> + return ret;
> +}
> +
> static int http_getc(HTTPContext *s)
> {
> int len;
Regards,
--
Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150709/31e0e42c/attachment.sig>
More information about the ffmpeg-devel
mailing list