[vlc-commits] flac: use library defined constant

2017-05-24 Thread Tristan Matthews
vlc | branch: master | Tristan Matthews  | Wed May 24 
20:35:04 2017 -0400| [c88a956c6424cd58ab5f8f51c83bdbd246c492d0] | committer: 
Tristan Matthews

flac: use library defined constant

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=c88a956c6424cd58ab5f8f51c83bdbd246c492d0
---

 modules/codec/flac.c | 6 ++
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/modules/codec/flac.c b/modules/codec/flac.c
index 65a78ea772..b53dc8406f 100644
--- a/modules/codec/flac.c
+++ b/modules/codec/flac.c
@@ -75,9 +75,7 @@ struct decoder_sys_t
 bool b_stream_info;
 };
 
-#define FLAC_MAX_CHANNELS 8
-
-static const int pi_channels_maps[FLAC_MAX_CHANNELS + 1] =
+static const int pi_channels_maps[FLAC__MAX_CHANNELS + 1] =
 {
 0,
 AOUT_CHAN_CENTER,
@@ -98,7 +96,7 @@ static const int pi_channels_maps[FLAC_MAX_CHANNELS + 1] =
 };
 
 /* XXX it supposes our internal format is WG4 */
-static const uint8_t ppi_reorder[1+FLAC_MAX_CHANNELS][FLAC_MAX_CHANNELS] =
+static const uint8_t ppi_reorder[1+FLAC__MAX_CHANNELS][FLAC__MAX_CHANNELS] =
 {
 { },
 { 0, },

___
vlc-commits mailing list
vlc-commits@videolan.org
https://mailman.videolan.org/listinfo/vlc-commits


[vlc-commits] vlc_stream_ReadLine: fix trimming

2017-05-24 Thread Rémi Denis-Courmont
vlc | branch: master | Rémi Denis-Courmont  | Wed May 24 
22:07:05 2017 +0300| [32ff0bbcada5e6d5c0e92e684b454764241fafa0] | committer: 
Rémi Denis-Courmont

vlc_stream_ReadLine: fix trimming

Removal of CR or LF final characters relied on the off-by-one bug fixed
in the previous changest, and no longer works. This adjusts it
accordingly.

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=32ff0bbcada5e6d5c0e92e684b454764241fafa0
---

 src/input/stream.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/input/stream.c b/src/input/stream.c
index a1eb20ec9e..1191e99b3d 100644
--- a/src/input/stream.c
+++ b/src/input/stream.c
@@ -332,8 +332,9 @@ char *vlc_stream_ReadLine( stream_t *s )
 }
 
 /* Remove trailing LF/CR */
-while( i_line >= 2 && ( p_line[i_line-2] == '\r' ||
-p_line[i_line-2] == '\n') ) i_line--;
+while( i_line >= 1 &&
+   (p_line[i_line - 1] == '\r' || p_line[i_line - 1] == '\n') )
+i_line--;
 
 /* Make sure the \0 is there */
 p_line[i_line] = '\0';

___
vlc-commits mailing list
vlc-commits@videolan.org
https://mailman.videolan.org/listinfo/vlc-commits


[vlc-commits] vlc_stream_ReadLine(): improve error reporting

2017-05-24 Thread Rémi Denis-Courmont
vlc | branch: master | Rémi Denis-Courmont  | Wed May 24 
19:43:39 2017 +0300| [0379cf77c90da4ecececfc9d3379a969d8736423] | committer: 
Rémi Denis-Courmont

vlc_stream_ReadLine(): improve error reporting

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=0379cf77c90da4ecececfc9d3379a969d8736423
---

 src/input/stream.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/input/stream.c b/src/input/stream.c
index 6045f53cee..e0c70b42cc 100644
--- a/src/input/stream.c
+++ b/src/input/stream.c
@@ -30,6 +30,7 @@
 #include 
 #include 
 #include 
+#include 
 
 #include 
 #include 
@@ -322,8 +323,8 @@ char *vlc_stream_ReadLine( stream_t *s )
 
 if( vlc_iconv( priv->text.conv, &p_in, &i_in, &p_out, &i_out ) == 
(size_t)-1 )
 {
-msg_Err( s, "iconv failed" );
-msg_Dbg( s, "original: %d, in %d, out %d", i_line, (int)i_in, 
(int)i_out );
+msg_Err( s, "conversion error: %s", vlc_strerror_c( errno ) );
+msg_Dbg( s, "original: %d, in %zu, out %zu", i_line, i_in, 
i_out );
 }
 free( p_line );
 p_line = psz_new_line;

___
vlc-commits mailing list
vlc-commits@videolan.org
https://mailman.videolan.org/listinfo/vlc-commits


[vlc-commits] vlc_stream_ReadLine: fix off-by-one

