Merge "libprocessgroup: remove legacy C string handling and build for host" am: 304dacae8a
am: 1748e6f9a0

Change-Id: I5eb14b25bca784abcf0aa405ada55b79c554f8bd

Merge "libprocessgroup: remove legacy C string handling and build for host"
am: 304dacae8a

Change-Id: Ib977b81965bbf6f9186606a267ef96f9e140e15c

Merge "libprocessgroup: remove legacy C string handling and build for host"

Merge "Mark crash_dump.policy as not supported on mips" am: c0d4786989
am: 3cfc767032

Change-Id: I84a405ce5522ab54b390866deff73441ef29d006

Merge "Mark crash_dump.policy as not supported on mips"
am: c0d4786989

Change-Id: I93451539cbfc92e50cbef697983d5d38eb3923bf

Merge "Mark crash_dump.policy as not supported on mips"

Merge changes I0783be05,Id8178913 am: 09d5e258ef
am: 2c1b3e9761

Change-Id: I764488803977604f85000cbf12ecf25f2c60c695

Merge changes I0783be05,Id8178913
am: 09d5e258ef

Change-Id: I611aab78988a959d3660c09fd5854e4f657e40b4

Merge changes I0783be05,Id8178913

* changes:
  adb: allow reentrant calls to fdevent_run_on_main_thread.
  adb: stop using quick_exit.

Mark crash_dump.policy as not supported on mips

The recent change to detect missing source files broke reading makefiles
for mips, since this didn't specify a source file.

Bug: 73904572
Test: lunch aosp_mips-eng; m nothing
Test: lunch aosp_arm-eng; m crash_dump.policy
Test: lunch aosp_arm64-eng; m crash_dump.policy
Test: lunch aosp_x86-eng; m crash_dump.policy
Test: lunch aosp_x86_64-eng; m crash_dump.policy
Change-Id: I28864b5af59267f1ab83084128f2c59b04039374

adb: allow reentrant calls to fdevent_run_on_main_thread.

Previously, reentrant calls to fdevent_run_on_main_thread would
deadlock.

Test: adb_test on host
Change-Id: I0783be0558dcaf61ddbe76d13ac6917fc2de0be0

adb: stop using quick_exit.

We don't actually need to use quick_exit to avoid calling static
destructors, since we have -Wexit-time-destructors to guarantee we
don't actually have any, and this precludes the use of asan's exit time
leak checking, so switch back to atexit/exit.

Test: ASAN_OPTIONS=detect_leaks=1:leak_check_at_exit=1 adb server nodaemon with a manually inserted leak
Change-Id: Id8178913f64cb02c820c5073351369a9e4d8c74d

libprocessgroup: remove legacy C string handling and build for host

Bug: 36970783
Test: kill zygote and see that its children die correctly
Change-Id: I1b0e9a9cdb834a000fec82323d690e3bd288692d

Merge "EndsWith *suffix*" am: 73664df55e
am: c4a42dd62d

Change-Id: Iad3f81efc988383ff99d9b1a08bfaa5d80693889

Merge "EndsWith *suffix*"
am: 73664df55e

Change-Id: I7c2a04e4620ecbf1b1e539e5b6e31c1802833d51

Merge "EndsWith *suffix*"

EndsWith *suffix*

Possibly an old copy and paste error.

Test: built
Change-Id: Ie86203d65b4021ed4b787d2330729ab0b1d73e9d

Merge "fs_mgr: add sysfs_path=%s to indicate a location of sysfs entry" am: badaa56639
am: 81e692c04b

Change-Id: I0e31e02f552cc518646e3a92bab6d1e824d3dce2

Merge "fs_mgr: add sysfs_path=%s to indicate a location of sysfs entry"
am: badaa56639

Change-Id: Ia26e3e4672eb5a4c3c243c7a56f505fe61f8e9c1

Merge "fs_mgr: add sysfs_path=%s to indicate a location of sysfs entry"

fs_mgr: add sysfs_path=%s to indicate a location of sysfs entry

This patch adds sysfs_path=%s in fstab to get the sysfs entry path.
One example is that vold is able to conduct UFS-specific operations given the
below fstab entry.

/dev/block/bootdevice/by-name/userdata  /data f2fs  noatime,nosuid,nodev,discard
  latemount,wait,check,fileencryption=ice,quota,formattable,sysfs_path=/sys/devices/platform/soc/1d84000.ufshc

Bug: 63264275
Bug: 68721792
Change-Id: I5097873893948f12296ac5eaa0c2a59a6e6dd9e1
Signed-off-by: Jaegeuk Kim <jaegeuk@google.com>

