diff options
| author | Felipe Leme <felipeal@google.com> | 2016-02-03 09:28:24 -0800 |
|---|---|---|
| committer | Felipe Leme <felipeal@google.com> | 2016-02-03 10:16:34 -0800 |
| commit | 111b9d06cc0fc72438782c9234f28675e5077ef4 (patch) | |
| tree | 6228b625775a8507841f77c26af24740c15ded45 /opengl/tests/gl_basic/gl_basic.cpp | |
| parent | 858a4332ff17ebbfb57df4cbac76ce61cdaf63e7 (diff) | |
Improved how the Shell directories are created.
When dumpstate is run for the first time, the
/data/data/com.android.shell/files/bugreports does not exist, which was
crashing dumpstate because the code that added the version.txt entry was
not checking if the zip_writer was NULL.
The crash itself was fixed by adding a sanity check in the functions
that add entries to the zip file, but that only hid the real problem:
it is necessary to create the parent directories before creating the zip
file, otherwise the first run will always generate a .txt file (since
dumpstate falls back to .txt when it cannot create the .zip).
This change also improves how the parent directories are created by
checking if they exist first, rather than always calling mkdir().
BUG: 26949960
Change-Id: I1434be5c36a3fad0b3a2a26c7eaaab03a1228c30
Diffstat (limited to 'opengl/tests/gl_basic/gl_basic.cpp')
0 files changed, 0 insertions, 0 deletions