[arch-commits] Commit in libjpeg-turbo/trunk (4 files)

Gaetan Bisson bisson at archlinux.org
Thu Nov 27 16:56:22 UTC 2014


    Date: Thursday, November 27, 2014 @ 17:56:21
  Author: bisson
Revision: 227154

fix FS#42922

Added:
  libjpeg-turbo/trunk/jstdhuff.patch
    (from rev 227153, libjpeg-turbo/trunk/libjpeg-turbo-1.3.1-jstdhuff.patch)
  libjpeg-turbo/trunk/overrun.patch
Modified:
  libjpeg-turbo/trunk/PKGBUILD
Deleted:
  libjpeg-turbo/trunk/libjpeg-turbo-1.3.1-jstdhuff.patch

------------------------------------+
 PKGBUILD                           |   14 -
 jstdhuff.patch                     |  301 +++++++++++++++++++++++++++++++++++
 libjpeg-turbo-1.3.1-jstdhuff.patch |  301 -----------------------------------
 overrun.patch                      |   21 ++
 4 files changed, 329 insertions(+), 308 deletions(-)

Modified: PKGBUILD
===================================================================
--- PKGBUILD	2014-11-27 13:27:41 UTC (rev 227153)
+++ PKGBUILD	2014-11-27 16:56:21 UTC (rev 227154)
@@ -5,7 +5,7 @@
 
 pkgname=libjpeg-turbo
 pkgver=1.3.1
-pkgrel=2
+pkgrel=3
 pkgdesc='JPEG image codec with accelerated baseline compression and decompression'
 url='http://libjpeg-turbo.virtualgl.org/'
 license=('GPL' 'custom')
@@ -12,9 +12,11 @@
 arch=('i686' 'x86_64')
 makedepends=('nasm')
 source=("http://downloads.sourceforge.net/project/${pkgname}/${pkgver}/${pkgname}-${pkgver}.tar.gz"
-        'libjpeg-turbo-1.3.1-jstdhuff.patch')
+        'jstdhuff.patch'
+        'overrun.patch')
 sha1sums=('5fa19252e5ca992cfa40446a0210ceff55fbe468'
-          'f459890257df430dc4e2d8bd3060173a1be7516e')
+          'f459890257df430dc4e2d8bd3060173a1be7516e'
+          '438aba2241b19857bab090c6f61bd3f249f27591')
 
 provides=('libjpeg=8.0.2' 'turbojpeg')
 conflicts=('libjpeg' 'turbojpeg')