Merge "init: Reboot after timeout passes during reboot" am: 89063827b1
am: d59fed22c5

Change-Id: Ibfd78b0381d3f4cccaae90af1334d21d2fe594d2

Merge "init: Reboot after timeout passes during reboot"
am: 89063827b1

Change-Id: I8db8b48e25b086c341787fbde2d4a2c7d922001d

Merge "init: Reboot after timeout passes during reboot"

Merge "libutils: remove subdirs" am: 876d9c3159
am: f664fbd336

Change-Id: I9d567371c77d436acec1e7335f8d6ffaf01a2e3f

Merge "adb: switch apacket over to a std::string payload." am: 581a4ceb00
am: 4d74066425

Change-Id: I15d0066c612b16b2874859c6ceaf27fcf9f2d878

Merge "libutils: remove subdirs"
am: 876d9c3159

Change-Id: Ic160753c87d7172f64c0498b5a0b9f4616c5b86c

Merge "adb: switch apacket over to a std::string payload."
am: 581a4ceb00

Change-Id: I1a4504b25b189d984f29aaf2b350a473679d1a2e

Merge "libutils: remove subdirs"

Merge "adb: switch apacket over to a std::string payload."

libutils: remove subdirs

no longer used in Android

Test/Bug: none

Change-Id: I155d5ade02527bf47893a8bf8bcbb07b0aae53d8

Merge "Document property expansion." am: a4644c9bc4
am: 31d17cf622

Change-Id: I47f0c5cb4cd6b1a6740a8f95c843c823ae00aed7

Merge "Document property expansion."
am: a4644c9bc4

Change-Id: I3dfaee7ccfcbe77cc5ae7cb02bf51551cbcc2f4b

Merge "Document property expansion."

Merge "debuggerd_fallback: don't recursively abort." am: e0068770e3
am: 47bee24270

Change-Id: I2f8d366cb875a596909f731b298b1d6141e3a8f5

Merge "debuggerd_fallback: don't recursively abort."
am: e0068770e3

Change-Id: Icd587c111d9b1713fdad5622971008b4437b1719

Merge "debuggerd_fallback: don't recursively abort."

Merge "Modify the offline handling interface." am: bc8446992f
am: c6f7a32ada

Change-Id: I4331d9771f13bb87ef30b6aef7bd958cea62d9f4

Merge "Modify the offline handling interface."
am: bc8446992f

Change-Id: If934f7a4c8244544a2f7dfa71d213122f5d37137

Merge "Modify the offline handling interface."

init: Reboot after timeout passes during reboot

There is currently a timeout for reboot, however if the system gets
stuck, particularly during file system operations, there is no safety
mechanism that guarantees the system will still reboot.

This change does all of the optional reboot steps in a separate thread
and waits for this thread with a timeout, such that if the reboot
steps get hung, the system is guaranteed to still reboot.

This is specific to 'reboot'.  Shutdown continues to run unbounded to
run fsck.

Bug: 72781711
Test: Reboot devices hitting and not hitting this timeout
Change-Id: Id5e1b3693bab00602177e28b9b662e1499c32961

Document property expansion.

Bug: N/A
Test: N/A
Change-Id: I54b9d2d08d98d83060b54981b1c5dc9660b93e8f

Merge "Adjust DEX file reading to follow layout changes in ART." am: 8fbdbba9f5
am: 1f2daf1b79

Change-Id: I25c5d81c5803b920f7e4d68cc6aebe023b439ea6

Merge "Adjust DEX file reading to follow layout changes in ART."
am: 8fbdbba9f5

Change-Id: I550ee4892c6c9ec83695ea861af9953fdb5c53fb

Merge "Adjust DEX file reading to follow layout changes in ART."

Merge "Remove fs_config entries for webview_zygote." am: 47354e76eb
am: 19fafd1ae9

Change-Id: Ia7b10849f46fd0fbcf19d355ead1d762718f2e3d

Merge "Remove fs_config entries for webview_zygote."
am: 47354e76eb

Change-Id: I9faa6c9c4a83ba6d214e37155ebd84b724bc325f

Modify the offline handling interface.

- Add a new function Backtrace::UnwindOffline that takes the stack data.
- Modify BacktraceMap::CreateOffline so it doesn't take the stack data.
  This makes it easier to reuse the map object created this way.

Reusing the map object increases simpleperf speed (unwinds per second) by 50%.

Test: backtrace_test libunwindstack_test
Change-Id: I90cfbae9e50d95d8a0e3cd394b33ba36d65d45f7

Merge "Remove fs_config entries for webview_zygote."

debuggerd_fallback: don't recursively abort.

