diff options
| author | Tom Cherry <tomcherry@google.com> | 2018-02-20 10:40:26 -0800 |
|---|---|---|
| committer | Tom Cherry <tomcherry@google.com> | 2018-02-20 10:47:55 -0800 |
| commit | b7ef7e7aff81893d327cbb14d2c803ac7c5e65e5 (patch) | |
| tree | 76e4500ccabbaa104f325ccc93def140db0c944a /debuggerd/debuggerd_test.cpp | |
| parent | e1ae2ff9e46547dfaa3947eedd5e471332c92a86 (diff) | |
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
Diffstat (limited to 'debuggerd/debuggerd_test.cpp')
0 files changed, 0 insertions, 0 deletions