Merge \\"Don\\'t depend on Bouncy Castle.\\" am: 1fb96c3ff5
am: 3b36c7960d

Change-Id: Ib33cb3a8aaa94df96731a16f20a7ef5415f42a3f

Merge \"Don\'t depend on Bouncy Castle.\"
am: 1fb96c3ff5

Change-Id: I04401121eb3ad2e026268f37ffe9d19d95478325

Merge "Don't depend on Bouncy Castle."

Merge \"Pass block_list param to mksquashfs\" into nyc-dev
am: 3bddf74752

Change-Id: I1679c255b297a36bc10b50ff9d6214c92d763489

Merge \"releasetools: Disable using imgdiff for squashfs.\" into nyc-dev
am: a8d3741f1a

Change-Id: If8392b72a865f53449a1d5ca7b52de7697c4b412

Merge "Pass block_list param to mksquashfs" into nyc-dev

Merge "releasetools: Disable using imgdiff for squashfs." into nyc-dev

Merge \\"Faster and cleaner way to obtain UTF-8 encoded form.\\" am: 98b4f07dfc
am: 5e8b2c9207

Change-Id: I48bc0772fcb1acf050d04f650c1b0eca8a6c79cb

Merge \"Faster and cleaner way to obtain UTF-8 encoded form.\"
am: 98b4f07dfc

Change-Id: Ic5e071d01ea0fd9a091097e3659f16c0b71f5ae6

Merge "Faster and cleaner way to obtain UTF-8 encoded form."

Merge \\"Use more prebuilt build-tools\\" am: 6c0c720389
am: ffb643e487

Change-Id: I4cdec816470a05193c4c303f3c6343815f2b671c

Merge \"Use more prebuilt build-tools\"
am: 6c0c720389

Change-Id: Icb67388bf084f2de872e9df8747b70cbde42b5f9

Merge "Use more prebuilt build-tools"

Faster and cleaner way to obtain UTF-8 encoded form.

Instead of specifying character encoding by name, the faster, cleaner,
and safer way is to use StandardCharsets.UTF_8.

Bug: 27461702
Change-Id: I897284d3ceeb44a21cc74de09a9b25f6aec8c205

Merge \\"APK JAR signature verifier.\\" am: cf89865b95
am: d1af20e103

Change-Id: I7cae6cda81bebf10b7abf3175d428452620e83bf

Merge \"APK JAR signature verifier.\"
am: cf89865b95

Change-Id: If54865fdf6d290ab38a3f6a0f3eec2142c55e8d0

Merge "APK JAR signature verifier."

Use more prebuilt build-tools

For acp, we've been using an old prebuilt in prebuilts/sdk, but it's not
part of the SDK. Instead, we'll use a prebuilt in the build-tools
repository.

For ijar, we've been using the host libstdc++ to workaround the lack of
libc++ on some unbundled branches. Instead, use a prebuilt that can use
libc++.

For ziptime, we've been disabling it on unbundled branches, due to the
lack of libc++. Instead, use a prebuilt version of ziptime that can use
the prebuilt libc++.

Change-Id: If80f845ea06f76e3fe6765964e77c864eaf303d0

Merge \\"Add a libc++ version of ijar for prebuilt use\\" am: 06f3e8349f
am: 38ff3ae039

Change-Id: I83c7313db392a137b0d54232d4f122b54120ce2a

Merge \\"Skip uses-library check for preopted apps.\\" am: 7b4c07f1d5
am: baf8869991

Change-Id: Id14f3aca9e19d900deb31ce35292810f9d35fafe

Merge \"Add a libc++ version of ijar for prebuilt use\"
am: 06f3e8349f

Change-Id: I284c7c85de71551e608e4210ab71b17e36229e09

Merge \"Skip uses-library check for preopted apps.\"
am: 7b4c07f1d5

Change-Id: I6e027842b3eb70b333efd20ccf6798e88baef436

Merge "Add a libc++ version of ijar for prebuilt use"

Don't depend on Bouncy Castle.

This switches PKCS#7 SignedData generation code from Bouncy Castle to
OpenJDK's proprietary internal API. This is to avoid depending on a
huge library that's not really needed. In the longer term, it's best
to add our own implementation of PKCS#7 SignedData building, parsing,
and verification. This will give the code more power to mimic what the
Android platform does.

