am a3f8c2aa: Merge "Fix a memory corruption issue when vector resize"

* commit 'a3f8c2aaf1bb82b436306f65b86d25d4ea955d4d':
  Fix a memory corruption issue when vector resize

Merge "Fix a memory corruption issue when vector resize"

am dfc30ae3: Merge "Command to move private app data between volumes."

* commit 'dfc30ae393fd610fcbd05a1ecfe7d4834a792de3':
  Command to move private app data between volumes.

Merge "Command to move private app data between volumes."

Command to move private app data between volumes.

New "mvuserdata" command will move all private app data from one
volume UUID to another.  It leverages the existing "cp" toybox
command to do the heavy lifting for all known users, preserving
details like timestamps and permissions.  It invokes restorecon()
to correctly label the new location when the copy is finished.

Changes installd to no longer drop capabilities, so we run as root
again.  This also allows us to exec "cp" with CAP_DAC_OVERRIDE and
CAP_FOWNER still in effect.

Bug: 19993667
Change-Id: I1f407a7c4a1af97ca5afc27b04eb16b4936cbdef

Fix a memory corruption issue when vector resize

There is memory corruption in below code

const Rect* prev = &dst[prevIndex];
dst.add(Rect(prev->right, top, right, bottom));

prev points to a memory of vector dst, when dst resize in add()
call, the memory that prev points to will be copy to the new
allocated vector memory and the old memory will become undefined

Avoid pointer in this case, use a local copy instead

Change-Id: I4d95ceedd00c8fb615ac153082ade1b1ce0d0fa8

am 44a38d93: Merge "Start accepting volume UUIDs from framework."

* commit '44a38d9337989742046c1e3faa6e7392ecc47cd4':
  Start accepting volume UUIDs from framework.

Merge "Start accepting volume UUIDs from framework."

Start accepting volume UUIDs from framework.

We're now parsing and passing through volume UUIDs sent across the
command socket.  The "!" argument value is treated as null, which
means internal storage.

Bug: 19993667
Change-Id: I17729a769ce687a2e94e85991a6338c77ded0b66

am 066fadb9: Merge "More volume UUID awareness."

* commit '066fadb983b117a7682af00780d23d16232ab0e2':
  More volume UUID awareness.

Merge "More volume UUID awareness."

More volume UUID awareness.

Teach free_cache() and restorecon_data() about building per-volume
paths.  Also clean up restorecon_data() by using std::string when
building paths.

Clearer names for path building utility methods, and tests to verify.

Bug: 19993667
Change-Id: Iacfbcdaa5b901cc2490bc8eba366dfdeb44f1d93

am f9ab2ad2: Merge "Installd: Fix Clang build"

* commit 'f9ab2ad2d5786b2dd04f08f5d5a67102bd3d796a':
  Installd: Fix Clang build

Merge "Installd: Fix Clang build"

Installd: Fix Clang build

Remove some unused parameters and variables.

Change-Id: I507ae48fe1744b800ceb29192d463fd26c2ccda7

am d107c305: Merge "Plumb through volume UUID when building paths."

* commit 'd107c305f74592e8078754b317d840a2dde661db':
  Plumb through volume UUID when building paths.

Merge "Plumb through volume UUID when building paths."

am ab91d03b: Merge "installd: Add support for SELF_PATCHOAT_NEEDED."

* commit 'ab91d03b0dfebf40e98034670892c278ae0a46e5':
  installd: Add support for SELF_PATCHOAT_NEEDED.

Merge "installd: Add support for SELF_PATCHOAT_NEEDED."

am ad436ab5: Merge "dumpstate: scrub APANIC"

* commit 'ad436ab54a649075b25ab755344fd011bd0c9262':
  dumpstate: scrub APANIC

Merge "dumpstate: scrub APANIC"

dumpstate: scrub APANIC

Bug: 19525200
Change-Id: Ia2d2857cac953fb4ceafb66ddf775d89d7a777b1

Plumb through volume UUID when building paths.

Since app data paths can live on expanded storage devices, accept the
target volume UUID when building paths.  The null UUID indicates the
default internal storage.

To improve readability, start using std::string in several places,
which throws when allocations fail.  For now, perform last-second
sanity checks on incoming path arguments, but we'll eventually want
to check arguments as they come through installd.cpp, instead of
crashing the entire daemon.

