ACPR/android_system_core
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

am b242ad40: Merge "Revert "Make sure that names of all entries have the same encoding.""

Browse source 
* commit 'b242ad40b1e0d3984e469e3d35876a4859f4952b':
  Revert "Make sure that names of all entries have the same encoding."
This commit is contained in:
Piotr Jastrzebski 2014-08-13 09:58:33 +00:00 • committed by Android Git Automerger
commit fa41212789
2 changed files with 1 additions and 25 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
include/ziparchive/zip_archive.h
View file

@ -157,11 +157,6 @@ int32_t Next(void* cookie, ZipEntry* data, ZipEntryName *name);
*/ */
void EndIteration(void* cookie); void EndIteration(void* cookie);
/*
* Whether entry names in an archive are encoded in UTF-8.
*/
bool HasUTF8Names(const ZipArchiveHandle handle);
/* /*
* Uncompress and write an entry to an open file identified by |fd|. * Uncompress and write an entry to an open file identified by |fd|.
* |entry->uncompressed_length| bytes will be written to the file at * |entry->uncompressed_length| bytes will be written to the file at

21
libziparchive/zip_archive.cc
View file

@ -192,11 +192,8 @@ struct DataDescriptor {
#undef DISALLOW_IMPLICIT_CONSTRUCTORS #undef DISALLOW_IMPLICIT_CONSTRUCTORS
// mask value that signifies that the entry has a DD static const uint32_t kGPBDDFlagMask = 0x0008; // mask value that signifies that the entry has a DD
static const uint32_t kGPBDDFlagMask = 0x0008;
static const uint32_t kMaxErrorLen = 1024; static const uint32_t kMaxErrorLen = 1024;
// mask value that signifies that the entry names are encoded in UTF-8
static const uint32_t kGPBEFSFlagMask = 0x0800;
// The maximum size of a central directory or a file // The maximum size of a central directory or a file
// comment in bytes. // comment in bytes.
@ -298,7 +295,6 @@ struct ZipArchive {
/* number of entries in the Zip archive */ /* number of entries in the Zip archive */
uint16_t num_entries; uint16_t num_entries;
bool utf8_names_encoding;
/* /*
* We know how many entries are in the Zip archive, so we can have a * We know how many entries are in the Zip archive, so we can have a
@ -314,7 +310,6 @@ struct ZipArchive {
directory_offset(0), directory_offset(0),
directory_map(NULL), directory_map(NULL),
num_entries(0), num_entries(0),
utf8_names_encoding(false),
hash_table_size(0), hash_table_size(0),
hash_table(NULL) {} hash_table(NULL) {}
@ -660,15 +655,6 @@ static int32_t ParseZipArchive(ZipArchive* archive) {
ptr - cd_ptr, cd_length, i); ptr - cd_ptr, cd_length, i);
goto bail; goto bail;
} }
if (i == 0) {
archive->utf8_names_encoding = cdr->gpb_flags & kGPBEFSFlagMask;
} else {
bool has_utf8_name_encoding = cdr->gpb_flags & kGPBEFSFlagMask;
if (archive->utf8_names_encoding != has_utf8_name_encoding) {
ALOGW("Zip: Entry names encoded with different encoding");
goto bail;
}
}
} }
ALOGV("+++ zip good scan %" PRIu16 " entries", num_entries); ALOGV("+++ zip good scan %" PRIu16 " entries", num_entries);
@ -990,11 +976,6 @@ int32_t Next(void* cookie, ZipEntry* data, ZipEntryName* name) {
return kIterationEnd; return kIterationEnd;
} }
bool HasUTF8Names(const ZipArchiveHandle handle) {
const ZipArchive* archive = reinterpret_cast<ZipArchive*>(handle);
return archive->utf8_names_encoding;
}
static int32_t InflateToFile(int fd, const ZipEntry* entry, static int32_t InflateToFile(int fd, const ZipEntry* entry,
uint8_t* begin, uint32_t length, uint8_t* begin, uint32_t length,
uint64_t* crc_out) { uint64_t* crc_out) {