Skip to content

F
ffmpeg.wasm-core
Switch branch/tag
  1. 21 Jun, 2019 3 commits
  3. 20 Jun, 2019 3 commits
  5. 19 Jun, 2019 1 commit
    • Bodecs Bela's avatar
      av_format/hlsenc: fix %v handling by format_name function · 09a48539
      Bodecs Bela authored 
      Hi All,

When multiple variant streams are specified by var_stream_map option, %v
placeholder in various names ensures that each variant has its unique
names. Most of %v handlng is done in format_name function. Currently
in this function the result buffer is the same as the
input pattern buffer, so you must allocate it before calling format_name
function. It also means, that it is silently assumed that the result
string will NOT be
longer that the pattern string. It is true most of the time, because %v
may appear only once in the pattern string and number of variant streams
is less than 100 in practical cases. But theoretically it will fail if
specified number of variant streams is greater than 100 (i.e. longer
than 2 digits).
This patch fixes this behaviour by altering format_name function to
allocate the
result buffer and return it to the caller.

Please, review this patch.

best,

Bela
>From 6377ebee8a106a9684d41b270c7d6c8e57cd3e7b Mon Sep 17 00:00:00 2001
From: Bela Bodecs <bodecsb@vivanet.hu>
Date: Mon, 17 Jun 2019 14:31:36 +0200
Subject: [PATCH] av_format/hlsenc: fix %v handling by format_name function

When multiple variant streams are specified by var_stream_map option, %v
placeholder in various names ensures that each variant has its unique
names. Most of %v handlng is done in format_name function. Currently
in this function the result buffer is the same as the input pattern
buffer, so you must allocate it before calling format_name function. It
also means, that it is silently assumed that the result string will NOT
be longer that the pattern string. It is true most of the time, because
%v may appear only once in the pattern string and number of variant
streams is less than 100 in practical cases. But theoretically it will
fail if specified number of variant streams is greater than 100. This
patch fixes this behaviour by altering format_name function to allocate
the result buffer and return it to the caller.
Signed-off-by: 's avatarBela Bodecs <bodecsb@vivanet.hu>
      09a48539
  7. 18 Jun, 2019 3 commits
  9. 17 Jun, 2019 1 commit
  11. 16 Jun, 2019 3 commits
  13. 14 Jun, 2019 20 commits
  15. 13 Jun, 2019 4 commits
  17. 12 Jun, 2019 2 commits
    • Alejandro Solozabal's avatar
      avcodec/omx: Correct av_log() log message · caabe1b4
      Alejandro Solozabal authored 
      Print the right library name, which is trying to open, on the log message.
Signed-off-by: 's avatarAlejandro Solozabal <a.solozabal@hotmail.com>
      caabe1b4
    • U. Artie Eoff's avatar
      vaapi_encode_mjpeg: fix bad component id bug · f70c3974
      U. Artie Eoff authored 
      The compound literals assigned to "components"
only exist within the scope of the if/else
block (thanks Mark Thompson for the better
explanation).

Thus, after this if/else block, "components"
ends up pointing to an arbitrary/undefined
array.  With some compilers and depending on
optimization settings, these arbitrary values
may end up being the same value (i.e. 0 with
GNU GCC 9.x).  Unfortunately, the GNU GCC
compiler, at least, never prints any warnings
about this.

This patch fixes this issue by assigning the
constant arrays to local variables at function
scope and then pointing "components" to those
as necessary.

Fixes #7915
Signed-off-by: 's avatarU. Artie Eoff <ullysses.a.eoff@intel.com>
      f70c3974