Also remove "lib" symlink code from install() and make_user_data(),
since we're no longer supporting /data/app-lib.  The framework
already uses linklib() to create the right symlink for the selected
ISA-specific library dir.

Bug: 19993667
Change-Id: Ib9343575ffb62bf3981e19375de8f3822fc31e28

installd: Add support for SELF_PATCHOAT_NEEDED.

Change-Id: Ib9a6373f98474f1242367b5285086251a9d580e5

am 00afb817: Merge "libgui: Plumb detachBuffer through ConsumerBase"

* commit '00afb8177fdbf144c75c00e9eacc915589807d35':
  libgui: Plumb detachBuffer through ConsumerBase

Merge "libgui: Plumb detachBuffer through ConsumerBase"

libgui: Plumb detachBuffer through ConsumerBase

Exposes IGraphicBufferConsumer::detachBuffer as a ConsumerBase
method. attachBuffer is not currently exposed, because all current
clients will be recycling buffers through the allocator.

Bug: 19628705
Change-Id: I3e519767fa43d5d880c1d5695e31b60f6ad588af

Merge "libgui: Plumb attach/detach through Surface" into stage-aosp-master

Merge "libgui: Remove IGBC::BufferItem" into stage-aosp-master

am 1cd030be: Merge "Switch installd to compile as C++."

* commit '1cd030be59c108cde25234f0d2da83a68a7cdd26':
  Switch installd to compile as C++.

am 2d42dc74: Merge "Valid APK paths now include expanded storage."

* commit '2d42dc74e9b647345a5d4ba79bcdadbf700002e2':
  Valid APK paths now include expanded storage.

Merge "Switch installd to compile as C++."

Merge "Valid APK paths now include expanded storage."

Switch installd to compile as C++.

This is the minimal change needed to switch it over to C++, which
paves the way for using more robust utilities like std::string.

Change-Id: I80ed6280146875eb6ddbbb340c05450388ca13f0

Valid APK paths now include expanded storage.

Apps on expanded storage live at /mnt/expand/<uuid>/app/com.example,
so we need to relax one more directory level.

Bug: 19993667
Change-Id: I347ec7b92435ea69e632ed5d5fdfabe38ce0b56e

libgui: Plumb attach/detach through Surface

Exposes the attachBuffer and detachNextBuffer calls from
IGraphicBufferProducer to the public Surface interface. Also moves
the version of connect that takes a producer callback from protected
to public.

Bug: 19628705
Change-Id: I9ebc3013c4d9c84c4e8ef150c00e03f8af80319e
(cherry picked from commit 231832eb27107fb561467f5f4a9be2c577c61ea8)

libgui: Remove IGBC::BufferItem

Removes IGraphicBufferConsumer::BufferItem. Depends on the
following changes:
    I187b3a7d05196b6289596afac8fb9a9d4aebff76
    I0ddd38df37500cfd6b21d1e768ed14e39c5cd9fc

Change-Id: I3edf0db8fba656fd78e18a5a7f1137f0fb6b237d
(cherry picked from commit 1c87e474d87d6d1380fb61d476d606b1a2fda1c1)

am 8f43f77d: Merge "Use AID_SYSTEM as uid when creating oat dir"

* commit '8f43f77de89df935e7dc2fdeddee132580bb5705':
  Use AID_SYSTEM as uid when creating oat dir

Merge "Use AID_SYSTEM as uid when creating oat dir"

Use AID_SYSTEM as uid when creating oat dir

Previously AID_INSTALL was used, which was causing permission denied errors
when PackageManager was trying to recursively rename staging directory

Bug: 19550105
Bug: 20087446
Change-Id: I3a9e3056c1fbc1ce0077a3ce52cf77ea6b5085ee

am a60ff367: Merge "Remove useless \'svcmgr_handle\' (which had wrong type)."

* commit 'a60ff367a4f55ea319625cf500687c5025604f40':
  Remove useless 'svcmgr_handle' (which had wrong type).

Merge "Remove useless 'svcmgr_handle' (which had wrong type)."

DO NOT MERGE Cherry-pick dataSpace and BufferItem changes

DO NOT MERGE Add dataSpace to buffer queues; remove old format enums.
(cherry picked from commit 82c6bcc9705eabcaf5b9e45bc81867b0e2d61a02)

