Commit graph

27276 commits

Author SHA1 Message Date
Jin Qian
f4b9def2db Merge "fs_mgr: fix clang static analyzer warning" am: d945f27cd6
am: a1299d28eb

Change-Id: I3aedde8f3f325f8f2af71ac4569921b3b4f2c0f7
2017-01-24 21:02:40 +00:00
Jin Qian
a1299d28eb Merge "fs_mgr: fix clang static analyzer warning"
am: d945f27cd6

Change-Id: I39f9be5faec32f85880a8c571f0773142322dbcd
2017-01-24 21:00:39 +00:00
Treehugger Robot
d945f27cd6 Merge "fs_mgr: fix clang static analyzer warning" 2017-01-24 20:58:19 +00:00
Dimitry Ivanov
34770bcf94 Merge "Revert "New protocol for property_service"" am: 17767caed9
am: 54339350af

Change-Id: I064b4ea247a6c22544f789f041194e510dfca0b3
2017-01-24 20:07:44 +00:00
Dimitry Ivanov
54339350af Merge "Revert "New protocol for property_service""
am: 17767caed9

Change-Id: I59613b73239d70216fbbe505f48dc913308c4dd0
2017-01-24 20:05:44 +00:00
Dimitry Ivanov
17767caed9 Merge "Revert "New protocol for property_service"" 2017-01-24 20:00:44 +00:00
Vijay Venkatraman
a7d53bcaf2 Merge "Replaced include log/logger.h with log/log.h in debuggerd" am: 5fe1cebe7c
am: 4a7eb14276

Change-Id: I75890537b17cd9ea6c081a0926320730bb01ab8a
2017-01-24 19:14:34 +00:00
Vijay Venkatraman
4a7eb14276 Merge "Replaced include log/logger.h with log/log.h in debuggerd"
am: 5fe1cebe7c

Change-Id: I9c5078ae7e9513767540897d791adac569bce894
2017-01-24 19:11:03 +00:00
Vijay Venkatraman
5fe1cebe7c Merge "Replaced include log/logger.h with log/log.h in debuggerd" 2017-01-24 19:07:30 +00:00
Dimitry Ivanov
70c4ecf2b5 Revert "New protocol for property_service"
This reverts commit dee4bd236b.
Bug: http://b/33926793
Bug: http://b/34670529

Change-Id: Ife3a5a471ee29cb12c2c41efef885ba40b8970e6
2017-01-24 18:38:09 +00:00
Colin Cross
33d0bb6483 Merge "Allow tkill" am: 9fd1190790
am: 81b76391d3

Change-Id: Ib141600dbd9a5471a1c452d2514f89e0d234b5d6
2017-01-24 04:37:57 +00:00
Colin Cross
81b76391d3 Merge "Allow tkill"
am: 9fd1190790

Change-Id: Id45daedeecc72308db9616b302a7bf23bd4beb28
2017-01-24 04:34:57 +00:00
Treehugger Robot
9fd1190790 Merge "Allow tkill" 2017-01-24 04:31:26 +00:00
Colin Cross
90d299f9f0 Merge "Allow restart_syscall" am: 1b3350e4ca
am: 5b0dd3cbe3

Change-Id: Ibd386810c4d6d2f955ff76e69c71b67101ec6c08
2017-01-24 04:12:26 +00:00
Vijay Venkatraman
a95acea5bf Replaced include log/logger.h with log/log.h in debuggerd
Test: Compile debuggerd
Change-Id: I33d12708993bd6818907b8cbb0a7a40672ad320d
2017-01-23 20:11:51 -08:00
Colin Cross
5b0dd3cbe3 Merge "Allow restart_syscall"
am: 1b3350e4ca

Change-Id: I204d0bfb4c80aada7d7e4027c274f3c07ce22783
2017-01-24 04:10:55 +00:00
Treehugger Robot
1b3350e4ca Merge "Allow restart_syscall" 2017-01-24 04:08:48 +00:00
Hung-ying Tyan
70410e7c41 Merge "init: add ro.boot.init_rc" am: 571cd23fa8
am: fc8f2db476

Change-Id: I9ab39983a551e9f13a6d163a9f0fa2efd904db0f
2017-01-24 04:03:54 +00:00
Hung-ying Tyan
fc8f2db476 Merge "init: add ro.boot.init_rc"
am: 571cd23fa8

Change-Id: Ie945f49d9ce1fe849140d522b53f6346865d2f1e
2017-01-24 04:01:25 +00:00
Treehugger Robot
571cd23fa8 Merge "init: add ro.boot.init_rc" 2017-01-24 03:59:13 +00:00
Dimitry Ivanov
ac4956bff3 Merge "New protocol for property_service" am: 038b8f086f
am: 8077c96655

