Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(84)

Side by Side Diff: src/common/android/include/link.h

Issue 2156173002: Don't define |r_debug| and |link_map| on Android releases 21 and later Base URL: https://chromium.googlesource.com/breakpad/breakpad.git@master
Patch Set: Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012, Google Inc. 1 // Copyright (c) 2012, Google Inc.
2 // All rights reserved. 2 // All rights reserved.
3 // 3 //
4 // Redistribution and use in source and binary forms, with or without 4 // Redistribution and use in source and binary forms, with or without
5 // modification, are permitted provided that the following conditions are 5 // modification, are permitted provided that the following conditions are
6 // met: 6 // met:
7 // 7 //
8 // * Redistributions of source code must retain the above copyright 8 // * Redistributions of source code must retain the above copyright
9 // notice, this list of conditions and the following disclaimer. 9 // notice, this list of conditions and the following disclaimer.
10 // * Redistributions in binary form must reproduce the above 10 // * Redistributions in binary form must reproduce the above
(...skipping 25 matching lines...) Expand all
36 36
37 // TODO(rmcilroy): Remove this file once the ndk is updated for other 37 // TODO(rmcilroy): Remove this file once the ndk is updated for other
38 // architectures - crbug.com/358831 38 // architectures - crbug.com/358831
39 #if !defined(__aarch64__) && !defined(__x86_64__) && \ 39 #if !defined(__aarch64__) && !defined(__x86_64__) && \
40 !(defined(__mips__) && _MIPS_SIM == _ABI64) 40 !(defined(__mips__) && _MIPS_SIM == _ABI64)
41 41
42 #ifdef __cplusplus 42 #ifdef __cplusplus
43 extern "C" { 43 extern "C" {
44 #endif // __cplusplus 44 #endif // __cplusplus
45 45
46 // The structures below are present in ndk 21 and later.
47 #if defined(ANDROID) && ANDROID_VERSION <= 20
rmcilroy 2016/07/18 15:04:08 I don't think the defined(ANDROID) is needed - the
46 struct r_debug { 48 struct r_debug {
47 int r_version; 49 int r_version;
48 struct link_map* r_map; 50 struct link_map* r_map;
49 ElfW(Addr) r_brk; 51 ElfW(Addr) r_brk;
50 enum { 52 enum {
51 RT_CONSISTENT, 53 RT_CONSISTENT,
52 RT_ADD, 54 RT_ADD,
53 RT_DELETE } r_state; 55 RT_DELETE } r_state;
54 ElfW(Addr) r_ldbase; 56 ElfW(Addr) r_ldbase;
55 }; 57 };
56 58
57 struct link_map { 59 struct link_map {
58 ElfW(Addr) l_addr; 60 ElfW(Addr) l_addr;
59 char* l_name; 61 char* l_name;
60 ElfW(Dyn)* l_ld; 62 ElfW(Dyn)* l_ld;
61 struct link_map* l_next; 63 struct link_map* l_next;
62 struct link_map* l_prev; 64 struct link_map* l_prev;
63 }; 65 };
66 #endif
64 67
65 #ifdef __cplusplus 68 #ifdef __cplusplus
66 } // extern "C" 69 } // extern "C"
67 #endif // __cplusplus 70 #endif // __cplusplus
68 71
69 #endif // !defined(__aarch64__) && !defined(__x86_64__) 72 #endif // !defined(__aarch64__) && !defined(__x86_64__)
70 73
71 #endif /* GOOGLE_BREAKPAD_ANDROID_INCLUDE_LINK_H */ 74 #endif /* GOOGLE_BREAKPAD_ANDROID_INCLUDE_LINK_H */
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698