DO NOT MERGE libgui: Prepare for IGBC::BufferItem removal
(cherry picked from commit cf3834db104e0b052056e3a06d46e3f222f0d372)

DO NOT MERGE SurfaceFlinger: Stop using IGBC::BufferItem
(cherry picked from commit 11611f9be590480d7ea27bf0153558573ddcded2)

Remove useless 'svcmgr_handle' (which had wrong type).

Also use the 'ptr' union member rather than 'handle'.

Signed-off-by: Serban Constantinescu <serban.constantinescu@arm.com>
Change-Id: I68e5336cd3af3bc61dbddd2b33d7e1512c0c329c

am 031fec0b: Merge "ServiceManager: Print kernel/userspace binder version"

* commit '031fec0b9da7f5985c5a1650ed570ca3872e63c0':
  ServiceManager: Print kernel/userspace binder version

Merge "ServiceManager: Print kernel/userspace binder version"

ServiceManager: Print kernel/userspace binder version

This patch adds extra logging for the binder version in case the userspace
and kernel versions differ.

Change-Id: I9859b29099726cfcfe5ca23d88ed2e101a06af1a
Signed-off-by: Serban Constantinescu <serban.constantinescu@arm.com>

am 1e954441: (-s ours) Merge "DO NOT MERGE libgui: Plumb attach/detach through Surface"

* commit '1e954441d22d0feb28caceee9fe14bc7729b7b90':
  DO NOT MERGE libgui: Plumb attach/detach through Surface

Merge "DO NOT MERGE libgui: Plumb attach/detach through Surface"

am 9cad1985: Merge "Fix errors caused by unused variables"

* commit '9cad1985880f700be3150f05ca3539444da850d2':
  Fix errors caused by unused variables

Merge "Fix errors caused by unused variables"

Fix errors caused by unused variables

Change-Id: Ie52ae0e9a642504ee7b78c6bc54b61549cfb0342

am 4eb88f88: Merge "Support for storing OAT files in app directory"

* commit '4eb88f883911eb1208e85a9c176481176242af13':
  Support for storing OAT files in app directory

Merge "Support for storing OAT files in app directory"

DO NOT MERGE libgui: Plumb attach/detach through Surface

Exposes the attachBuffer and detachNextBuffer calls from
IGraphicBufferProducer to the public Surface interface. Also moves
the version of connect that takes a producer callback from protected
to public.

Bug: 19628705
Change-Id: I9ebc3013c4d9c84c4e8ef150c00e03f8af80319e
(cherry picked from commit c14ecb9de243af8864610fd3c74342e3ca2cb4bc)

am 23b204e1: Merge "servicemanager: service_manager missing include for string.h"

* commit '23b204e1ee9069b9321b5547e8d17a5014efbacd':
  servicemanager: service_manager missing include for string.h

Merge "servicemanager: service_manager missing include for string.h"

am ef62116d: (-s ours) Merge "DO NOT MERGE libgui: Remove IGBC::BufferItem"

* commit 'ef62116d802237ebac77465d6ff6779ecfe038ae':
  DO NOT MERGE libgui: Remove IGBC::BufferItem

Merge "DO NOT MERGE libgui: Remove IGBC::BufferItem"

servicemanager: service_manager missing include for string.h

service_manager.c gets string.h inherited from
private/android_filesystem_config.h it should
not rely on this in the future. The intent is
to move fs_config function into libcutils and
thus deprecate any need for string.h in this
include file.

Bug: 19908228
Change-Id: Icc95ee02bf02c596463868b1330d209d1bd5c58a

am 5881c8dd: Merge "Installd: Support dex2oat threads system property"

* commit '5881c8dd3b5b290cddca0e0f01e2c2f1641826b0':
  Installd: Support dex2oat threads system property

Merge "Installd: Support dex2oat threads system property"

Installd: Support dex2oat threads system property

Check dalvik.vm.dex2oat-threads in installd and pass to dex2oat.

Bug: 19992386
Change-Id: I5e7806cf560607d31a1d6901dffb14bee538c9cc

Support for storing OAT files in app directory

Changes to installd:
- dexopt now allows oat_dir param for custom output directory
- Added helper method calculate_oat_file_path for calculating oat file
  output location based on oat_dir and apk_path.