2017-05-24 Thread Rémi Denis-Courmont
vlc | branch: master | Rémi Denis-Courmont  | Wed May 24 
19:47:12 2017 +0300| [dbb8a1891fd41938897b66d58aa8b012695d7f30] | committer: 
Rémi Denis-Courmont

vlc_stream_ReadLine: fix off-by-one

Do not erase the last converted byte. This bug has apparently existed
ever since UTF-16 support was added.

If the conversion fails, this bug resulted in a heap underflow (writing
zero right before the beginning of the buffer).

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=dbb8a1891fd41938897b66d58aa8b012695d7f30
---

 src/input/stream.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/input/stream.c b/src/input/stream.c
index e0c70b42cc..a1eb20ec9e 100644
--- a/src/input/stream.c
+++ b/src/input/stream.c
@@ -336,7 +336,7 @@ char *vlc_stream_ReadLine( stream_t *s )
 p_line[i_line-2] == '\n') ) i_line--;
 
 /* Make sure the \0 is there */
-p_line[i_line-1] = '\0';
+p_line[i_line] = '\0';
 
 return p_line;
 }

___
vlc-commits mailing list
vlc-commits@videolan.org
https://mailman.videolan.org/listinfo/vlc-commits


[vlc-commits] contribs: microdns: Bump to 0.0.7

2017-05-24 Thread Hugo Beauzée-Luyssen
vlc | branch: master | Hugo Beauzée-Luyssen  | Wed May 24 
16:06:22 2017 +0200| [3000c773af0e6f7d4bb17db86d2da4f140588a8e] | committer: 
Hugo Beauzée-Luyssen

contribs: microdns: Bump to 0.0.7

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=3000c773af0e6f7d4bb17db86d2da4f140588a8e
---

 contrib/src/microdns/SHA512SUMS | 2 +-
 contrib/src/microdns/rules.mak  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/contrib/src/microdns/SHA512SUMS b/contrib/src/microdns/SHA512SUMS
index fc0c9f6bd8..604e590410 100644
--- a/contrib/src/microdns/SHA512SUMS
+++ b/contrib/src/microdns/SHA512SUMS
@@ -1 +1 @@
-12fb017312709732bb8295cb6d8e56fef9e311be3c66c8033dcc6b996b2f9f815f3b63c65ad797d5fd9e938960183ea84410b5c9e4e5710aa5198b9dd432db0c
  microdns-0.0.6.tar.gz
+616a9327171f47c18c728383d175e4ec97a07c83a6ddfd69b68d129616d726b47b2cfddd856ccb44e8374d4edb448f2ab0936c54929d9619851323bc79aec0c3
  microdns-0.0.7.tar.gz
diff --git a/contrib/src/microdns/rules.mak b/contrib/src/microdns/rules.mak
index e3ea8a21dd..693d33e73e 100644
--- a/contrib/src/microdns/rules.mak
+++ b/contrib/src/microdns/rules.mak
@@ -1,6 +1,6 @@
 # libmicrodns
 
-LIBMICRODNS_VERSION := 0.0.6
+LIBMICRODNS_VERSION := 0.0.7
 LIBMICRODNS_URL := 
https://github.com/videolabs/libmicrodns/releases/download/$(LIBMICRODNS_VERSION)/microdns-$(LIBMICRODNS_VERSION).tar.gz
 
 ifndef HAVE_MACOSX

___
vlc-commits mailing list
vlc-commits@videolan.org
https://mailman.videolan.org/listinfo/vlc-commits


[vlc-commits] Revert "subtitle: Fix potential memory leaks"

2017-05-24 Thread Hugo Beauzée-Luyssen
vlc/vlc-2.2 | branch: refs/tags/2.2.6 | Hugo Beauzée-Luyssen  
| Wed May 24 12:36:19 2017 +0200| [dbe888f9ca9c3b102478b4a16a3d1d985c267899] | 
committer: Hugo Beauzée-Luyssen

Revert "subtitle: Fix potential memory leaks"

This reverts commit 85f38701259e07a843a7ee1dd51959151a7bb050.

> http://git.videolan.org/gitweb.cgi/vlc/vlc-2.2.git/?a=commit;h=dbe888f9ca9c3b102478b4a16a3d1d985c267899
---

 modules/demux/subtitle.c | 13 +
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/modules/demux/subtitle.c b/modules/demux/subtitle.c
index c4402edfb9..3e40f28065 100644
--- a/modules/demux/subtitle.c
+++ b/modules/demux/subtitle.c
@@ -1667,7 +1667,7 @@ static int ParseJSS( demux_t *p_demux, subtitle_t 
*p_subtitle, int i_idx )
 
 demux_sys_t  *p_sys = p_demux->p_sys;
 text_t   *txt = &p_sys->txt;
