Closed Bug 1109940 Opened 9 years ago Closed 9 years ago

Add symbol wrapping for Android DNS resolver

Categories

(Firefox Build System :: General, defect)

x86_64
Android
defect
Not set
normal

Tracking

(firefox36 fixed, firefox37 fixed)

RESOLVED FIXED
mozilla37
Tracking Status
firefox36 --- fixed
firefox37 --- fixed

People

(Reporter: esawin, Assigned: esawin)

References

Details

Attachments

(1 file, 1 obsolete file)

Bug 1077366 removed all symbol wrappers for Android, which breaks the conditional getaddrinfo call chaining for Android 3 and later versions.

We need to add the symbol wrappers back for the Android DNS resolver functions.
Depends on: 1077366
Attached patch wrap-dns-resolver-func (obsolete) — Splinter Review
Assignee: nobody → esawin
Status: NEW → ASSIGNED
Attachment #8534659 - Flags: review?(mh+mozilla)
Blocks: 1077366
No longer depends on: 1077366
Comment on attachment 8534659 [details] [diff] [review]
wrap-dns-resolver-func

Review of attachment 8534659 [details] [diff] [review]:
-----------------------------------------------------------------

::: memory/build/mozmemory_wrap.h
@@ +62,5 @@
> + * - On Android, getaddrinfo, freeaddrinfo and gai_strerror are prefixed with
> + *   "__wrap_". Additionally, C++ allocation functions (operator new/delete)
> + *   are also exported and unprefixed.
> + *
> + * - On Gonk, all functions are left unprefixed. Additionally,

This comment is about memory allocator symbols. Please don't change it.
Attachment #8534659 - Flags: review?(mh+mozilla) → review+
Removed comment edits.
Attachment #8534659 - Attachment is obsolete: true
Attachment #8534993 - Flags: review+
Blocks: 991923
Blocks: 1110529
https://hg.mozilla.org/mozilla-central/rev/ca8735e95bad
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
The changeset https://hg.mozilla.org/integration/fx-team/rev/d9fd6f35c45d references this bug, but actually belongs to bug 1110529.
Comment on attachment 8534993 [details] [diff] [review]
wrap-dns-resolver

Approval Request Comment
[Feature/regressing bug #]: bug 1077366 
[User impact if declined]: blocks bug 991923, major connectivity issues when switching networks or using VPNs
[Describe test coverage new/current, TBPL]: has been on nightly for some days
[Risks and why]: none, failure to resolve function symbols is deterministic and would have rendered nightly unable to resolve any address or load any page
[String/UUID change made/needed]: none
Attachment #8534993 - Flags: approval-mozilla-aurora?
Attachment #8534993 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.