Bug: 19550105
Change-Id: I6b079207310583adeb6dad918a58034a059e34c4

am 0faebfb2: Merge "Add trace_event_clock_sync to atrace"

* commit '0faebfb2f5a07939e7264a94fdcebecc4df39873':
  Add trace_event_clock_sync to atrace

Merge "Add trace_event_clock_sync to atrace"

Add trace_event_clock_sync to atrace

Change-Id: I8adce15210d82df09ff3a39ddd8df617aeb16a42

am bc5aceb1: Merge "Store odex files in oat/<isa>/ directory."

* commit 'bc5aceb1f0a68e96dced1d843278b3bf14504f2c':
  Store odex files in oat/<isa>/ directory.

Merge "Store odex files in oat/<isa>/ directory."

Store odex files in oat/<isa>/ directory.

Previously odex files were stored alongside the dex location as:
  dex location: /foo/bar/base.apk
  odex location: /foo/bar/<isa>/base.odex

This changes where odex files are stored, adding an "oat" directory:
  dex location: /foo/bar/base.apk
  odex location: /foo/bar/oat/<isa>/base.odex

See also the corresponding changes in platform/art and platform/build.

Bug: 19550105
Change-Id: I4c6be4f0c41ff175904846db8e360c4af815b265

am a527c07b: Merge "Fix mistake using width instead of height."

* commit 'a527c07b8d078f7da1114b7055f65b479a148729':
  Fix mistake using width instead of height.

Merge "Fix mistake using width instead of height."

Fix mistake using width instead of height.

Found by rodrigo.chiossi@intel.com.

Change-Id: I49d590ed39f6b47ba58a77a75d66b2964ea1cb1d

am 6dad1713: Merge "Use verify-at-runtime if debug.usejit is true"

* commit '6dad1713040a46ec6e2ca1b0df2cbe4240e9e2e9':
  Use verify-at-runtime if debug.usejit is true

Merge "Use verify-at-runtime if debug.usejit is true"

Use verify-at-runtime if debug.usejit is true

Bug: 19735273
Change-Id: Ib65b50dbde00184544a1341f351a0fe0ef504d32

DO NOT MERGE libgui: Remove IGBC::BufferItem

Removes IGraphicBufferConsumer::BufferItem. Depends on the
following changes:
    I187b3a7d05196b6289596afac8fb9a9d4aebff76
    I0ddd38df37500cfd6b21d1e768ed14e39c5cd9fc

Cherry-pick of Id1fa56d092188f2cb712768d5d2fc6a9027fb73c

Change-Id: I3edf0db8fba656fd78e18a5a7f1137f0fb6b237d

am 701473cf: (-s ours) Merge "DO NOT MERGE SurfaceFlinger: Stop using IGBC::BufferItem"

* commit '701473cf9bb1d1da2a0f55d9732d3016e1f33d59':
  DO NOT MERGE SurfaceFlinger: Stop using IGBC::BufferItem

Merge "DO NOT MERGE SurfaceFlinger: Stop using IGBC::BufferItem"

am 7d89f5b6: (-s ours) Merge "DO NOT MERGE libgui: Prepare for IGBC::BufferItem removal"

* commit '7d89f5b6562da8bb316f04f91dd7cc9fce33d7fd':
  DO NOT MERGE libgui: Prepare for IGBC::BufferItem removal

Merge "DO NOT MERGE libgui: Prepare for IGBC::BufferItem removal"

am f44d13e7: (-s ours) Merge "DO NOT MERGE Add dataSpace to buffer queues; remove old format enums."

* commit 'f44d13e7e574f181750fa37f439ab50cf05ab3df':
  DO NOT MERGE Add dataSpace to buffer queues; remove old format enums.

Merge "DO NOT MERGE Add dataSpace to buffer queues; remove old format enums."

DO NOT MERGE SurfaceFlinger: Stop using IGBC::BufferItem

Switches all uses of IGraphicBufferConsumer::BufferItem (and
BufferQueue::BufferItem) to the BufferItem in libgui. Depends on
frameworks/native I699ed0a6837076867ca756b28d1ffb2238f7a0d9.

Cherry pick of I187b3a7d05196b6289596afac8fb9a9d4aebff76

Change-Id: I5bc79fb96b6cba6021af64b20890967aa3b7fcbf