Calls to abort() will always result in our signal handler being called,
because abort will manually unblock SIGABRT before raising it. This
can lead to deadlock when handling address space exhaustion in the
fallback handler. To fix this, switch our mutex to a recursive mutex,
and manually keep track of our lock count.

Bug: http://b/72929749
Test: debuggerd_test --gtest_filter="CrasherTest.seccomp_crash_oom"
Change-Id: I609f263ce93550350b17757189326b627129d4a7

Merge "Change the GetAdjustedRelPc to GetPcAdjustment." am: cdf778f5d9
am: 241481eb7e

Change-Id: I29b66066cccb153eb3e88e48f6cf69af32c1176b

Merge "Change the GetAdjustedRelPc to GetPcAdjustment."
am: cdf778f5d9

Change-Id: I3f0f8a9e1a4ceea708714439e54ec06bd7a25f36

Merge "Change the GetAdjustedRelPc to GetPcAdjustment."

Merge "init: log control messages along with the process that sent them" am: 0f8a67d6a8
am: 76583e1637

Change-Id: Id60a6e1ecac59f5ed875e0b7ff5f5c289e7e617e

Merge "init: log control messages along with the process that sent them"
am: 0f8a67d6a8

Change-Id: If7e485e01fb326c6ff7920c96e02e3e59b46c04b

Merge "init: log control messages along with the process that sent them"

Change the GetAdjustedRelPc to GetPcAdjustment.

This cleans up a bit of the Unwinder code to make it clear what's
going on.

Modify the offline unit tests to verify the pc and sp to make sure
that those values get computed correctly.

Test: Passes unit tests.
Test: Passes 137-cfi art tests.
Change-Id: I0787a1d77b8726d3defd08f31c7476f6798f8d0d

init: log control messages along with the process that sent them

It's currently not clear that init stops processes due to being sent a
control message nor who sent that message.

Bug: 73343913
Test: send control messages and see the logs
Change-Id: I9e9eff2001e649814107ea961b3b747a1f6da598

adb: switch apacket over to a std::string payload.

Test: python test_device.py with walleye/x86_64 emulator
Change-Id: I0a18941af1cb2279e5019a24ace25741def1202f

Merge "Add ability to skip function name resolution." am: 2733708cfb
am: d41821eade

Change-Id: Idb3e5e804df99e39281ed5c79c2ea3c747abce8f

Merge "Add ability to skip function name resolution."
am: 2733708cfb

Change-Id: If75fe6d573d44ddb6ef0e562688bea88654717b3

Merge "Add ability to skip function name resolution."

Merge "Base: Add an NDK-compiled version of libbase" am: 9b4d7dd51e
am: fdabcbb61a

Change-Id: I9ac3346158b182dc10cbf78094658162755d9ff1

Merge "Base: Add an NDK-compiled version of libbase"
am: 9b4d7dd51e

Change-Id: If0dddc912cdb1aa49ad5c166fc136d2dc9d1896a

Merge "Base: Add an NDK-compiled version of libbase"

Base: Add an NDK-compiled version of libbase

To support building libraries for CTS.

Bug: 73206894
Bug: 73659009
Test: mmma system/core/base
Change-Id: I630be88f44cf5cefcb13be7959fc1a710686e29b

Add ability to skip function name resolution.

Bug: 73558129

Test: New unit tests pass.
Change-Id: I3a6b17d2590aacb367ab3e3a51cc85be73ad64eb

Merge "Expose __android_log_buf_print and __android_log_buf_write to NDK" am: 887b89fe5e
am: 4ebf2495a9

Change-Id: Id5a0bbcd461a21167d8507ca75c681df4371d158

Merge "Expose __android_log_buf_print and __android_log_buf_write to NDK"
am: 887b89fe5e

Change-Id: Idbbdf1940d3e6320a5ddb8d56d5ca1510abfb2ce

Merge "Expose __android_log_buf_print and __android_log_buf_write to NDK"

Merge "Fix intermittent crash in property_service_test" am: ca26cbc5d3
am: f157725091

Change-Id: If6f95cec386cb4351abfa08f6c85ce6ac3846fb4

Merge "Fix intermittent crash in property_service_test"
am: ca26cbc5d3

Change-Id: Ieaa5218faf9deee0443b88fff6be43d4a0877ec9

Expose __android_log_buf_print and __android_log_buf_write to NDK

These are needed to build libbase with the NDK.

Bug: 73658597
Test: tree hugger
Change-Id: Icf4a82c5c08975731c180be155743ac63f4bfd50

Merge "Fix intermittent crash in property_service_test"

Fix intermittent crash in property_service_test

