Hi,

On Sat, Apr 9, 2011 at 7:50 AM, Reinhard Tartler <siret...@tauware.de> wrote:
> From: Georgi Chorbadzhiyski <g...@unixsol.org>
>
> This patch adds support for setting transport_stream_id,
> original_network_id, service_id, pmt_start_pid and start_pid
> in mpegts muxer.
>
> Signed-off-by: Ronald S. Bultje <rsbul...@gmail.com>
> (cherry picked from commit 445996aa51f4f1d9a26456a8511988291a720ba0)
> ---
>  libavformat/mpegtsenc.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c
> index 40b906e..66ccfc6 100644
> --- a/libavformat/mpegtsenc.c
> +++ b/libavformat/mpegtsenc.c
> @@ -421,7 +421,7 @@ static MpegTSService *mpegts_add_service(MpegTSWrite *ts,
>     service = av_mallocz(sizeof(MpegTSService));
>     if (!service)
>         return NULL;
> -    service->pmt.pid = ts->pmt_start_pid + ts->nb_services - 1;
> +    service->pmt.pid = ts->pmt_start_pid + ts->nb_services;
>     service->sid = sid;
>     service->provider_name = av_strdup(provider_name);
>     service->name = av_strdup(name);

The commit msg is wrong. I think some commits were mixed up here. The
commit msg should likely read:

"mpegtsenc: fix setting start_pid, which was off by one."

Then it's OK, please apply.

Ronald
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to