DO NOT MERGE libgui: Prepare for IGBC::BufferItem removal

Currently, there are two instances of BufferItem: one inside of
IGraphicBufferConsumer, and a standalone one inside of libgui. They
only differ in the name of one of the fields, and we want to remove
the IGBC version. This changes things so that client code may be
incrementally switched over to the libgui version.

This is a squashed commit containing the following changes:
    I64f495105f56cbf5803cea4aa6b072ea29b70cf5
    I1394e693314429ada93427889f10b7b01c948053
    I9c3bc8037fa9438d4d9080b8afb694219ef2f71f
    I699ed0a6837076867ca756b28d1ffb2238f7a0d9
    Iac8425e1241774304a131da2fb9dec6e82922f13

Change-Id: Ic4d51f5df6dbc70b376d13fceba2335b9bae4f3d

DO NOT MERGE Add dataSpace to buffer queues; remove old format enums.

- Wire up new dataSpace parameter through buffer queue stack
- Update tests to include the parameter
- Switch eglApi to using dataSpace to indicate sRGB gamma/linear
  difference
- Remove RAW_SENSOR in favor of RAW16
- Remove use of sRGB format enums
- Add default dataspace to buffer queue core
- Add query for default dataspace

Cherry pick of I070bd2e7c56506055c419004c29e2e3feac725df

Change-Id: I461952389c18051176c6b75e664f20ad369f5760

Merge "dumpstate: add O_CLOEXEC"

dumpstate: add O_CLOEXEC

Add O_CLOEXEC to various file descriptor calls, to avoid
leaking file descriptors to dumpstate's child processes.

Bug: 18342188
Change-Id: I74c47a98dfddc29c618067ad53d879b98ed1d87a

Merge "Surface: Permit an IProducerListener on connect"

Merge "libgui: Only allow one return from useFenceSync"

libgui: Only allow one return from useFenceSync

An #ifdef was causing multiple returns in this function, which was
triggering an unreachable code error on some build targets. This
adds an #else to ensure that only one return is exposed in any given
build.

Cherry pick of I9ed7c3e769bc9094ad31d810944227d1d8302886

Change-Id: Ibddc5706159d168031d1805fcce3eedc068296cf

Merge "GLConsumer: Fix unsigned subtraction during crop"

Merge "Update maxNumber to be smaller."

Merge "libgui: Enable -Weverything and -Werror"

Surface: Permit an IProducerListener on connect

This change allows clients of Surface to provide an IProducerListener
callback object to Surface::connect, which will be passed down to the
underlying IGraphicBufferProducer.

Cherry pick of I5ea5229bf3a329bf02c6bd20e7247039c75d136b

Change-Id: I6f8f52c72654e4cee649721383819bafe378f964

GLConsumer: Fix unsigned subtraction during crop

Since some variables had been switched from signed to unsigned, there
was a section of code that was guaranteed to be incorrect because it
effectively did 'if (a < b) { c = a - b; }'. This change fixes it.

Cherry pick of I9cdd6c9a0179801addebb5d6dc1fbaddf8f53c62

Bug: 19346631
Change-Id: Id13a46f74c9ae7278463ce22b586f4dc21b5e453

Update maxNumber to be smaller.

There shouldn't be more than 4096 fds (probably signficantly smaller) and
there shouldn't be more than 4096 ints.

Cherry pick of I3a3e50ee3078a4710e9737114e65afc923ed0573

Bug: 18076253
Change-Id: I82a883572b401f115d252dcd3d00aa7252b49b0e

libgui: Enable -Weverything and -Werror

Enables -Weverything and -Werror, with just a few exceptions for
warnings we can't (or shouldn't need to) work around.

Cherry pick of I034abec27bf4020d84af60d7acc1939c59986dd6 plus a
couple of minor changes to CpuConsumer.cpp to make it work with a
prior change:
    Uncomment CC_LOGV on line 46
    Change C-style cast to static_cast on line 71

Change-Id: Iaec610477ea0122317b0578fb74caf2383d4cf08

Merge "[MIPS] Fix mips32r1 build"

am 5a25a63d: Merge "Installd: Pass debuggable flag"

* commit '5a25a63de2693177b15817fb19b684ca7bfec582':
  Installd: Pass debuggable flag