diff options
author | Olivier Robin <olivierrobin@chromium.org> | 2016-01-21 11:34:12 -0500 |
---|---|---|
committer | Mark Mentovai <mark@chromium.org> | 2016-01-21 11:34:12 -0500 |
commit | 658eb44f3f6d0286ab8986bf34562159b40c99e9 (patch) | |
tree | 2043331576199e9689cd02d147bf6bb01195d1d1 /src/client/mac/handler | |
parent | exploitability_unittest: fix warnings (diff) | |
download | breakpad-658eb44f3f6d0286ab8986bf34562159b40c99e9.tar.xz |
Revert "Fix deprecatation warning when building for recent SDKs on iOS/OS X."
This reverts CL https://codereview.chromium.org/1563223004/
This reverts commit 7cc0d8562bf8b20b88cc941ba72593cb7230ecf6.
CL 1563223004 introduces two bugs on iOS.
- Encoding the minidump name with extra percent causing crash server to fail
processing the file.
- Using a released pointer causing random crashes on upload. The
data, resp, err pointers returned in the NSURLSession completion
handler is released at the end of the block. When used later (to get
the crash ID), it causes a crash.
BUG=569158
R=blundell@chromium.org, mark@chromium.org
Review URL: https://codereview.chromium.org/1619603002 .
Patch from Olivier Robin <olivierrobin@chromium.org>.
Diffstat (limited to 'src/client/mac/handler')
-rw-r--r-- | src/client/mac/handler/minidump_generator.cc | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/client/mac/handler/minidump_generator.cc b/src/client/mac/handler/minidump_generator.cc index 0e299180..7d6e81db 100644 --- a/src/client/mac/handler/minidump_generator.cc +++ b/src/client/mac/handler/minidump_generator.cc @@ -154,9 +154,9 @@ void MinidumpGenerator::GatherSystemInformation() { if (!data) { return; } - CFDictionaryRef list = - static_cast<CFDictionaryRef>(CFPropertyListCreateWithData( - NULL, data, kCFPropertyListImmutable, NULL, NULL)); + CFDictionaryRef list = static_cast<CFDictionaryRef> + (CFPropertyListCreateFromXMLData(NULL, data, kCFPropertyListImmutable, + NULL)); CFRelease(data); if (!list) { return; |