Add new limits for extra field, updated test file.

[email protected] (cherry picked from commit 8c0020b) Bug: 775160 Change-Id: Ic0f2a818386c78cf7b610035f7841f08e5477ab9 Reviewed-on: https://chromium-review.googlesource.com/723783 Reviewed-by: Peter Kotwicz <[email protected]> Commit-Queue: Scott Kirkwood <[email protected]> Cr-Original-Commit-Position: refs/heads/master@{#509491} Reviewed-on: https://chromium-review.googlesource.com/725526 Reviewed-by: agrieve <[email protected]> Cr-Commit-Position: refs/branch-heads/3239@{#52} Cr-Branched-From: adb61db-refs/heads/master@{#508578}
nwjs · Oct 18, 2017 · f9a91c4 · f9a91c4
1 parent a80e60e
commit f9a91c4
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/.../android/webapk/libs/client/src/org/chromium/webapk/lib/client/WebApkVerifySignature.java b/.../android/webapk/libs/client/src/org/chromium/webapk/lib/client/WebApkVerifySignature.java
@@ -83,8 +83,9 @@ public class WebApkVerifySignature {
     /** Maximum file comment length permitted. */
     private static final int MAX_FILE_COMMENT_LENGTH = 0;
 
-    /** Maximum extra field length permitted. */
-    private static final int MAX_EXTRA_LENGTH = 8;
+    /** Maximum extra field length permitted.
+     * Support .so alignment and a 64 bytes bytes for any extras. */
+    private static final int MAX_EXTRA_LENGTH = 4096 + 64;
 
     /** The memory buffer we are going to read the zip from. */
     private final ByteBuffer mBuffer;

diff --git a/chrome/test/data/webapks/extra-field-too-large.apk b/chrome/test/data/webapks/extra-field-too-large.apk