-char *psz_text, *psz_orig = NULL;
+char *psz_text, *psz_orig;
 char *psz_text2, *psz_orig2;
 int h1, h2, m1, m2, s1, s2, f1, f2;
 
@@ -1683,7 +1683,6 @@ static int ParseJSS( demux_t *p_demux, subtitle_t 
*p_subtitle, int i_idx )
 /* Parse the main lines */
 for( ;; )
 {
-free(psz_orig);
 const char *s = TextGetLine( txt );
 if( !s )
 return VLC_EGENERIC;
@@ -1774,10 +1773,16 @@ static int ParseJSS( demux_t *p_demux, subtitle_t 
*p_subtitle, int i_idx )
 sscanf( &psz_text[shift], "%d", &p_sys->jss.i_time_resolution 
);
 break;
 }
+free( psz_orig );
+continue;
+}
+else
+/* Unkown type line, probably a comment */
+{
+free( psz_orig );
+continue;
 }
 }
-free( psz_orig );
-psz_orig = NULL;
 
 while( psz_text[ strlen( psz_text ) - 1 ] == '\\' )
 {

___
vlc-commits mailing list
vlc-commits@videolan.org
https://mailman.videolan.org/listinfo/vlc-commits


[vlc-commits] Revert "subtitle: Fix potential memory leaks"

2017-05-24 Thread Hugo Beauzée-Luyssen
vlc/vlc-2.2 | branch: master | Hugo Beauzée-Luyssen  | Wed May 
24 12:36:19 2017 +0200| [dbe888f9ca9c3b102478b4a16a3d1d985c267899] | committer: 
Hugo Beauzée-Luyssen

Revert "subtitle: Fix potential memory leaks"

This reverts commit 85f38701259e07a843a7ee1dd51959151a7bb050.

> http://git.videolan.org/gitweb.cgi/vlc/vlc-2.2.git/?a=commit;h=dbe888f9ca9c3b102478b4a16a3d1d985c267899
---

 modules/demux/subtitle.c | 13 +
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/modules/demux/subtitle.c b/modules/demux/subtitle.c
index c4402edfb9..3e40f28065 100644
--- a/modules/demux/subtitle.c
+++ b/modules/demux/subtitle.c
@@ -1667,7 +1667,7 @@ static int ParseJSS( demux_t *p_demux, subtitle_t 
*p_subtitle, int i_idx )
 
 demux_sys_t  *p_sys = p_demux->p_sys;
 text_t   *txt = &p_sys->txt;
-char *psz_text, *psz_orig = NULL;
+char *psz_text, *psz_orig;
 char *psz_text2, *psz_orig2;
 int h1, h2, m1, m2, s1, s2, f1, f2;
 
@@ -1683,7 +1683,6 @@ static int ParseJSS( demux_t *p_demux, subtitle_t 
*p_subtitle, int i_idx )
 /* Parse the main lines */
 for( ;; )
 {
-free(psz_orig);
 const char *s = TextGetLine( txt );
 if( !s )
 return VLC_EGENERIC;
@@ -1774,10 +1773,16 @@ static int ParseJSS( demux_t *p_demux, subtitle_t 
*p_subtitle, int i_idx )
 sscanf( &psz_text[shift], "%d", &p_sys->jss.i_time_resolution 
);
 break;
 }
+free( psz_orig );
+continue;
+}
+else
+/* Unkown type line, probably a comment */
+{
+free( psz_orig );
+continue;
 }
 }
