On 05/22/2018 09:53 AM, Stephen Smalley wrote: > On 05/22/2018 09:11 AM, Stephen Smalley wrote: >> On 05/22/2018 09:01 AM, Stephen Smalley wrote: >>> On 05/22/2018 07:37 AM, Richard Haines wrote: >>>> Could you try this version where I've packed the transaction structures. >>>> I could not get the tests to fail on my two systems (but then V3 didn't). >>> >>> Hmmm...I saw one instance of a failure in test 6 when running ./test by >>> hand but am now having problems replicating it. >>> >>> dmesg output during all 6 tests was: >>> >>> [ 263.831513] binder: release 2025:2025 transaction 2 out, still active >>> [ 263.831519] binder: 2024:2024 transaction failed 29189/0, size 24-8 line 2788 >>> [ 263.831522] binder: send failed reply for transaction 2, target dead >>> [ 263.846321] binder: 2026:2026 transaction failed 29201/-1, size 24-8 line 2864 >>> [ 263.858613] binder: 2024:2027 transaction failed 29201/-1, size 0-0 line 2864 >>> [ 263.872764] binder: 2028:2028 transaction failed 29201/-1, size 24-8 line 3050 >>> [ 263.883684] binder: 2029:2029 ioctl 40046207 0 returned -13 >>> [ 263.895261] binder: 2030 RLIMIT_NICE not set >>> [ 264.151699] binder: 2030 RLIMIT_NICE not set >>> [ 264.151879] binder: 2030 RLIMIT_NICE not set >>> [ 264.152212] binder: undelivered transaction 19, process died. >>> [ 264.152219] binder: 2030 RLIMIT_NICE not set >>> [ 264.153438] binder: 2030 RLIMIT_NICE not set >>> >>> Are all of those expected? >> >> Now it is repeating upon a fresh reboot and running ./test by hand repeatedly. >> Also seeing these errors: >> [ 176.467915] binder_alloc: 1998: binder_alloc_buf, no vma >> [ 176.468046] binder: undelivered TRANSACTION_ERROR: 29189 >> >> Running it via make test passes though, oddly enough. >> >> This is with completely stock 4.17-rc5 on F28. > > ./test -v reports the following for test 6: > Manager PID: 1949 Process context: > unconfined_u:unconfined_r:test_binder_mgr_no_fd_t:s0-s0:c0.c1023 > Service Provider PID: 1950 Process context: > unconfined_u:unconfined_r:test_binder_provider_t:s0-s0:c0.c1023 > Service Provider sending transaction to Manager - ADD_TEST_SERVICE > Service Provider read_consumed: 48 > Service Provider command: BR_NOOP > Service Provider command: BR_INCREFS > Service Provider command: BR_ACQUIRE > Service Provider command: BR_TRANSACTION_COMPLETE > Manager read_consumed: 72 > Manager command: BR_NOOP > Manager command: BR_TRANSACTION > Manager BR_TRANSACTION data: > handle: 0 > cookie: 0 > code: 100 > flag: TF_ACCEPT_FDS > sender pid: 1950 > sender euid: 0 > data_size: 24 > offsets_size: 8 > hdr: BINDER_TYPE_HANDLE > handle: 1 > flags: priority: 0x7f accept FDS: YES > cookie: 0 > Manager has BINDER_TYPE_HANDLE obj->handle: 1 > Manager acquired handle: 1 for Service Provider > Manager sending BC_REPLY to obtain its FD > Manager handle: 0 and its FD: 3 > Manager read_consumed: 8 > Manager command: BR_NOOP > Manager command: BR_TRANSACTION_COMPLETE > Service Provider read_consumed: 72 > Service Provider command: BR_NOOP > Service Provider command: BR_REPLY > Service Provider BR_REPLY data: > handle: 0 > cookie: 0 > code: 100 > flag: TF_ACCEPT_FDS > sender pid: 0 > sender euid: 0 > data_size: 24 > offsets_size: 8 > hdr: BINDER_TYPE_FD > fd: 5 > flags: priority: 0x7f accept FDS: YES > cookie: 0 > Service Provider retrieved Managers fd: 5 st_dev: 6 > Service Provider read_consumed: 8 > Service Provider command: BR_NOOP > Service Provider command: BR_TRANSACTION_COMPLETE > Service Provider using Managers FD > Manager read_consumed: 4 > Manager command: BR_NOOP > not ok 6 > # Failed test at ./test line 84. > # Looks like you failed 1 test of 6. Never mind, bug in user, forgot to setsebool domain_fd_use=off before running after a reboot (or re-running make -C policy load). Test passes for me now.
