* apt-inst/contrib/extracttar.cc:

  - let apt-utils work with encoded tar headers if uid/gid are large.
    Thanks to Nobuhiro Hayashi for the patch! (Closes: #330162)

diff --cc apt-pkg/contrib/strutl.cc
index 987f4c3a48414ec99cab139feccc8d99841b9425,daf87c87fa30ffb4e338bef98261cddba2546b53..f3704581044252ee39fb06959932bda0aaf402a0
--- 1/apt-pkg/contrib/strutl.cc
--- 2/apt-pkg/contrib/strutl.cc
+++ b/apt-pkg/contrib/strutl.cc
@@@ -968,6 -968,24 +968,23 @@@ bool StrToNum(const char *Str,unsigned 
     return true;
  }
                                                                        /*}}}*/
 -bool Base256ToNum(const char *Str,unsigned long &Res,unsigned Len)
+ // Base256ToNum - Convert a fixed length binary to a number             /*{{{*/
+ // ---------------------------------------------------------------------
+ /* This is used in decoding the 256bit encoded fixed length fields in
+    tar files */
 -   int i;
++bool Base256ToNum(const char *Str,unsigned long &Res,unsigned int Len)
+ {
 -      for(i=1; i<Len; i++)
+    if ((Str[0] & 0x80) == 0)
+       return false;
+    else
+    {
+       Res = Str[0] & 0x7F;
++      for(unsigned int i = 1; i < Len; ++i)
+          Res = (Res<<8) + Str[i];
+       return true;
+    }
+ }
+                                                                       /*}}}*/
  // HexDigit - Convert a hex character into an integer                 /*{{{*/
  // ---------------------------------------------------------------------
  /* Helper for Hex2Num */

diff --cc apt-pkg/contrib/strutl.h
index a457ff047da314d428bca60455ebfac643e192b2,591c992d035fbe649bc75b1d116ed9b83385b8bf..6e0e253cf35e918a4629884384e607baf41c4adf
--- 1/apt-pkg/contrib/strutl.h
--- 2/apt-pkg/contrib/strutl.h
+++ b/apt-pkg/contrib/strutl.h
@@@ -52,6 -52,7 +52,7 @@@ string LookupTag(const string &Message,
  int StringToBool(const string &Text,int Default = -1);
  bool ReadMessages(int Fd, vector<string> &List);
  bool StrToNum(const char *Str,unsigned long &Res,unsigned Len,unsigned Base = 0);
 -bool Base256ToNum(const char *Str,unsigned long &Res,unsigned Len);
++bool Base256ToNum(const char *Str,unsigned long &Res,unsigned int Len);
  bool Hex2Num(const string &Str,unsigned char *Num,unsigned int Length);
  bool TokSplitString(char Tok,char *Input,char **List,
                    unsigned long ListMax);

diff --cc debian/changelog
index 6028cc8f4db954669c7efe4b2497404a86397176,96a2bd15b02a66ecdc6a69bb8bfc78bc8a407929..3a59ac1712ebc296a45745350aa34d25532ebae0
--- 1/debian/changelog
--- 2/debian/changelog
+++ b/debian/changelog
@@@ -1,37 -1,3 +1,40 @@@
-  -- David Kalnischkies <kalnischkies@gmail.com>  Wed, 12 Jan 2011 23:59:38 +0100
 +apt (0.8.11+wheezy) unstable; urgency=low
 +
 +  * apt-pkg/depcache.cc:
 +    - add SetCandidateRelease() to set a candidate version and
 +      the candidates of dependencies if needed to a specified
 +      release (Closes: #572709)
 +  * cmdline/apt-get.cc:
 +    - if --print-uris is used don't setup downloader as we don't need
 +      progress, lock nor the directories it would create otherwise
 +    - show dependencies of essential packages which are going to remove
 +      only if they cause the remove of this essential (Closes: #601961)
 +    - keep not installed garbage packages uninstalled instead of showing
 +      in the autoremove section and installing those (Closes: #604222)
 +    - change pkg/release behavior to use the new SetCandidateRelease
 +      so installing packages from experimental or backports is easier
 +    - really do not show packages in the extra section if they were
 +      requested on the commandline, e.g. with a modifier
 +  * debian/control:
 +    - add Vcs-Browser now that loggerhead works again (Closes: #511168)
 +  * apt-pkg/contrib/fileutl.cc:
 +    - add a RealFileExists method and check that your configuration files
 +      are real files to avoid endless loops if not (Closes: #604401)
 +  * apt-pkg/contrib/weakptr.h:
 +    - include stddefs.h to fix compile error (undefined NULL) with gcc-4.6
 +  * methods/https.cc:
 +    - fix CURLOPT_SSL_VERIFYHOST by really passing 2 to it if enabled
 +  * deb/dpkgpm.cc:
 +    - fix popen/fclose mismatch reported by cppcheck. Thanks to Petter
 +      Reinholdtsen for report and patch! (Closes: #607803)
 +  * doc/apt.conf.5.xml:
 +    - fix multipl{y,e} spelling error reported by Jakub Wilk (Closes: #607636)
++  * apt-inst/contrib/extracttar.cc:
++    - let apt-utils work with encoded tar headers if uid/gid are large.
++      Thanks to Nobuhiro Hayashi for the patch! (Closes: #330162)
 +
++ -- David Kalnischkies <kalnischkies@gmail.com>  Thu, 13 Jan 2011 00:25:32 +0100
 +
  apt (0.8.10) unstable; urgency=low
  
    [ Programs translations ]

diff --cc test/integration/deb-bug-330162-encoded-tar-header.deb
index 0000000000000000000000000000000000000000,0000000000000000000000000000000000000000..f38b1aa70c66c9d87491207a3f719ad57e95aa3d
new file mode 100644 (file)
Binary files differ

diff --cc test/integration/test-bug-330162-encoded-tar-header
index 0000000000000000000000000000000000000000,0000000000000000000000000000000000000000..fa01e0379a064399a3ba83160ff63f6f22570d2f
new file mode 100755 (executable)
--- /dev/null
--- /dev/null
+++ b/test/integration/test-bug-330162-encoded-tar-header
@@@ -1,0 -1,0 +1,11 @@@
++#!/bin/sh
++set -e
++
++local TESTDIR=$(readlink -f $(dirname $0))
++. $TESTDIR/framework
++setupenvironment
++configarchitecture "i386"
++
++msgtest 'Test apt-ftparchive with encoded tar header package'
++cp $TESTDIR/deb-bug-330162-encoded-tar-header.deb aptarchive/
++test -z "$(aptftparchive packages aptarchive/ 2>&1 | grep 'E:')" && msgpass || msgfail