-free( psz_orig );
-psz_orig = NULL;
 
 while( psz_text[ strlen( psz_text ) - 1 ] == '\\' )
 {

___
vlc-commits mailing list
vlc-commits@videolan.org
https://mailman.videolan.org/listinfo/vlc-commits


[vlc-commits] Revert "subtitle: Fix potential memory leaks"

2017-05-24 Thread Hugo Beauzée-Luyssen
vlc | branch: master | Hugo Beauzée-Luyssen  | Wed May 24 
12:35:34 2017 +0200| [3fe69299a738d29abd01d4dd4c6256a70e6cff85] | committer: 
Hugo Beauzée-Luyssen

Revert "subtitle: Fix potential memory leaks"

This reverts commit c2e45c4ecf2d51820dfe095d89e8b9379c8df4ca.

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=3fe69299a738d29abd01d4dd4c6256a70e6cff85
---

 modules/demux/subtitle.c | 13 +
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/modules/demux/subtitle.c b/modules/demux/subtitle.c
index 9b417ce454..9726bc26bb 100644
--- a/modules/demux/subtitle.c
+++ b/modules/demux/subtitle.c
@@ -1689,7 +1689,7 @@ static int ParseJSS( vlc_object_t *p_obj, 
subs_properties_t *p_props,
  text_t *txt, subtitle_t *p_subtitle, size_t i_idx )
 {
 VLC_UNUSED( i_idx );
-char *psz_text, *psz_orig = NULL;
+char *psz_text, *psz_orig;
 char *psz_text2, *psz_orig2;
 int h1, h2, m1, m2, s1, s2, f1, f2;
 
@@ -1705,7 +1705,6 @@ static int ParseJSS( vlc_object_t *p_obj, 
subs_properties_t *p_props,
 /* Parse the main lines */
 for( ;; )
 {
-free( psz_orig );
 const char *s = TextGetLine( txt );
 if( !s )
 return VLC_EGENERIC;
@@ -1796,10 +1795,16 @@ static int ParseJSS( vlc_object_t *p_obj, 
subs_properties_t *p_props,
 sscanf( &psz_text[shift], "%d", 
&p_props->jss.i_time_resolution );
 break;
 }
+free( psz_orig );
+continue;
+}
+else
+/* Unkown type line, probably a comment */
+{
+free( psz_orig );
+continue;
 }
 }
-free( psz_orig );
-psz_orig = NULL;
 
 while( psz_text[ strlen( psz_text ) - 1 ] == '\\' )
 {

___
vlc-commits mailing list
vlc-commits@videolan.org
https://mailman.videolan.org/listinfo/vlc-commits


[vlc-commits] On the road to 2.2.7

2017-05-24 Thread Hugo Beauzée-Luyssen
vlc/vlc-2.2 | branch: master | Hugo Beauzée-Luyssen  | Wed May 
24 11:30:49 2017 +0200| [c5a99e76a8fa4fa6c248eef152172416a75a6071] | committer: 
Hugo Beauzée-Luyssen

On the road to 2.2.7

> http://git.videolan.org/gitweb.cgi/vlc/vlc-2.2.git/?a=commit;h=c5a99e76a8fa4fa6c248eef152172416a75a6071
---

 NEWS | 3 +++
 configure.ac | 4 ++--
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/NEWS b/NEWS
index 883218244c..e6997d641c 100644
--- a/NEWS
+++ b/NEWS
@@ -1,3 +1,6 @@
+Changes between 2.2.6 and 2.2.7:
+
+
 Changes between 2.2.5.1 and 2.2.6:
 --
 
diff --git a/configure.ac b/configure.ac
index 235edf577b..6ee411a835 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2,10 +2,10 @@ dnl Autoconf settings for vlc
 
 AC_COPYRIGHT([Copyright 1999-2017 VLC authors and VideoLAN])
 
-AC_INIT(vlc, 2.2.6)
+AC_INIT(vlc, 2.2.7)
 VERSION_MAJOR=2
 VERSION_MINOR=2
-VERSION_REVISION=6
+VERSION_REVISION=7
 VERSION_EXTRA=0
 VERSION_DEV=
 

___
vlc-commits mailing list
vlc-commits@videolan.org
https://mailman.videolan.org/listinfo/vlc-commits


[vlc-commits] Tag 2.2.6 : VLC media player 2.2.6 "Umbrella"

2017-05-24 Thread git
[vlc/vlc-2.2] [branch: refs/tags/2.2.6]
Tag:bb65b52be51faabec3378cb62b36c677ee3f2480
> http://git.videolan.org/gitweb.cgi/vlc/vlc-2.2.git?a=tag;h=bb65b52be51faabec3378cb62b36c677ee3f2480

Tagger: Hugo Beauzée-Luyssen 
Date:   Wed May 24 11:27:21 2017 +0200

VLC media player 2.2.6 "Umbrella"

This release fixes 2 subtitle security issues (when parsing JSS
subtitles) and improves the direct3d video output, mostly for nvidia
cards, but not only.
___
vlc-commits mailing list
vlc-commits@videolan.org
https://mailman.videolan.org/listinfo/vlc-commits


[vlc-commits] Update NEWS

2017-05-24 Thread Hugo Beauzée-Luyssen
vlc/vlc-2.2 | branch: master | Hugo Beauzée-Luyssen  | Wed May 
24 11:27:06 2017 +0200| [1aae78981c0222cbde4c6858fcb643295d96712c] | committer: 
Hugo Beauzée-Luyssen

Update NEWS

> http://git.videolan.org/gitweb.cgi/vlc/vlc-2.2.git/?a=commit;h=1aae78981c0222cbde4c6858fcb643295d96712c
---

 NEWS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/NEWS b/NEWS
index 61bd1a100f..883218244c 100644
--- a/NEWS
+++ b/NEWS
@@ -3,6 +3,7 @@ Changes between 2.2.5.1 and 2.2.6:
 
 Video output:
  * Fix systematic green line on nvidia
+ * Fix direct3d SPU texture offsets handling
 
 Demuxer:
  * Fix heap buffer overflows

___
vlc-commits mailing list
vlc-commits@videolan.org
https://mailman.videolan.org/listinfo/vlc-commits