Bug: 27461702
Change-Id: I29f1ceea1293e35424fde69c0f2969d551345100

Add a libc++ version of ijar for prebuilt use

I'll remove the old version once we've moved over to the prebuilts.

Change-Id: Ifd8d396f8b653abbe958cd34e69831a6ca378649

APK JAR signature verifier.

This adds JAR signature verification to ApkVerifier.

Bug: 27461702
Change-Id: Id2b72bea7869be66268f6bc1387e1559ee02ff9d

Merge "Skip uses-library check for preopted apps."

Skip uses-library check for preopted apps.
am: b00263f96a

Change-Id: Icc9170bc494750eac12f141e7afd09d36564a2cc

Merge \\"Add option to link static lite protobuf library.\\" am: e6fae50455
am: bebdc16a68

Change-Id: I60b9fb9437fe15079c62c3d47401341b54ebc7bc

Skip uses-library check for preopted apps.

Bug: 26880306

(cherry-picked from commit b00263f96a90c9f77cf7d8d90742a0884290bf60)

Change-Id: I22beccaf61ca30cf1487a25f80e1c3fd7bdf2c62

Skip uses-library check for preopted apps.

Bug: 26880306
Change-Id: I97c75cceca30624b5f0c984a5032fdfaf632a90a

Merge \"Add option to link static lite protobuf library.\"
am: e6fae50455

Change-Id: Idcc3c54033724a9cef17a219b4c118689eaad297

Merge "Add option to link static lite protobuf library."

Merge \\"More general OutputStreamDataSink.\\" am: dca955a9bd
am: 58c8559979

Change-Id: I1caf9f4ae2650dfadcb003315c12a30a3288fd4f

Merge \"More general OutputStreamDataSink.\"
am: dca955a9bd

Change-Id: Id94e07f6007c3288c0501d6cd9017680108e4dbf

Merge "More general OutputStreamDataSink."

Merge \\"Fix inefficiency in APK entry data alignment.\\" am: a85e0aa883
am: 7fe6ba2197

Change-Id: I97934b2a4cdfabb463f5012149ef28a15768c321

Merge \"Fix inefficiency in APK entry data alignment.\"
am: a85e0aa883

Change-Id: I8c1b8116ab66c83e9b73f103adaf6237e51a6c1f

Merge "Fix inefficiency in APK entry data alignment."

More general OutputStreamDataSink.

This replaces the less general DataSink which outputs into a
ByteArrayOutputStream with a more general DataSink which outputs into
an OutputStream.

Bug: 27461702
Change-Id: I9467f38c41f586b71f35edb3602fd6e57153184f

releasetools: Disable using imgdiff for squashfs.

We use imgdiff to handle files in zip format (e.g. jar/zip/apk) for
higher compression ratio.

For system/vendor in squashfs, a) all files are compressed in LZ4
format; b) we use 4096-byte block size in their sparse images, but the
files in squashfs may not be laid out as 4K-aligned. So the blocks for
a given file as listed in block map may not form a valid zip file, which
may fail the patch generation with imgdiff.

Disable using imgdiff for squashfs images, and use bsdiff instead.

Bug: 22322817
Change-Id: Ie76aa4cece5c9d38cb1d1a34c505a4a8f37512d3

Pass block_list param to mksquashfs

Needed to generate image.map and to optimize the OTA size

BUG: 22322817
Change-Id: I7802f4dbb9116a94ea99a00b68c3a7ff180ef08a
Signed-off-by: Mohamad Ayyash <mkayyash@google.com>

Merge \\"bvb: Update path to key since repo was moved.\\" am: 8b3b1727da
am: 8ab7487660

Change-Id: I04b9b35c392b6e99b15f4c78358c70c9a9794532

Merge \"bvb: Update path to key since repo was moved.\"
am: 8b3b1727da

Change-Id: I0256523824d49aa59ba61eec18b85a9a2a9324d2

Merge "bvb: Update path to key since repo was moved."

bvb: Update path to key since repo was moved.

We recently moved Brillo verified boot from system/bvb to external/bvb
so update the path accordingly.

TEST=Build with 'BOARD_BVB_ENABLE := true' succeeds.
BUG=29099910

Change-Id: I66f3b90e23d6b2afc09f81af571ede7b573325e9

Merge \\"Do not force turn off of Jack warning\\" am: 0a26fc0ace
am: 5185323de9

