[SCM] handbrake/master: Consider CFLAGS, CPPFLAGS and LDFLAGS in libhb build and fix format string security errors.
fabian-guest at users.alioth.debian.org
fabian-guest at users.alioth.debian.org
Sun Jun 10 21:19:25 UTC 2012
The following commit has been merged in the master branch:
commit 7e4c29ccdaa49e8eabbdb6f491b04c85c6f16233
Author: Fabian Greffrath <fabian at greffrath.com>
Date: Sun Jun 10 23:19:16 2012 +0200
Consider CFLAGS, CPPFLAGS and LDFLAGS in libhb build and fix format string security errors.
diff --git a/debian/patches/0012-FLAGS.patch b/debian/patches/0012-FLAGS.patch
new file mode 100644
index 0000000..3358399
--- /dev/null
+++ b/debian/patches/0012-FLAGS.patch
@@ -0,0 +1,26 @@
+Author: Fabian Greffrath <fabian+debian at greffrath.com>
+Subject: Consider CFLAGS, CPPFLAGS and LDFLAGS in libhb build.
+
+--- a/make/include/gcc.defs
++++ b/make/include/gcc.defs
+@@ -67,15 +67,15 @@ GCC.args.L = -L$(1)
+ GCC.args.l = -l$(1)
+ GCC.args.end = -Wl,--end-group
+
+-GCC.args.extra =
++GCC.args.extra = $(CFLAGS) $(CPPFLAGS)
+ GCC.args.extra.h_o =
+ GCC.args.extra.c_o =
+-GCC.args.extra.dylib =
+-GCC.args.extra.exe =
++GCC.args.extra.dylib = $(LDFLAGS) -Wl,-z,defs -Wl,--as-needed
++GCC.args.extra.exe = $(LDFLAGS) -Wl,-z,defs -Wl,--as-needed
+ GCC.args.extra.hpp_o =
+ GCC.args.extra.cpp_o =
+-GCC.args.extra.dylib++ =
+-GCC.args.extra.exe++ =
++GCC.args.extra.dylib++ = $(LDFLAGS) -Wl,-z,defs -Wl,--as-needed
++GCC.args.extra.exe++ = $(LDFLAGS) -Wl,-z,defs -Wl,--as-needed
+
+ ###############################################################################
+
diff --git a/debian/patches/0013-format-security.patch b/debian/patches/0013-format-security.patch
new file mode 100644
index 0000000..a995b52
--- /dev/null
+++ b/debian/patches/0013-format-security.patch
@@ -0,0 +1,25 @@
+Author: Fabian Greffrath <fabian+debian at greffrath.com>
+Subject: Fix format string security errors.
+
+--- a/libhb/deccc608sub.c
++++ b/libhb/deccc608sub.c
+@@ -1607,7 +1607,7 @@ int write_cc_buffer_as_srt (struct eia60
+ if (debug_608)
+ {
+ hb_log ("\n- - - SRT caption - - -\n");
+- hb_log (timeline);
++ hb_log ("%s", timeline);
+ }
+ //fwrite (enc_buffer,enc_buffer_used,1,wb->fh);
+ XMLRPC_APPEND(wb->enc_buffer,wb->enc_buffer_used);
+--- a/libhb/decmpeg2.c
++++ b/libhb/decmpeg2.c
+@@ -644,7 +644,7 @@ static int hb_libmpeg2_decode( hb_libmpe
+ hb_audio_t *audio = hb_list_item( m->title->list_audio, 0 );
+ if( audio )
+ {
+- snprintf( subtitle->iso639_2, sizeof( subtitle->iso639_2 ),
++ snprintf( "%s", subtitle->iso639_2, sizeof( subtitle->iso639_2 ),
+ audio->config.lang.iso639_2);
+ } else {
+ snprintf( subtitle->iso639_2, sizeof( subtitle->iso639_2 ), "und");
diff --git a/debian/patches/series b/debian/patches/series
index 61ed664..ed26d00 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -9,3 +9,5 @@
#0009-Remove-forcing-PGS-subtitles-to-be-included.patch
0010-Remove-FAAC-dependency.patch
0011-First-try-at-removing-some-of-mp4v2.patch
+0012-FLAGS.patch
+0013-format-security.patch
--
handbrake packaging
More information about the pkg-multimedia-commits
mailing list