diff options
| author | Sylvain Defresne <sdefresne@chromium.org> | 2016-08-09 12:43:39 -0700 |
|---|---|---|
| committer | Sylvain Defresne <sdefresne@chromium.org> | 2016-08-09 23:13:07 +0000 |
| commit | 600ec35c5b0bf503b3f437c5f378eadbc8d58601 (patch) | |
| tree | 1f0317e46ebeeb4d1f86fe28ae2cf6c1fffa2261 /src/common/test_assembler.cc | |
| parent | Don't define |r_debug| and |link_map| on Android releases 21 and later (diff) | |
| download | breakpad-600ec35c5b0bf503b3f437c5f378eadbc8d58601.tar.xz | |
Fail with a proper error message if input file is not found.
Previously, if the input file was missing, the symupload tool on Mac
would happily process, try to parse it (calling a method on nil) and
fail when trying to create the payload to send to the server as one
of the method raised a NSInvalidArgumentException when receiving a
nil value.
Change to code to instead check the file for existence which makes it
easier to understand what is happening when part of the build system
is misconfigured and invoke symupload without first creating the symbol
file.
BUG=449348
Change-Id: Icc0f08958114da4be0cbbd7a7c2aeef905bc0db1
Reviewed-on: https://chromium-review.googlesource.com/367260
Reviewed-by: Mark Mentovai <mark@chromium.org>
Diffstat (limited to 'src/common/test_assembler.cc')
0 files changed, 0 insertions, 0 deletions