Change-Id: I054396d9bdb88822ca75be61a88e18f08213242d

Merge \"Do not force turn off of Jack warning\"
am: 0a26fc0ace

Change-Id: Iddfabd648876281596bc8c571968d18589f1f581

Merge "Do not force turn off of Jack warning"

Fix inefficiency in APK entry data alignment.

26f00cda4b979d7e74db6872990682335b36612b introduced a bug where an
APK entry's extra field is padded for alignment purposes when no
padding is necessary because the entry is aligned without any padding
bytes.

Bug: 27461702
Change-Id: Icb164dbaa26d9686412e2920318a9f40c5ce9751

Merge \\"Redirect subprocess stderr to stdout in verbose mode.\\" am: 1318ec93e7
am: 74d7b31f7d

Change-Id: Ia9757f09dead577349b0cf4b97cb062f30d0f45d

Merge \"Redirect subprocess stderr to stdout in verbose mode.\"
am: 1318ec93e7

Change-Id: If411ad0cd8aba960e4ab8600a5dd1532e5bd9dca

Merge "Redirect subprocess stderr to stdout in verbose mode."

Redirect subprocess stderr to stdout in verbose mode.

This patch uses subprocess.communicate instead of subprocess.wait to
prevent deadlock if any of the child processes outputs too much data,
and redirects the subprocess output to stdout when running in verbose
mode.

With this patch `ota_from_target_files -v` prints the delta_generator
output in stdout, and no output if '-v' is not passed.

Bug: None
TEST=ota_from_target_files -v ...

Change-Id: Id66e4f3360a6f91d61a3ce96d53afbccdaa19da5

Merge \\"goldfish_setup: grant /system/bin/sh exec access\\" am: 7a866d783c
am: 0db549d72c

Change-Id: I861c80be973d94e666aa9c26c1519c93b220c094

Merge \"goldfish_setup: grant /system/bin/sh exec access\"
am: 7a866d783c

Change-Id: I340d78d01171de2eda4f25a2b4dbe0d637f35d59

Merge "goldfish_setup: grant /system/bin/sh exec access"

Add build.version.incremental to metadata
am: d06f07eef4

Change-Id: I4f0b89d69154ef32bbb04fa905fdc014c911d571

Add build.version.incremental to metadata

Add the build property "build.version.incremental" of the source (if
present) and target files to the metadata of the ota update package.

Example of metadata:
....
post-build-incremental=2951741
post-timestamp=1465345123
pre-build-incremental=2943039
pre-device=bullhead
...

Bug: 28658632
Change-Id: I889e8ccf39633b1b35590751001a42d1b05d5514

goldfish_setup: grant /system/bin/sh exec access

The goldfish_setup shell script needs the ability to execute
the shell script interpreter. Allow it.

Addresses the following denial:

avc: denied { getattr } for pid=1220 comm="init.goldfish.s"
path="/system/bin/sh" dev="vda" ino=442 scontext=u:r:goldfish_setup:s0
tcontext=u:object_r:shell_exec:s0 tclass=file permissive=0

(cherrypicked from commit 501c88c0290eb2308c110398f106411260863bb8)

Bug: 28941573
Change-Id: I22d26e90f107c8d801229354a5e0513c37e6c31d

resolve merge conflicts of 07886f1 to nyc-dev-plus-aosp

Change-Id: Iae0444b9e9e7b4db3bfd94f5712e22aa08a122b5

Merge \"Add variable PRODUCT_SHIPPING_API_LEVEL and make files with the new read only product property ro.product.first_api_level.\"
am: e5c1af2dd0

Change-Id: I0dd1cd7ebe6b7ede5f019566d844b12fe40fb40c

Merge "Add variable PRODUCT_SHIPPING_API_LEVEL and make files with the new read only product property ro.product.first_api_level."

Merge \\"Rewrite LDLIBS and SHARED_LIBRARIES\\" am: a8d13592ac
am: d8210177af

Change-Id: If466674f33ba7d98a0f8da249bc0dc57efdcf621

Merge \"Rewrite LDLIBS and SHARED_LIBRARIES\"
am: a8d13592ac

Change-Id: Ic1cdfd510ff26c279a2628b58c3a1db7ef172b5c

Merge "Rewrite LDLIBS and SHARED_LIBRARIES"

