1510301 - __dl_get_mappable_length seems to crash on Android x86

Status: RESOLVED FIXED

(Core :: mozglue, defect, P3)

Description

[Markus Stange [:mstange]]

This was encountered in bug 1507709, which, for some reason, caused the test "test_ext_geckoProfiler_control.js" to be run on the Android x86 platform where it wasn't running before.

Try job with this crash: https://treeherder.mozilla.org/#/jobs?repo=try&revision=0e4d489ba70d813d69918172cfaae7293f5ecead&selectedJob=212157393

> PROCESS-CRASH | xpcshell.ini:toolkit/components/extensions/test/xpcshell/test_ext_geckoProfiler_control.js | application crashed [@ libmozglue.so + 0x1a1f1]
> Crash dump filename: /tmp/tmpIe1CNb/3665800a-018c-0feb-9f22-624edca2e637.dmp
> Operating system: Android
>                   0.0.0 Linux 2.6.29 #1 PREEMPT Thu Nov 7 22:27:50 UTC 2013 i686
> CPU: x86
>      GenuineIntel family 6 model 3 stepping 3
>      1 CPU
> GPU: UNKNOWN
> Crash reason:  SIGSEGV
> Crash address: 0x7363708c
> Process uptime: not available
> Thread 0 (crashed)
>  0  libmozglue.so + 0x1a1f1
>     eip = 0xb7cfa1f1   esp = 0xbfa2a8f0   ebp = 0xbfa2a908   ebx = 0xb7e2fff4
>     esi = 0xb802c850   edi = 0xb802c860   eax = 0x73637078   ecx = 0x00000001
>     edx = 0x00000000   efl = 0x00000246
>     Found by: given as instruction pointer in context
>  1  libxul.so!AutoObjectMapperFaultyLib::Map(void**, unsigned int*, std::__ndk1::basic_string<char, std::__ndk1::char_traits<char>, std::__ndk1::allocator<char> >) [AutoObjectMapper.cpp:0e4d489ba70d813d69918172cfaae7293f5ecead : 186 + 0x8]
>     eip = 0xb4c44695   esp = 0xbfa2a910   ebp = 0xbfa2a9a8
>     Found by: previous frame's frame pointer
>  2  libxul.so!read_procmaps(lul::LUL*) [platform-linux-lul.cpp:0e4d489ba70d813d69918172cfaae7293f5ecead : 49 + 0x12]
>     eip = 0xb4c4aa9d   esp = 0xbfa2a9b0   ebp = 0xbfa2aa28   ebx = 0xb7b43ff4
>     esi = 0xbfa2a9c0   edi = 0xacac5000
>     Found by: call frame info

AutoObjectMapper.cpp:0e4d489ba70d813d69918172cfaae7293f5ecead : 186 is:

> size_t sz = __dl_get_mappable_length(hdl);

Comment 1

[Eric Rahm [:erahm]]

Mike, any thoughts on this?

Comment 2

[Mike Hommey [:glandium]]

bug 1648654 removed this code.