Change-Id: Icfd892a8f756e85b0992ab4b8b40b196c07f2aa1
2017-01-24 02:58:27 +00:00
Dimitry Ivanov
8077c96655 Merge "New protocol for property_service"
am: 038b8f086f

Change-Id: Ica1faa48b08b638ab9a77cb29d5d74a74fd0071e
2017-01-24 02:56:29 +00:00
Dimitry Ivanov
038b8f086f Merge "New protocol for property_service" 2017-01-24 02:54:34 +00:00
Josh Gao
a5a61e28cb Merge changes I8b3e8a3b,I4976abef,I31dadb9c am: a2fd28ee06
am: 33762ff915

Change-Id: I4e244fb4240c12a45489234c04290fcd7e972dec
2017-01-24 02:37:34 +00:00
Josh Gao
44877b9c29 tombstoned: silence spurious error messages. am: 8498016b81
am: 8fb50dc76c

Change-Id: I903ece25c820b81b26bff35e8c09f0e126ea32c1
2017-01-24 02:37:29 +00:00
Josh Gao
bf8bd27bb3 Actually don't start tombstoned until /data is mounted. am: 0ad5107e51
am: 1170d8201d

Change-Id: I654b9f033eba8db137e83effe63a79a80c1abb0f
2017-01-24 02:37:24 +00:00
Josh Gao
1529fe4c93 debuggerd_handler: add SIGSYS to the list of handled signals. am: 4decbe0d6c
am: 7325c7a220

Change-Id: Ib74ec1abe20d719b23f1e6d119f85722c136b1eb
2017-01-24 02:37:18 +00:00
Josh Gao
33762ff915 Merge changes I8b3e8a3b,I4976abef,I31dadb9c
am: a2fd28ee06

Change-Id: I9fd36550b5a03b7cbccd77d39f7facb877ec1426
2017-01-24 02:35:00 +00:00
Josh Gao
8fb50dc76c tombstoned: silence spurious error messages.
am: 8498016b81

Change-Id: I6385f9fbcdd17e67d7e60bcfddd9c164b24e7f8c
2017-01-24 02:34:59 +00:00
Josh Gao
1170d8201d Actually don't start tombstoned until /data is mounted.
am: 0ad5107e51

Change-Id: I829a088921adb6253a1dec3c39a1ffe171b90f5c
2017-01-24 02:34:54 +00:00
Josh Gao
7325c7a220 debuggerd_handler: add SIGSYS to the list of handled signals.
am: 4decbe0d6c

Change-Id: I01f3ff76051bddc16abe8b4909ffb22b62e0f372
2017-01-24 02:34:49 +00:00
Josh Gao
a2fd28ee06 Merge changes I8b3e8a3b,I4976abef,I31dadb9c
* changes:
  tombstoned: silence spurious error messages.
  Actually don't start tombstoned until /data is mounted.
  debuggerd_handler: add SIGSYS to the list of handled signals.
2017-01-24 02:27:20 +00:00
Colin Cross
2e253cbcc6 Allow tkill
tkill and tgkill are syscalls that do not have glibc wrappers, but
they are required in order to send a signal to a specific thread.
Non-android software may call them directly with syscall().  Bionic
provides a wrapper for tgkill, so seccomp allows it, but not for
tkill.  Add tkill to the whitelist.

This can be reproduced with:
sleep 1000 & sleep 1 ; strace -p $!
then hit ctrl-C, and the shell will print "Bad system call" because
the strace process died with SIGSYS

Bug: 34586922
Test: repro case is fixed
Change-Id: Ib6962a967f2cc757f2906de7905e75e1b6d6f39f
2017-01-23 17:10:51 -08:00
Colin Cross
47afc6b64e Allow restart_syscall
restart_syscall is used by the kernel whenever a syscall with a
timeout is interrupted.  Whitelist it in seccomp to prevent
processes being killed with SIGSYS when being ptraced.