Merge \\"APK Signature Scheme v2 APK verifier.\\" am: 4f8bde47ae
am: 9cf15a318b

Change-Id: Iada08b1831d737ab702b6f7aee7cb21648a06bf0

Merge \"APK Signature Scheme v2 APK verifier.\"
am: 4f8bde47ae

Change-Id: I0a9476be7b5da7166a1d20c2b64b82fcdfae4973

Merge "APK Signature Scheme v2 APK verifier."

Merge \\"Fix DSA APK signatures for API Level 8 and lower.\\" am: ebe68ec237
am: b926969a84

Change-Id: Ie93b77c066e7637cdb704d3167dc65936aca16b1

Merge \"Fix DSA APK signatures for API Level 8 and lower.\"
am: ebe68ec237

Change-Id: I1bb11e4d47a572f5e0b9e9205910ca4dc675f671

APK Signature Scheme v2 APK verifier.

This adds the ApkVerifier class which verifies APKs using APK
Signature Scheme v2 only. In a follow-up commit this class will be
extended to verify APKs using JAR signature scheme when necessary.

The APK verifier is designed to not just verify an APK, but also
report errors, warnings, and information about signers in a structured
way, to enable tools to surface this information to users in various
ways.

Bug: 27461702
Change-Id: I10c6ba436021d86b6dbf6d3cf44494652adacb66

Merge "Fix DSA APK signatures for API Level 8 and lower."

Do not force turn off of Jack warning

When ANDROID_JACK_EXTRA_ARGS is defined there is no reason to still turn
off those warnings.

Change-Id: Ic7ac02c72ed3b7b0eaca9394c9fc7d92d5dfa871

Rewrite LDLIBS and SHARED_LIBRARIES

LOCAL_LDLIBS was the only correct way to use NDK libraries, but few used
it correctly. It also often got confused with LOCAL_LDFLAGS, so move the
flags to the correct variable.

For binaries that weren't using the NDK (empty LOCAL_SDK_VERSION), it
was never valid to use LOCAL_LDLIBS, as dependencies would not be
properly set up, and could lead to random build failures. So convert any
-l linker flags to using LOCAL_SHARED_LIBRARIES automatically.

For binaries built using the NDK (LOCAL_SDK_VERSION set), they were
required to use LOCAL_LDLIBS for prebuilt NDK libraries, otherwise they
would get headers and dependencies to the platform versions. Any
non-prebuilt LOCAL_LDLIBS would miss dependencies. So move the NDK
prebuilt libraries to LDLIBS from SHARED_LIBRARIES, and move everything
else to SHARED_LIBRARIES.

So now, for device modules, LOCAL_SHARED_LIBRARIES should always be
used, and we'll do the right thing. LOCAL_LDLIBS should only be used for
host libraries from the system.

Change-Id: Ide34c7afdcfb6507a378d45a42471729e489a9e0

Add option to link static lite protobuf library.

Bug: 28114205

Change-Id: Id2b7e42017a53138c60b92de8d196ca92bc38053

Merge \"Removing N from PLATFORM_VERSION_ALL_CODENAMES\" into nyc-dev
am: ddc0c5e0a0

Change-Id: I1f68aa77d52fb85a3d87a3155b9eefb1ff647c31

Merge "Removing N from PLATFORM_VERSION_ALL_CODENAMES" into nyc-dev

Merge "Add variable PRODUCT_SHIPPING_API_LEVEL." into nyc-dev
am: 837b86aa56

* commit '837b86aa5696fea28a0acfde7361a3c5d5824624':
  Add variable PRODUCT_SHIPPING_API_LEVEL.

Change-Id: I692a44938baa8e462041786b1f394dc5b41ac8e0

Merge "Add variable PRODUCT_SHIPPING_API_LEVEL." into nyc-dev

Merge "Add NATIVE_TESTS class, move host native tests" am: e72fc63901
am: 10461b78f0

* commit '10461b78f0955d9dba8d32045d1e35ff0f9a5c8c':
  Add NATIVE_TESTS class, move host native tests

Change-Id: Ia11b54676d3b33e58c177a5a375f29233657524e

Merge "Add NATIVE_TESTS class, move host native tests"
am: e72fc63901

* commit 'e72fc639014809b4d4239d7f7754c6950f1ad546':
  Add NATIVE_TESTS class, move host native tests

Change-Id: Ifb0401242afe1e6e16666001faf3d55471200c3f