@@ -22,10 +24,8 @@
 
 prepare() {
 	cd "${srcdir}/${pkgname}-${pkgver}"
-	# Backport fix for motion JPEG video frames that are encoded without the
-	# default Huffman tables. Fixes HD webcam input in Chromium (FS#42747).
-	# http://sourceforge.net/p/libjpeg-turbo/code/1104/
-	patch -Np0 -i ../libjpeg-turbo-1.3.1-jstdhuff.patch
+	patch -p0 -i ../jstdhuff.patch # FS#42747
+	patch -p1 -i ../overrun.patch # FS#42922
 }
 
 build() {

Copied: libjpeg-turbo/trunk/jstdhuff.patch (from rev 227153, libjpeg-turbo/trunk/libjpeg-turbo-1.3.1-jstdhuff.patch)
===================================================================
--- jstdhuff.patch	                        (rev 0)
+++ jstdhuff.patch	2014-11-27 16:56:21 UTC (rev 227154)
@@ -0,0 +1,301 @@
+Index: jcparam.c
+===================================================================
+--- jcparam.c	(revision 1103)
++++ jcparam.c	(revision 1104)
+@@ -16,6 +16,7 @@
+ #define JPEG_INTERNALS
+ #include "jinclude.h"
+ #include "jpeglib.h"
++#include "jstdhuff.c"
+ 
+ 
+ /*
+@@ -166,116 +167,6 @@
+ 
+ 
+ /*
+- * Huffman table setup routines
+- */
+-
+-LOCAL(void)
+-add_huff_table (j_compress_ptr cinfo,
+-		JHUFF_TBL **htblptr, const UINT8 *bits, const UINT8 *val)
+-/* Define a Huffman table */
+-{
+-  int nsymbols, len;
+-
+-  if (*htblptr == NULL)
+-    *htblptr = jpeg_alloc_huff_table((j_common_ptr) cinfo);
+-
+-  /* Copy the number-of-symbols-of-each-code-length counts */
+-  MEMCOPY((*htblptr)->bits, bits, SIZEOF((*htblptr)->bits));
+-
+-  /* Validate the counts.  We do this here mainly so we can copy the right
+-   * number of symbols from the val[] array, without risking marching off
+-   * the end of memory.  jchuff.c will do a more thorough test later.
+-   */
+-  nsymbols = 0;
+-  for (len = 1; len <= 16; len++)
+-    nsymbols += bits[len];
+-  if (nsymbols < 1 || nsymbols > 256)
+-    ERREXIT(cinfo, JERR_BAD_HUFF_TABLE);
+-
+-  MEMCOPY((*htblptr)->huffval, val, nsymbols * SIZEOF(UINT8));
+-
+-  /* Initialize sent_table FALSE so table will be written to JPEG file. */
+-  (*htblptr)->sent_table = FALSE;
+-}
+-
+-
+-LOCAL(void)
+-std_huff_tables (j_compress_ptr cinfo)
+-/* Set up the standard Huffman tables (cf. JPEG standard section K.3) */
+-/* IMPORTANT: these are only valid for 8-bit data precision! */
+-{
+-  static const UINT8 bits_dc_luminance[17] =
+-    { /* 0-base */ 0, 0, 1, 5, 1, 1, 1, 1, 1, 1, 0, 0, 0, 0, 0, 0, 0 };
+-  static const UINT8 val_dc_luminance[] =
+-    { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11 };
+-  
+-  static const UINT8 bits_dc_chrominance[17] =
+-    { /* 0-base */ 0, 0, 3, 1, 1, 1, 1, 1, 1, 1, 1, 1, 0, 0, 0, 0, 0 };
+-  static const UINT8 val_dc_chrominance[] =
+-    { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11 };
+-  
+-  static const UINT8 bits_ac_luminance[17] =
+-    { /* 0-base */ 0, 0, 2, 1, 3, 3, 2, 4, 3, 5, 5, 4, 4, 0, 0, 1, 0x7d };
+-  static const UINT8 val_ac_luminance[] =
+-    { 0x01, 0x02, 0x03, 0x00, 0x04, 0x11, 0x05, 0x12,
+-      0x21, 0x31, 0x41, 0x06, 0x13, 0x51, 0x61, 0x07,
+-      0x22, 0x71, 0x14, 0x32, 0x81, 0x91, 0xa1, 0x08,
+-      0x23, 0x42, 0xb1, 0xc1, 0x15, 0x52, 0xd1, 0xf0,
+-      0x24, 0x33, 0x62, 0x72, 0x82, 0x09, 0x0a, 0x16,
+-      0x17, 0x18, 0x19, 0x1a, 0x25, 0x26, 0x27, 0x28,
+-      0x29, 0x2a, 0x34, 0x35, 0x36, 0x37, 0x38, 0x39,
+-      0x3a, 0x43, 0x44, 0x45, 0x46, 0x47, 0x48, 0x49,
+-      0x4a, 0x53, 0x54, 0x55, 0x56, 0x57, 0x58, 0x59,
+-      0x5a, 0x63, 0x64, 0x65, 0x66, 0x67, 0x68, 0x69,
+-      0x6a, 0x73, 0x74, 0x75, 0x76, 0x77, 0x78, 0x79,
+-      0x7a, 0x83, 0x84, 0x85, 0x86, 0x87, 0x88, 0x89,
+-      0x8a, 0x92, 0x93, 0x94, 0x95, 0x96, 0x97, 0x98,
+-      0x99, 0x9a, 0xa2, 0xa3, 0xa4, 0xa5, 0xa6, 0xa7,
+-      0xa8, 0xa9, 0xaa, 0xb2, 0xb3, 0xb4, 0xb5, 0xb6,
+-      0xb7, 0xb8, 0xb9, 0xba, 0xc2, 0xc3, 0xc4, 0xc5,
+-      0xc6, 0xc7, 0xc8, 0xc9, 0xca, 0xd2, 0xd3, 0xd4,
+-      0xd5, 0xd6, 0xd7, 0xd8, 0xd9, 0xda, 0xe1, 0xe2,
+-      0xe3, 0xe4, 0xe5, 0xe6, 0xe7, 0xe8, 0xe9, 0xea,
+-      0xf1, 0xf2, 0xf3, 0xf4, 0xf5, 0xf6, 0xf7, 0xf8,
+-      0xf9, 0xfa };
+-  
+-  static const UINT8 bits_ac_chrominance[17] =
+-    { /* 0-base */ 0, 0, 2, 1, 2, 4, 4, 3, 4, 7, 5, 4, 4, 0, 1, 2, 0x77 };
+-  static const UINT8 val_ac_chrominance[] =
+-    { 0x00, 0x01, 0x02, 0x03, 0x11, 0x04, 0x05, 0x21,
+-      0x31, 0x06, 0x12, 0x41, 0x51, 0x07, 0x61, 0x71,
+-      0x13, 0x22, 0x32, 0x81, 0x08, 0x14, 0x42, 0x91,
+-      0xa1, 0xb1, 0xc1, 0x09, 0x23, 0x33, 0x52, 0xf0,
+-      0x15, 0x62, 0x72, 0xd1, 0x0a, 0x16, 0x24, 0x34,
+-      0xe1, 0x25, 0xf1, 0x17, 0x18, 0x19, 0x1a, 0x26,
+-      0x27, 0x28, 0x29, 0x2a, 0x35, 0x36, 0x37, 0x38,
+-      0x39, 0x3a, 0x43, 0x44, 0x45, 0x46, 0x47, 0x48,
+-      0x49, 0x4a, 0x53, 0x54, 0x55, 0x56, 0x57, 0x58,
+-      0x59, 0x5a, 0x63, 0x64, 0x65, 0x66, 0x67, 0x68,
+-      0x69, 0x6a, 0x73, 0x74, 0x75, 0x76, 0x77, 0x78,
+-      0x79, 0x7a, 0x82, 0x83, 0x84, 0x85, 0x86, 0x87,
+-      0x88, 0x89, 0x8a, 0x92, 0x93, 0x94, 0x95, 0x96,
+-      0x97, 0x98, 0x99, 0x9a, 0xa2, 0xa3, 0xa4, 0xa5,
+-      0xa6, 0xa7, 0xa8, 0xa9, 0xaa, 0xb2, 0xb3, 0xb4,
+-      0xb5, 0xb6, 0xb7, 0xb8, 0xb9, 0xba, 0xc2, 0xc3,
+-      0xc4, 0xc5, 0xc6, 0xc7, 0xc8, 0xc9, 0xca, 0xd2,
+-      0xd3, 0xd4, 0xd5, 0xd6, 0xd7, 0xd8, 0xd9, 0xda,
+-      0xe2, 0xe3, 0xe4, 0xe5, 0xe6, 0xe7, 0xe8, 0xe9,
+-      0xea, 0xf2, 0xf3, 0xf4, 0xf5, 0xf6, 0xf7, 0xf8,
+-      0xf9, 0xfa };
+-  
+-  add_huff_table(cinfo, &cinfo->dc_huff_tbl_ptrs[0],
+-		 bits_dc_luminance, val_dc_luminance);
+-  add_huff_table(cinfo, &cinfo->ac_huff_tbl_ptrs[0],
+-		 bits_ac_luminance, val_ac_luminance);
+-  add_huff_table(cinfo, &cinfo->dc_huff_tbl_ptrs[1],
+-		 bits_dc_chrominance, val_dc_chrominance);
+-  add_huff_table(cinfo, &cinfo->ac_huff_tbl_ptrs[1],
+-		 bits_ac_chrominance, val_ac_chrominance);
+-}
+-
+-
+-/*
+  * Default parameter setup for compression.
+  *
+  * Applications that don't choose to use this routine must do their
+@@ -313,7 +204,7 @@
+   /* Set up two quantization tables using default quality of 75 */
+   jpeg_set_quality(cinfo, 75, TRUE);
+   /* Set up two Huffman tables */
+-  std_huff_tables(cinfo);
++  std_huff_tables((j_common_ptr) cinfo);
+ 
+   /* Initialize default arithmetic coding conditioning */
+   for (i = 0; i < NUM_ARITH_TBLS; i++) {
+Index: jstdhuff.c
+===================================================================
+--- jstdhuff.c	(revision 0)
++++ jstdhuff.c	(revision 1104)
+@@ -0,0 +1,133 @@
++/*
++* jstdhuff.c
++*
++* This file was part of the Independent JPEG Group's software:
++* Copyright (C) 1991-1998, Thomas G. Lane.
++* libjpeg-turbo Modifications:
++* Copyright (C) 2013, D. R. Commander.
++* For conditions of distribution and use, see the accompanying README file.
++*
++* This file contains routines to set the default Huffman tables, if they are
++* not already set.
++*/ 
++
++/*
++ * Huffman table setup routines
++ */
++
++LOCAL(void)
++add_huff_table (j_common_ptr cinfo,
++		JHUFF_TBL **htblptr, const UINT8 *bits, const UINT8 *val)
++/* Define a Huffman table */
++{
++  int nsymbols, len;
++
++  if (*htblptr == NULL)
++    *htblptr = jpeg_alloc_huff_table(cinfo);
++  else
++    return;
++
++  /* Copy the number-of-symbols-of-each-code-length counts */
++  MEMCOPY((*htblptr)->bits, bits, SIZEOF((*htblptr)->bits));
++
++  /* Validate the counts.  We do this here mainly so we can copy the right
++   * number of symbols from the val[] array, without risking marching off
++   * the end of memory.  jchuff.c will do a more thorough test later.
++   */
++  nsymbols = 0;
++  for (len = 1; len <= 16; len++)
++    nsymbols += bits[len];
++  if (nsymbols < 1 || nsymbols > 256)
++    ERREXIT(cinfo, JERR_BAD_HUFF_TABLE);
++
++  MEMCOPY((*htblptr)->huffval, val, nsymbols * SIZEOF(UINT8));
++
++  /* Initialize sent_table FALSE so table will be written to JPEG file. */
++  (*htblptr)->sent_table = FALSE;
++}
++
++
++LOCAL(void)
++std_huff_tables (j_common_ptr cinfo)
++/* Set up the standard Huffman tables (cf. JPEG standard section K.3) */
++/* IMPORTANT: these are only valid for 8-bit data precision! */
++{
++  JHUFF_TBL **dc_huff_tbl_ptrs, **ac_huff_tbl_ptrs;
++
++  static const UINT8 bits_dc_luminance[17] =
++    { /* 0-base */ 0, 0, 1, 5, 1, 1, 1, 1, 1, 1, 0, 0, 0, 0, 0, 0, 0 };
++  static const UINT8 val_dc_luminance[] =
++    { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11 };
++  
++  static const UINT8 bits_dc_chrominance[17] =
++    { /* 0-base */ 0, 0, 3, 1, 1, 1, 1, 1, 1, 1, 1, 1, 0, 0, 0, 0, 0 };
++  static const UINT8 val_dc_chrominance[] =
++    { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11 };
++  
++  static const UINT8 bits_ac_luminance[17] =
++    { /* 0-base */ 0, 0, 2, 1, 3, 3, 2, 4, 3, 5, 5, 4, 4, 0, 0, 1, 0x7d };
++  static const UINT8 val_ac_luminance[] =
++    { 0x01, 0x02, 0x03, 0x00, 0x04, 0x11, 0x05, 0x12,
++      0x21, 0x31, 0x41, 0x06, 0x13, 0x51, 0x61, 0x07,
++      0x22, 0x71, 0x14, 0x32, 0x81, 0x91, 0xa1, 0x08,
++      0x23, 0x42, 0xb1, 0xc1, 0x15, 0x52, 0xd1, 0xf0,
++      0x24, 0x33, 0x62, 0x72, 0x82, 0x09, 0x0a, 0x16,
++      0x17, 0x18, 0x19, 0x1a, 0x25, 0x26, 0x27, 0x28,
++      0x29, 0x2a, 0x34, 0x35, 0x36, 0x37, 0x38, 0x39,
++      0x3a, 0x43, 0x44, 0x45, 0x46, 0x47, 0x48, 0x49,
++      0x4a, 0x53, 0x54, 0x55, 0x56, 0x57, 0x58, 0x59,
++      0x5a, 0x63, 0x64, 0x65, 0x66, 0x67, 0x68, 0x69,
++      0x6a, 0x73, 0x74, 0x75, 0x76, 0x77, 0x78, 0x79,
++      0x7a, 0x83, 0x84, 0x85, 0x86, 0x87, 0x88, 0x89,
++      0x8a, 0x92, 0x93, 0x94, 0x95, 0x96, 0x97, 0x98,
++      0x99, 0x9a, 0xa2, 0xa3, 0xa4, 0xa5, 0xa6, 0xa7,
++      0xa8, 0xa9, 0xaa, 0xb2, 0xb3, 0xb4, 0xb5, 0xb6,
++      0xb7, 0xb8, 0xb9, 0xba, 0xc2, 0xc3, 0xc4, 0xc5,
++      0xc6, 0xc7, 0xc8, 0xc9, 0xca, 0xd2, 0xd3, 0xd4,
++      0xd5, 0xd6, 0xd7, 0xd8, 0xd9, 0xda, 0xe1, 0xe2,
++      0xe3, 0xe4, 0xe5, 0xe6, 0xe7, 0xe8, 0xe9, 0xea,
++      0xf1, 0xf2, 0xf3, 0xf4, 0xf5, 0xf6, 0xf7, 0xf8,
++      0xf9, 0xfa };
++  
++  static const UINT8 bits_ac_chrominance[17] =
++    { /* 0-base */ 0, 0, 2, 1, 2, 4, 4, 3, 4, 7, 5, 4, 4, 0, 1, 2, 0x77 };
++  static const UINT8 val_ac_chrominance[] =
++    { 0x00, 0x01, 0x02, 0x03, 0x11, 0x04, 0x05, 0x21,
++      0x31, 0x06, 0x12, 0x41, 0x51, 0x07, 0x61, 0x71,
++      0x13, 0x22, 0x32, 0x81, 0x08, 0x14, 0x42, 0x91,
++      0xa1, 0xb1, 0xc1, 0x09, 0x23, 0x33, 0x52, 0xf0,
++      0x15, 0x62, 0x72, 0xd1, 0x0a, 0x16, 0x24, 0x34,
++      0xe1, 0x25, 0xf1, 0x17, 0x18, 0x19, 0x1a, 0x26,
++      0x27, 0x28, 0x29, 0x2a, 0x35, 0x36, 0x37, 0x38,
++      0x39, 0x3a, 0x43, 0x44, 0x45, 0x46, 0x47, 0x48,
++      0x49, 0x4a, 0x53, 0x54, 0x55, 0x56, 0x57, 0x58,
++      0x59, 0x5a, 0x63, 0x64, 0x65, 0x66, 0x67, 0x68,
++      0x69, 0x6a, 0x73, 0x74, 0x75, 0x76, 0x77, 0x78,
++      0x79, 0x7a, 0x82, 0x83, 0x84, 0x85, 0x86, 0x87,
++      0x88, 0x89, 0x8a, 0x92, 0x93, 0x94, 0x95, 0x96,
++      0x97, 0x98, 0x99, 0x9a, 0xa2, 0xa3, 0xa4, 0xa5,
++      0xa6, 0xa7, 0xa8, 0xa9, 0xaa, 0xb2, 0xb3, 0xb4,
++      0xb5, 0xb6, 0xb7, 0xb8, 0xb9, 0xba, 0xc2, 0xc3,
++      0xc4, 0xc5, 0xc6, 0xc7, 0xc8, 0xc9, 0xca, 0xd2,
++      0xd3, 0xd4, 0xd5, 0xd6, 0xd7, 0xd8, 0xd9, 0xda,
++      0xe2, 0xe3, 0xe4, 0xe5, 0xe6, 0xe7, 0xe8, 0xe9,
++      0xea, 0xf2, 0xf3, 0xf4, 0xf5, 0xf6, 0xf7, 0xf8,
++      0xf9, 0xfa };
++
++  if (cinfo->is_decompressor) {
++    dc_huff_tbl_ptrs = ((j_decompress_ptr)cinfo)->dc_huff_tbl_ptrs;
++    ac_huff_tbl_ptrs = ((j_decompress_ptr)cinfo)->ac_huff_tbl_ptrs;
++  } else {
++    dc_huff_tbl_ptrs = ((j_compress_ptr)cinfo)->dc_huff_tbl_ptrs;
++    ac_huff_tbl_ptrs = ((j_compress_ptr)cinfo)->ac_huff_tbl_ptrs;
++  }
++
++  add_huff_table(cinfo, &dc_huff_tbl_ptrs[0], bits_dc_luminance,
++                 val_dc_luminance);
++  add_huff_table(cinfo, &ac_huff_tbl_ptrs[0], bits_ac_luminance,
++                 val_ac_luminance);
++  add_huff_table(cinfo, &dc_huff_tbl_ptrs[1], bits_dc_chrominance,
++                 val_dc_chrominance);
++  add_huff_table(cinfo, &ac_huff_tbl_ptrs[1], bits_ac_chrominance,
++                 val_ac_chrominance);
++}
+Index: jdhuff.c
+===================================================================
+--- jdhuff.c	(revision 1103)
++++ jdhuff.c	(revision 1104)
+@@ -21,6 +21,7 @@
+ #include "jpeglib.h"
+ #include "jdhuff.h"		/* Declarations shared with jdphuff.c */
+ #include "jpegcomp.h"
++#include "jstdhuff.c"
+ 
+ 
+ /*
+@@ -795,6 +796,12 @@
+   huff_entropy_ptr entropy;
+   int i;
+ 
++  /* Motion JPEG frames typically do not include the Huffman tables if they
++     are the default tables.  Thus, if the tables are not set by the time
++     the Huffman decoder is initialized (usually within the body of
++     jpeg_start_decompress()), we set them to default values. */
++  std_huff_tables((j_common_ptr) cinfo);
++
+   entropy = (huff_entropy_ptr)
+     (*cinfo->mem->alloc_small) ((j_common_ptr) cinfo, JPOOL_IMAGE,
+ 				SIZEOF(huff_entropy_decoder));

Deleted: libjpeg-turbo-1.3.1-jstdhuff.patch
===================================================================
--- libjpeg-turbo-1.3.1-jstdhuff.patch	2014-11-27 13:27:41 UTC (rev 227153)
+++ libjpeg-turbo-1.3.1-jstdhuff.patch	2014-11-27 16:56:21 UTC (rev 227154)
@@ -1,301 +0,0 @@
-Index: jcparam.c
-===================================================================
---- jcparam.c	(revision 1103)
-+++ jcparam.c	(revision 1104)
-@@ -16,6 +16,7 @@
- #define JPEG_INTERNALS
- #include "jinclude.h"
- #include "jpeglib.h"
-+#include "jstdhuff.c"
- 
- 
- /*
-@@ -166,116 +167,6 @@
- 
- 
- /*
-- * Huffman table setup routines
-- */
--
--LOCAL(void)
--add_huff_table (j_compress_ptr cinfo,
--		JHUFF_TBL **htblptr, const UINT8 *bits, const UINT8 *val)
--/* Define a Huffman table */
--{
--  int nsymbols, len;
--
--  if (*htblptr == NULL)
--    *htblptr = jpeg_alloc_huff_table((j_common_ptr) cinfo);
--
--  /* Copy the number-of-symbols-of-each-code-length counts */
--  MEMCOPY((*htblptr)->bits, bits, SIZEOF((*htblptr)->bits));
--
--  /* Validate the counts.  We do this here mainly so we can copy the right
--   * number of symbols from the val[] array, without risking marching off
--   * the end of memory.  jchuff.c will do a more thorough test later.
--   */
--  nsymbols = 0;
--  for (len = 1; len <= 16; len++)
--    nsymbols += bits[len];
--  if (nsymbols < 1 || nsymbols > 256)
--    ERREXIT(cinfo, JERR_BAD_HUFF_TABLE);
--
--  MEMCOPY((*htblptr)->huffval, val, nsymbols * SIZEOF(UINT8));
--
--  /* Initialize sent_table FALSE so table will be written to JPEG file. */
--  (*htblptr)->sent_table = FALSE;
--}
--
--
--LOCAL(void)
--std_huff_tables (j_compress_ptr cinfo)
--/* Set up the standard Huffman tables (cf. JPEG standard section K.3) */
--/* IMPORTANT: these are only valid for 8-bit data precision! */
--{
--  static const UINT8 bits_dc_luminance[17] =
--    { /* 0-base */ 0, 0, 1, 5, 1, 1, 1, 1, 1, 1, 0, 0, 0, 0, 0, 0, 0 };
--  static const UINT8 val_dc_luminance[] =
--    { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11 };
--  
--  static const UINT8 bits_dc_chrominance[17] =
--    { /* 0-base */ 0, 0, 3, 1, 1, 1, 1, 1, 1, 1, 1, 1, 0, 0, 0, 0, 0 };
--  static const UINT8 val_dc_chrominance[] =
--    { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11 };
--  
--  static const UINT8 bits_ac_luminance[17] =
--    { /* 0-base */ 0, 0, 2, 1, 3, 3, 2, 4, 3, 5, 5, 4, 4, 0, 0, 1, 0x7d };
--  static const UINT8 val_ac_luminance[] =
--    { 0x01, 0x02, 0x03, 0x00, 0x04, 0x11, 0x05, 0x12,
--      0x21, 0x31, 0x41, 0x06, 0x13, 0x51, 0x61, 0x07,
--      0x22, 0x71, 0x14, 0x32, 0x81, 0x91, 0xa1, 0x08,
--      0x23, 0x42, 0xb1, 0xc1, 0x15, 0x52, 0xd1, 0xf0,
--      0x24, 0x33, 0x62, 0x72, 0x82, 0x09, 0x0a, 0x16,
--      0x17, 0x18, 0x19, 0x1a, 0x25, 0x26, 0x27, 0x28,
--      0x29, 0x2a, 0x34, 0x35, 0x36, 0x37, 0x38, 0x39,
--      0x3a, 0x43, 0x44, 0x45, 0x46, 0x47, 0x48, 0x49,
--      0x4a, 0x53, 0x54, 0x55, 0x56, 0x57, 0x58, 0x59,
--      0x5a, 0x63, 0x64, 0x65, 0x66, 0x67, 0x68, 0x69,
--      0x6a, 0x73, 0x74, 0x75, 0x76, 0x77, 0x78, 0x79,
--      0x7a, 0x83, 0x84, 0x85, 0x86, 0x87, 0x88, 0x89,
--      0x8a, 0x92, 0x93, 0x94, 0x95, 0x96, 0x97, 0x98,
--      0x99, 0x9a, 0xa2, 0xa3, 0xa4, 0xa5, 0xa6, 0xa7,
--      0xa8, 0xa9, 0xaa, 0xb2, 0xb3, 0xb4, 0xb5, 0xb6,
--      0xb7, 0xb8, 0xb9, 0xba, 0xc2, 0xc3, 0xc4, 0xc5,
--      0xc6, 0xc7, 0xc8, 0xc9, 0xca, 0xd2, 0xd3, 0xd4,
--      0xd5, 0xd6, 0xd7, 0xd8, 0xd9, 0xda, 0xe1, 0xe2,
--      0xe3, 0xe4, 0xe5, 0xe6, 0xe7, 0xe8, 0xe9, 0xea,
--      0xf1, 0xf2, 0xf3, 0xf4, 0xf5, 0xf6, 0xf7, 0xf8,
--      0xf9, 0xfa };
--  
--  static const UINT8 bits_ac_chrominance[17] =
--    { /* 0-base */ 0, 0, 2, 1, 2, 4, 4, 3, 4, 7, 5, 4, 4, 0, 1, 2, 0x77 };
--  static const UINT8 val_ac_chrominance[] =
--    { 0x00, 0x01, 0x02, 0x03, 0x11, 0x04, 0x05, 0x21,
--      0x31, 0x06, 0x12, 0x41, 0x51, 0x07, 0x61, 0x71,
--      0x13, 0x22, 0x32, 0x81, 0x08, 0x14, 0x42, 0x91,
--      0xa1, 0xb1, 0xc1, 0x09, 0x23, 0x33, 0x52, 0xf0,
--      0x15, 0x62, 0x72, 0xd1, 0x0a, 0x16, 0x24, 0x34,
--      0xe1, 0x25, 0xf1, 0x17, 0x18, 0x19, 0x1a, 0x26,
--      0x27, 0x28, 0x29, 0x2a, 0x35, 0x36, 0x37, 0x38,
--      0x39, 0x3a, 0x43, 0x44, 0x45, 0x46, 0x47, 0x48,
--      0x49, 0x4a, 0x53, 0x54, 0x55, 0x56, 0x57, 0x58,
--      0x59, 0x5a, 0x63, 0x64, 0x65, 0x66, 0x67, 0x68,
--      0x69, 0x6a, 0x73, 0x74, 0x75, 0x76, 0x77, 0x78,
--      0x79, 0x7a, 0x82, 0x83, 0x84, 0x85, 0x86, 0x87,
--      0x88, 0x89, 0x8a, 0x92, 0x93, 0x94, 0x95, 0x96,
--      0x97, 0x98, 0x99, 0x9a, 0xa2, 0xa3, 0xa4, 0xa5,
--      0xa6, 0xa7, 0xa8, 0xa9, 0xaa, 0xb2, 0xb3, 0xb4,
--      0xb5, 0xb6, 0xb7, 0xb8, 0xb9, 0xba, 0xc2, 0xc3,
--      0xc4, 0xc5, 0xc6, 0xc7, 0xc8, 0xc9, 0xca, 0xd2,
--      0xd3, 0xd4, 0xd5, 0xd6, 0xd7, 0xd8, 0xd9, 0xda,
--      0xe2, 0xe3, 0xe4, 0xe5, 0xe6, 0xe7, 0xe8, 0xe9,
--      0xea, 0xf2, 0xf3, 0xf4, 0xf5, 0xf6, 0xf7, 0xf8,
--      0xf9, 0xfa };
--  
--  add_huff_table(cinfo, &cinfo->dc_huff_tbl_ptrs[0],
--		 bits_dc_luminance, val_dc_luminance);
--  add_huff_table(cinfo, &cinfo->ac_huff_tbl_ptrs[0],
--		 bits_ac_luminance, val_ac_luminance);
--  add_huff_table(cinfo, &cinfo->dc_huff_tbl_ptrs[1],
--		 bits_dc_chrominance, val_dc_chrominance);
--  add_huff_table(cinfo, &cinfo->ac_huff_tbl_ptrs[1],
--		 bits_ac_chrominance, val_ac_chrominance);
--}
--
--
--/*
-  * Default parameter setup for compression.
-  *
-  * Applications that don't choose to use this routine must do their
-@@ -313,7 +204,7 @@
-   /* Set up two quantization tables using default quality of 75 */
-   jpeg_set_quality(cinfo, 75, TRUE);
-   /* Set up two Huffman tables */
--  std_huff_tables(cinfo);
-+  std_huff_tables((j_common_ptr) cinfo);
- 
-   /* Initialize default arithmetic coding conditioning */
-   for (i = 0; i < NUM_ARITH_TBLS; i++) {
-Index: jstdhuff.c
-===================================================================
---- jstdhuff.c	(revision 0)
-+++ jstdhuff.c	(revision 1104)
-@@ -0,0 +1,133 @@
-+/*
-+* jstdhuff.c
-+*
-+* This file was part of the Independent JPEG Group's software:
-+* Copyright (C) 1991-1998, Thomas G. Lane.
-+* libjpeg-turbo Modifications:
-+* Copyright (C) 2013, D. R. Commander.
-+* For conditions of distribution and use, see the accompanying README file.
-+*
-+* This file contains routines to set the default Huffman tables, if they are
-+* not already set.
-+*/ 
-+
-+/*
-+ * Huffman table setup routines
-+ */
-+
-+LOCAL(void)
-+add_huff_table (j_common_ptr cinfo,
-+		JHUFF_TBL **htblptr, const UINT8 *bits, const UINT8 *val)
-+/* Define a Huffman table */
-+{
-+  int nsymbols, len;
-+
-+  if (*htblptr == NULL)
-+    *htblptr = jpeg_alloc_huff_table(cinfo);
-+  else
-+    return;
-+
-+  /* Copy the number-of-symbols-of-each-code-length counts */
-+  MEMCOPY((*htblptr)->bits, bits, SIZEOF((*htblptr)->bits));
-+
-+  /* Validate the counts.  We do this here mainly so we can copy the right
-+   * number of symbols from the val[] array, without risking marching off
-+   * the end of memory.  jchuff.c will do a more thorough test later.
-+   */
-+  nsymbols = 0;
-+  for (len = 1; len <= 16; len++)
-+    nsymbols += bits[len];
-+  if (nsymbols < 1 || nsymbols > 256)
-+    ERREXIT(cinfo, JERR_BAD_HUFF_TABLE);
-+
-+  MEMCOPY((*htblptr)->huffval, val, nsymbols * SIZEOF(UINT8));
-+
-+  /* Initialize sent_table FALSE so table will be written to JPEG file. */
-+  (*htblptr)->sent_table = FALSE;
-+}
-+
-+
-+LOCAL(void)
-+std_huff_tables (j_common_ptr cinfo)
-+/* Set up the standard Huffman tables (cf. JPEG standard section K.3) */
-+/* IMPORTANT: these are only valid for 8-bit data precision! */
-+{
-+  JHUFF_TBL **dc_huff_tbl_ptrs, **ac_huff_tbl_ptrs;
-+
-+  static const UINT8 bits_dc_luminance[17] =
-+    { /* 0-base */ 0, 0, 1, 5, 1, 1, 1, 1, 1, 1, 0, 0, 0, 0, 0, 0, 0 };
-+  static const UINT8 val_dc_luminance[] =
-+    { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11 };
-+  
-+  static const UINT8 bits_dc_chrominance[17] =
-+    { /* 0-base */ 0, 0, 3, 1, 1, 1, 1, 1, 1, 1, 1, 1, 0, 0, 0, 0, 0 };
-+  static const UINT8 val_dc_chrominance[] =
-+    { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11 };
-+  
-+  static const UINT8 bits_ac_luminance[17] =
-+    { /* 0-base */ 0, 0, 2, 1, 3, 3, 2, 4, 3, 5, 5, 4, 4, 0, 0, 1, 0x7d };
-+  static const UINT8 val_ac_luminance[] =
-+    { 0x01, 0x02, 0x03, 0x00, 0x04, 0x11, 0x05, 0x12,
-+      0x21, 0x31, 0x41, 0x06, 0x13, 0x51, 0x61, 0x07,
-+      0x22, 0x71, 0x14, 0x32, 0x81, 0x91, 0xa1, 0x08,
-+      0x23, 0x42, 0xb1, 0xc1, 0x15, 0x52, 0xd1, 0xf0,
-+      0x24, 0x33, 0x62, 0x72, 0x82, 0x09, 0x0a, 0x16,
-+      0x17, 0x18, 0x19, 0x1a, 0x25, 0x26, 0x27, 0x28,
-+      0x29, 0x2a, 0x34, 0x35, 0x36, 0x37, 0x38, 0x39,
-+      0x3a, 0x43, 0x44, 0x45, 0x46, 0x47, 0x48, 0x49,
-+      0x4a, 0x53, 0x54, 0x55, 0x56, 0x57, 0x58, 0x59,
-+      0x5a, 0x63, 0x64, 0x65, 0x66, 0x67, 0x68, 0x69,
-+      0x6a, 0x73, 0x74, 0x75, 0x76, 0x77, 0x78, 0x79,
-+      0x7a, 0x83, 0x84, 0x85, 0x86, 0x87, 0x88, 0x89,
-+      0x8a, 0x92, 0x93, 0x94, 0x95, 0x96, 0x97, 0x98,
-+      0x99, 0x9a, 0xa2, 0xa3, 0xa4, 0xa5, 0xa6, 0xa7,
-+      0xa8, 0xa9, 0xaa, 0xb2, 0xb3, 0xb4, 0xb5, 0xb6,
-+      0xb7, 0xb8, 0xb9, 0xba, 0xc2, 0xc3, 0xc4, 0xc5,
-+      0xc6, 0xc7, 0xc8, 0xc9, 0xca, 0xd2, 0xd3, 0xd4,
-+      0xd5, 0xd6, 0xd7, 0xd8, 0xd9, 0xda, 0xe1, 0xe2,
-+      0xe3, 0xe4, 0xe5, 0xe6, 0xe7, 0xe8, 0xe9, 0xea,
-+      0xf1, 0xf2, 0xf3, 0xf4, 0xf5, 0xf6, 0xf7, 0xf8,
-+      0xf9, 0xfa };
-+  
-+  static const UINT8 bits_ac_chrominance[17] =
-+    { /* 0-base */ 0, 0, 2, 1, 2, 4, 4, 3, 4, 7, 5, 4, 4, 0, 1, 2, 0x77 };
-+  static const UINT8 val_ac_chrominance[] =
-+    { 0x00, 0x01, 0x02, 0x03, 0x11, 0x04, 0x05, 0x21,
-+      0x31, 0x06, 0x12, 0x41, 0x51, 0x07, 0x61, 0x71,
-+      0x13, 0x22, 0x32, 0x81, 0x08, 0x14, 0x42, 0x91,
-+      0xa1, 0xb1, 0xc1, 0x09, 0x23, 0x33, 0x52, 0xf0,
-+      0x15, 0x62, 0x72, 0xd1, 0x0a, 0x16, 0x24, 0x34,
-+      0xe1, 0x25, 0xf1, 0x17, 0x18, 0x19, 0x1a, 0x26,
-+      0x27, 0x28, 0x29, 0x2a, 0x35, 0x36, 0x37, 0x38,
-+      0x39, 0x3a, 0x43, 0x44, 0x45, 0x46, 0x47, 0x48,
-+      0x49, 0x4a, 0x53, 0x54, 0x55, 0x56, 0x57, 0x58,
-+      0x59, 0x5a, 0x63, 0x64, 0x65, 0x66, 0x67, 0x68,
-+      0x69, 0x6a, 0x73, 0x74, 0x75, 0x76, 0x77, 0x78,
-+      0x79, 0x7a, 0x82, 0x83, 0x84, 0x85, 0x86, 0x87,
-+      0x88, 0x89, 0x8a, 0x92, 0x93, 0x94, 0x95, 0x96,
-+      0x97, 0x98, 0x99, 0x9a, 0xa2, 0xa3, 0xa4, 0xa5,
-+      0xa6, 0xa7, 0xa8, 0xa9, 0xaa, 0xb2, 0xb3, 0xb4,
-+      0xb5, 0xb6, 0xb7, 0xb8, 0xb9, 0xba, 0xc2, 0xc3,
-+      0xc4, 0xc5, 0xc6, 0xc7, 0xc8, 0xc9, 0xca, 0xd2,
-+      0xd3, 0xd4, 0xd5, 0xd6, 0xd7, 0xd8, 0xd9, 0xda,
-+      0xe2, 0xe3, 0xe4, 0xe5, 0xe6, 0xe7, 0xe8, 0xe9,
-+      0xea, 0xf2, 0xf3, 0xf4, 0xf5, 0xf6, 0xf7, 0xf8,
-+      0xf9, 0xfa };
-+
-+  if (cinfo->is_decompressor) {
-+    dc_huff_tbl_ptrs = ((j_decompress_ptr)cinfo)->dc_huff_tbl_ptrs;
-+    ac_huff_tbl_ptrs = ((j_decompress_ptr)cinfo)->ac_huff_tbl_ptrs;
-+  } else {
-+    dc_huff_tbl_ptrs = ((j_compress_ptr)cinfo)->dc_huff_tbl_ptrs;
-+    ac_huff_tbl_ptrs = ((j_compress_ptr)cinfo)->ac_huff_tbl_ptrs;
-+  }
-+
-+  add_huff_table(cinfo, &dc_huff_tbl_ptrs[0], bits_dc_luminance,
-+                 val_dc_luminance);
-+  add_huff_table(cinfo, &ac_huff_tbl_ptrs[0], bits_ac_luminance,
-+                 val_ac_luminance);
-+  add_huff_table(cinfo, &dc_huff_tbl_ptrs[1], bits_dc_chrominance,
-+                 val_dc_chrominance);
-+  add_huff_table(cinfo, &ac_huff_tbl_ptrs[1], bits_ac_chrominance,
-+                 val_ac_chrominance);
-+}
-Index: jdhuff.c
-===================================================================
---- jdhuff.c	(revision 1103)
-+++ jdhuff.c	(revision 1104)
-@@ -21,6 +21,7 @@
- #include "jpeglib.h"
- #include "jdhuff.h"		/* Declarations shared with jdphuff.c */
- #include "jpegcomp.h"
-+#include "jstdhuff.c"
- 
- 
- /*
-@@ -795,6 +796,12 @@
-   huff_entropy_ptr entropy;
-   int i;
- 
-+  /* Motion JPEG frames typically do not include the Huffman tables if they
-+     are the default tables.  Thus, if the tables are not set by the time
-+     the Huffman decoder is initialized (usually within the body of
-+     jpeg_start_decompress()), we set them to default values. */
-+  std_huff_tables((j_common_ptr) cinfo);
-+
-   entropy = (huff_entropy_ptr)
-     (*cinfo->mem->alloc_small) ((j_common_ptr) cinfo, JPOOL_IMAGE,
- 				SIZEOF(huff_entropy_decoder));

Added: overrun.patch
===================================================================
--- overrun.patch	                        (rev 0)
+++ overrun.patch	2014-11-27 16:56:21 UTC (rev 227154)
@@ -0,0 +1,21 @@
+diff -Naur old/jchuff.c new/jchuff.c
+--- old/jchuff.c	2014-11-27 06:48:21.009442853 -1000
++++ new/jchuff.c	2014-11-27 06:48:46.289700004 -1000
+@@ -391,7 +391,16 @@
+ #endif
+ 
+ 
+-#define BUFSIZE (DCTSIZE2 * 2)
++/* Although it is exceedingly rare, it is possible for a Huffman-encoded
++ * coefficient block to be larger than the 128-byte unencoded block.  For each
++ * of the 64 coefficients, PUT_BITS is invoked twice, and each invocation can
++ * theoretically store 16 bits (for a maximum of 2048 bits or 256 bytes per
++ * encoded block.)  If, for instance, one artificially sets the AC
++ * coefficients to alternating values of 32767 and -32768 (using the JPEG
++ * scanning order-- 1, 8, 16, etc.), then this will produce an encoded block
++ * larger than 200 bytes.
++ */
++#define BUFSIZE (DCTSIZE2 * 4)
+ 
+ #define LOAD_BUFFER() { \
+   if (state->free_in_buffer < BUFSIZE) { \



More information about the arch-commits mailing list