Re: [libav-devel] [PATCH] cmdutils: Read errno before av_log() as the callback from av_log() might affect errno

2014-10-25 Thread Luca Barbato
On 26/10/14 00:19, Vittorio Giovara wrote: > On Sat, Oct 25, 2014 at 11:14 PM, Vittorio Giovara > wrote: >> On Sat, Oct 25, 2014 at 10:39 PM, Martin Storsjö wrote: >>> From: Michael Niedermayer >> >> Looks ok. >> -- >> Vittorio > > By the way, there are a couple of lines in libavdevice (v4l2 an

Re: [libav-devel] [PATCH] cmdutils: Read errno before av_log() as the callback from av_log() might affect errno

2014-10-25 Thread Vittorio Giovara
On Sat, Oct 25, 2014 at 11:14 PM, Vittorio Giovara wrote: > On Sat, Oct 25, 2014 at 10:39 PM, Martin Storsjö wrote: >> From: Michael Niedermayer > > Looks ok. > -- > Vittorio By the way, there are a couple of lines in libavdevice (v4l2 and oss_audio) that could have the same treatment. Anyone u

Re: [libav-devel] [PATCH] cmdutils: Read errno before av_log() as the callback from av_log() might affect errno

2014-10-25 Thread Vittorio Giovara
On Sat, Oct 25, 2014 at 10:39 PM, Martin Storsjö wrote: > From: Michael Niedermayer > > --- > cmdutils.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/cmdutils.c b/cmdutils.c > index eff011c..d932171 100644 > --- a/cmdutils.c > +++ b/cmdutils.c > @@ -1393,9 +1393,

Re: [libav-devel] [PATCH] bethsoftvid: check return value and clean memory

2014-10-25 Thread Luca Barbato
On 25/10/14 23:19, Vittorio Giovara wrote: > CC: libav-sta...@libav.org > Bug-Id: CID 733777 > --- > libavformat/bethsoftvid.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/libavformat/bethsoftvid.c b/libavformat/bethsoftvid.c > index 120e145..85abf25 100644 > --- a/libavformat/bet

[libav-devel] [PATCH] cmdutils: Read errno before av_log() as the callback from av_log() might affect errno

2014-10-25 Thread Martin Storsjö
From: Michael Niedermayer --- cmdutils.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/cmdutils.c b/cmdutils.c index eff011c..d932171 100644 --- a/cmdutils.c +++ b/cmdutils.c @@ -1393,9 +1393,10 @@ int cmdutils_read_file(const char *filename, char **bufptr, size_t *si

[libav-devel] [PATCH] bethsoftvid: check return value and clean memory

2014-10-25 Thread Vittorio Giovara
CC: libav-sta...@libav.org Bug-Id: CID 733777 --- libavformat/bethsoftvid.c | 4 1 file changed, 4 insertions(+) diff --git a/libavformat/bethsoftvid.c b/libavformat/bethsoftvid.c index 120e145..85abf25 100644 --- a/libavformat/bethsoftvid.c +++ b/libavformat/bethsoftvid.c @@ -189,6 +189,10

Re: [libav-devel] [PATCH 7/8] mxfdec: avoid out-of-bounds write

2014-10-25 Thread Vittorio Giovara
On Sat, Oct 25, 2014 at 9:46 PM, Tomas Härdin wrote: > On Fri, 2014-10-24 at 13:11 +0100, Vittorio Giovara wrote: >> On Fri, Oct 24, 2014 at 7:15 AM, Anton Khirnov wrote: >> > Quoting Vittorio Giovara (2014-10-24 01:05:58) >> >> CC: libav-sta...@libav.org >> >> Bug-Id: CID 732262 >> >> --- >> >>

Re: [libav-devel] [PATCH 7/8] mxfdec: avoid out-of-bounds write

2014-10-25 Thread Tomas Härdin
On Fri, 2014-10-24 at 13:11 +0100, Vittorio Giovara wrote: > On Fri, Oct 24, 2014 at 7:15 AM, Anton Khirnov wrote: > > Quoting Vittorio Giovara (2014-10-24 01:05:58) > >> CC: libav-sta...@libav.org > >> Bug-Id: CID 732262 > >> --- > >> libavformat/mxfdec.c | 5 +++-- > >> 1 file changed, 3 insert

Re: [libav-devel] [PATCH 3/7] configure: Check for gmtime_r

2014-10-25 Thread Martin Storsjö
On Fri, 24 Oct 2014, Luca Barbato wrote: On 24/10/14 10:37, Martin Storsjö wrote: --- configure | 2 ++ 1 file changed, 2 insertions(+) diff --git a/configure b/configure index a82bef7..f53332d 100755 --- a/configure +++ b/configure @@ -1453,6 +1453,7 @@ SYSTEM_FUNCS=" getservbyport

Re: [libav-devel] [PATCH 6/7] filmstripdec: avoid integer overflow

2014-10-25 Thread Luca Barbato
On 24/10/14 14:15, Vittorio Giovara wrote: > CC: libav-sta...@libav.org > Bug-Id: CID 732246 > --- > libavformat/filmstripdec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/filmstripdec.c b/libavformat/filmstripdec.c > index 3fa6842..e4dee11 100644 > --- a/

Re: [libav-devel] [PATCH 1/7] bethsoftvid: check return value

2014-10-25 Thread Luca Barbato
On 24/10/14 14:15, Vittorio Giovara wrote: > CC: libav-sta...@libav.org > Bug-Id: CID 733777 > --- > libavformat/bethsoftvid.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavformat/bethsoftvid.c b/libavformat/bethsoftvid.c > index 120e145..9729505 100644 > --- a/libavformat/beths

[libav-devel] [PATCH] download: Reorder the sections and update the explanations

2014-10-25 Thread Luca Barbato
We do support releases. --- We spend an unordinate amount of time keeping the release branches up to date. Any suggestion or change, please in patch form. src/download | 241 +++ 1 file changed, 110 insertions(+), 131 deletions(-) diff --g

Re: [libav-devel] [PATCH 1/7] bethsoftvid: check return value

2014-10-25 Thread Luca Barbato
On 24/10/14 14:15, Vittorio Giovara wrote: > CC: libav-sta...@libav.org > Bug-Id: CID 733777 > --- > libavformat/bethsoftvid.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavformat/bethsoftvid.c b/libavformat/bethsoftvid.c > index 120e145..9729505 100644 > --- a/libavformat/beths

[libav-devel] How the capture avdevices are assumed to work

2014-10-25 Thread Luca Barbato
## Usage The avdevices are usually used so you have some audio and some video, supposedly in sync, captured at realtime. The common situation is x11grab + pulse right now. ## avconv and syncronization Avconv doesn't make any assumption on the time references so the capture should have matching t

Re: [libav-devel] [PATCH] configure: Place all temporary files in one separate directory

2014-10-25 Thread Luca Barbato
On 27/09/14 00:38, Diego Biurrun wrote: > This reduces TMPDIR pollution and possibly avoids race conditions with > temporary files that are not atomically created. > > Based on a patch from Michał Górny . > --- > Possibly ok. lu ___ libav-devel maili