Merge "Add NATIVE_TESTS class, move host native tests"

Merge "Switch some build tools to Soong" am: c52d09e694
am: 8a78c90a95

* commit '8a78c90a95a3a0c5886b333e90994721eeb92734':
  Switch some build tools to Soong

Change-Id: I481784785375dfb6790d14681eb110e0f67bf65c

Merge "Switch some build tools to Soong"
am: c52d09e694

* commit 'c52d09e6944e531e02242cf971020cb2c0cf9844':
  Switch some build tools to Soong

Change-Id: I4ba945cad1eda7c696cd770243e226a09ff735ac

Merge "Switch some build tools to Soong"

Fix DSA APK signatures for API Level 8 and lower.

This modifies JAR signing code to produce DSA signatures which are
accepted by all Android platforms rather than only API Level 9 and
higher.

The issue is that by default Bouncy Castle uses OID 1.2.840.10040.4.3
(dsaWithSha1) in PKCS #7 CMS SignerInfo whereas Android accepts that
only since API Level 9. However, OID 1.2.840.10040.4.1 (dsa) is
accepted by all Android platforms.

Bug: 27461702
Change-Id: I24256a255bcdc2108bdb447557af7568a2c096e3

Switch some build tools to Soong

In preparation for including them in prebuilts/build-tools.

acp: We use a prebuilt in prebuilts/sdk, but it's not part of the sdk.

ijar: We use the host libstdc++ to workaround the lack of libc++ on
some unbundled branches.

ziptime: We disable this on unbundled builds, due to the lack of libc++.

Change-Id: Ib9766b1dbddd151c38ff27c529865200ab37fce1

Merge "Remove dead rgb2565" am: a571e83095
am: d1f0eb15c8

* commit 'd1f0eb15c812b8c4ea8697b9e2171846e6510884':
  Remove dead rgb2565

Change-Id: I6a95c5cf31a05822d85fce16d1e9b311dd759cee

Merge "Remove dead rgb2565"
am: a571e83095

* commit 'a571e83095a72d595e5e4e365e0ae52aefd64356':
  Remove dead rgb2565

Change-Id: I6dae3bbd804591a0e91f3467f021c5ee2dde4bf7

Merge "Remove dead rgb2565"

Merge "Drop CLANG and CLANG_CXX in the environment" am: 27a55cd3c7
am: f03398b651

* commit 'f03398b6519e1d9695c61da450346d5a0f0f8528':
  Drop CLANG and CLANG_CXX in the environment

Change-Id: I9fce3d3c203b89295529c72de4d094ff62d178fd

Merge "Drop CLANG and CLANG_CXX in the environment"
am: 27a55cd3c7

* commit '27a55cd3c703c190f933630e503d963ef7478183':
  Drop CLANG and CLANG_CXX in the environment

Change-Id: I3850b1783fb47cda05f1a7799ff8ceddb589e033

Merge "Improve efficiency of using the DataSource abstraction." am: 7353eabd14
am: 732b29a5c3

* commit '732b29a5c38a3a33bdffea662875e6a8dab894b3':
  Improve efficiency of using the DataSource abstraction.

Change-Id: I4b2e05ffc80d59ef19a1b78c3560f0d21c5b04b2

Merge "Drop CLANG and CLANG_CXX in the environment"

Merge "Improve efficiency of using the DataSource abstraction."
am: 7353eabd14

* commit '7353eabd14cbfae2c123a58cae4ce1e770ee351d':
  Improve efficiency of using the DataSource abstraction.

Change-Id: Ia1ab6c7d0b94c59ab6d7b41828e8e5d676c212e5

Merge "Improve efficiency of using the DataSource abstraction."

Remove dead rgb2565

Change-Id: I2397dc97768f24a7c416bff96dd983fc796cbb53

Drop CLANG and CLANG_CXX in the environment

We'll explicitly pass these variables to the static analyzer builds in
binary.mk

Bug: 29071982
Change-Id: Ifc7633d36f5785fe16b63f54b885396c2cab542a

Merge "Switch signapk to apksigner-core." am: 1a39e8eb34
am: 2f24b771ea

* commit '2f24b771eac14924ba9d5ab1a0e69da1641df582':
  Switch signapk to apksigner-core.

Change-Id: I3b8ddfcf2369247713b1570cb901cc5604622c75