There is a race in the very_long_name_35166374 test of
property_service.  The test first sends a size value that is beyond
the limit that init will handle, then sends a dummy data value.
However, init closes the socket upon seeing the faulty size, and if
this happens before the test sends the dummy data, the test will crash
due to SIGPIPE.

Since there is no reason to send the dummy data at all, this change no
longer sends it to prevent the crash.  It also now checks explicitly
that init returns an error through the socket.

Bug: 73619375
Test: the unit test in question
Change-Id: I2565a69fa54910cee0e15fc798445e18c91156ec

Merge "bpfmt." am: 0609e8d231
am: b57755c429

Change-Id: Ieeedd2bd96048496ad4141326af3114d3f44eaf1

Merge "bpfmt."
am: 0609e8d231

Change-Id: I810b09372da24d4b84bbeaa5d0c03ec0baf928fd

Merge "bpfmt."

Merge "Fix a memory leak" am: cb1b68f9b3
am: 894337ee0b

Change-Id: Ieb9b695cc681831fc3684bcb5d19ddc41099949f

Merge "Fix a memory leak"
am: cb1b68f9b3

Change-Id: Ia7a5fbc737d702615499bccd6492aac9ce3d0ce3

Merge "Fix a memory leak"

Merge "debuggerd seccomp: explain why we define PROT_READ/WRITE." am: 665e4c1537
am: 5504477722

Change-Id: I935382f7581f1e0220c9a362bfe0f659d7ee598a

Merge "debuggerd seccomp: explain why we define PROT_READ/WRITE."
am: 665e4c1537

Change-Id: I3762171a451d169d273a4affa3bf3c6eee7a3b3c

Merge "Define PROT_READ and PROT_WRITE when generating the policy files" am: a1721a5043
am: 158208cde0

Change-Id: I6855db3d6f56067983fd89e2c330223cb82ab9a2

Merge "debuggerd seccomp: explain why we define PROT_READ/WRITE."

Merge "Define PROT_READ and PROT_WRITE when generating the policy files"
am: a1721a5043

Change-Id: Ibb4459bac1136ce9c54b7a109bb7b7dc72986563

Merge "Define PROT_READ and PROT_WRITE when generating the policy files"

Fix a memory leak

If create_service_thread fails, we'll leak `arg`.

This fixes a static analyzer complaint:
system/core/adb/services.cpp:298:13: warning: Potential leak of memory
pointed to by 'arg'

Bug: None
Test: Reran the static analyzer. No more complaints about this leak.
Change-Id: I5aec7fd78f2cc775b650501b02bdf0039d1647ca

debuggerd seccomp: explain why we define PROT_READ/WRITE.

Add a comment explaining why we define PROT_READ/PROT_WRITE, even
though a current libminijail supports both cosntants.

Bug: http://b/73273658
Test: treehugger
Change-Id: I51c1be1b1b569e94dbc9045a90bc28221b7dc9c7

Define PROT_READ and PROT_WRITE when generating the policy files

When generating crash_dump.*.policy, replace PROT_READ and PROT_WRITE
to numeric constants to make the policy backward compatible with old
libminijail.so.

Bug: 73273658
Test: use the new policy in OMR1 devices
Change-Id: I936a733340ad4df8aef6562c03eb10c29ffdada2

Merge "Modify elf cache to handle elf_offsets properly." am: 277a95bfef
am: e89745202c

Change-Id: Idf566f7aa0cc1df292cdf4da7c982388b4fa5bde

Merge "Fix darwin build" am: 8c0c1ba989
am: e68fa81351

Change-Id: Ia9d754f5a71ba2a6d926139e3eb1b1082b1ef1cb

Merge "Modify elf cache to handle elf_offsets properly."
am: 277a95bfef

Change-Id: I71c15df8d527548610cc15484f7c382e85b54552

Merge "Fix darwin build"
am: 8c0c1ba989

Change-Id: I8acb6a80f4ab5458f01fb6196203c4c664438934

Merge "Modify elf cache to handle elf_offsets properly."

bpfmt.

Bug: N/A
Test: builds
Change-Id: I89ad00e1c4c7e0767bc80a7ac7935a4d55e090ac

Merge "Fix darwin build"

Fix darwin build

Darwin needs libbase too.

Test: 'm'
Change-Id: I9d50face92b7d5a4399dbcebd39d4062477b3a2c

Modify elf cache to handle elf_offsets properly.

Bug: 73498823

Test: All unit tests pass.
Test: Simpleperf run that previously failed, passes now.
Change-Id: Iff3a1f2f641a46ab9a0326579af3649f0c76fc65

Merge "Fix soname reading code." am: 76eda07ff6
am: 299aa7a646

Change-Id: Iac543521ec34db809ce5b0612885450c21b6707d