Bug: 34586922
Test: hwui_unit_tests
Change-Id: Ic47dcad33f3082eb5673c3c67fe17200d4daaf74
2017-01-23 17:05:01 -08:00
Josh Gao
8498016b81 tombstoned: silence spurious error messages.
Bug: none
Test: booted after deleting /data/tombstones/*
Test: crasher creates a tombstone
Change-Id: I8b3e8a3b521952412ebc955b2437bf8150220c16
2017-01-23 16:01:14 -08:00
Josh Gao
0ad5107e51 Actually don't start tombstoned until /data is mounted.
Bug: http://b/34461270
Test: boot is actually faster
Test: tombstoned still started by init
Change-Id: I4976abef108bbb6fad264f9b68cbc1fba711085b
2017-01-23 16:01:14 -08:00
Josh Gao
4decbe0d6c debuggerd_handler: add SIGSYS to the list of handled signals.
Bug: http://b/34586922
Test: kill -SYS $$
Change-Id: I31dadb9c65141d0c5556cc7256439e0a8d1519ab
2017-01-23 16:01:14 -08:00
Dimitry Ivanov
dee4bd236b New protocol for property_service
New protocol assumes that there is no limit on name or value
and effectively removed limit on property name length.

It also send back a uint32_t with error code (or 0 on success)

Bug: http://b/33926793
Test: mm, boot, run bionic-unit-tests --gtest_filter=prop*
Change-Id: Iac6290398ddc495e03f8fbbc3a79e923eff5df6f
2017-01-23 15:30:05 -08:00
Mark Salyzyn
568ae3226e Merge "logcat: test: inject messages if necessary for tail test requirements" am: d965fbfc9a
am: 2256a8d13e

Change-Id: Icf290d54c055f9f5d4940c36dfabc524f59d24ba
2017-01-23 23:23:25 +00:00
Mark Salyzyn
2256a8d13e Merge "logcat: test: inject messages if necessary for tail test requirements"
am: d965fbfc9a

Change-Id: I54c0a687c054d49fe047f96e3b72a5c38c776158
2017-01-23 23:22:26 +00:00
Treehugger Robot
d965fbfc9a Merge "logcat: test: inject messages if necessary for tail test requirements" 2017-01-23 23:11:56 +00:00
Josh Gao
38988c0720 Merge "debuggerd_handler: properly crash when PR_GET_DUMPABLE is 0." am: 90e05f68e2
am: 4e0648e7ca

Change-Id: If6821ade9e57e439e46a10efd008c8f55a6b160c
2017-01-23 22:38:22 +00:00
Josh Gao
4e0648e7ca Merge "debuggerd_handler: properly crash when PR_GET_DUMPABLE is 0."
am: 90e05f68e2

Change-Id: Ie98457c464d36c183fe1d1c366d7b93ac2594657
2017-01-23 22:37:21 +00:00
Josh Gao
90e05f68e2 Merge "debuggerd_handler: properly crash when PR_GET_DUMPABLE is 0." 2017-01-23 22:30:56 +00:00
Josh Gao
fca7ca3585 debuggerd_handler: properly crash when PR_GET_DUMPABLE is 0.
Actually exit when receiving a signal via kill(2) or raise(2) and
PR_GET_DUMPABLE is 0.

Bug: none
Test: /data/nativetest/debuggerd_test/debuggerd_test32
Test: /data/nativetest64/bionic-unit-tests/bionic-unit-tests --gtest_filter=pthread_DeathTest.pthread_mutex_lock_null_64
Change-Id: I833a2a34238129237bd9f953959ebda51d8d04d7
2017-01-23 14:13:36 -08:00
Mark Salyzyn
15e585853f logcat: test: inject messages if necessary for tail test requirements
For the gTests logcat.tz, logcat.tail_3, logcat.tail_10,
logcat.tail_100, logcat.tail_1000 and logcat.tail_time tests inject
messages if we come up short should the background logging not be
sufficient to feed the test requirements.  Test frames run the
tests right after reboot requiring injection.  Will have radio
silence, so we also add kernel logs if available to help add to the
background logging activity level.

We also will inject a radio message for logcat.buckets so that test
can survive no radio content.

Test: set device to airplane mode (to stress this problem) then:
      adb logcat -b all -c && adb shell su root \
          /data/nativetest/logcat-unit-tests/logcat-unit-tests \
          --gtest_filter=logcat.tail_1000 (or others in this set)
Bug: 34454772
Change-Id: I5ec246552f3ab1fc9c5864ed69d63b851fdf538d
2017-01-23 13:33:02 -08:00
Josh Gao
142f66d2dc Merge changes Iacaa796f,I7549f674,Ic9d70880,I96cb09b7,I5c2658a8 am: 400973fa88
am: 450ad07aaf

Change-Id: Iaa6e3fa67d6dc62a3a8dc1441028aea173733915
2017-01-23 20:59:42 +00:00
Josh Gao
40e84df9a3 crash_dump: set a watchdog timer. am: 7c6e3133f5
am: dac69b6bf1

Change-Id: I566ac308ee1e98931ca77616bb8e2c6b3c9f9431
2017-01-23 20:59:37 +00:00
Josh Gao
85a09f61b4 crash_dump: switch to PTRACE_SEIZE. am: 122479f4a0
am: 6121036e3a

Change-Id: Iafabed942028cd2ffe4db0d68c654b7f8f4f733f
2017-01-23 20:59:32 +00:00
Josh Gao
122587b316 crash_dump: clear the default crash handlers. am: 575941115e
am: 110f6f8df8

Change-Id: Ifb2fac743d0e44dc483ad8634081792b9c4dfb02
2017-01-23 20:59:27 +00:00