Closed
Bug 1154416
Opened 10 years ago
Closed 10 years ago
Attempt to pthread close an invalid file descriptor found during stability test
Categories
(Firefox OS Graveyard :: Stability, defect)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 1154411
People
(Reporter: ggrisco, Assigned: sotaro, NeedInfo)
References
Details
(Keywords: crash, Whiteboard: [b2g-crash][caf-crash 621][caf priority: p1][CR 822151])
Crash Data
Attachments
(6 files)
CAF debug code signals an abort when an attempt to close an invalid file descriptor is made. This instance is being found during stability test.
cafbot will upload logs.
Comment 1•10 years ago
|
||
Comment 2•10 years ago
|
||
Updated•10 years ago
|
Whiteboard: [CR 822151]
Updated•10 years ago
|
Whiteboard: [CR 822151] → [caf priority: p1][CR 822151]
Comment 3•10 years ago
|
||
fabrice/sotaro, need some help on initial investigation here as well. Similar to :https://bugzilla.mozilla.org/show_bug.cgi?id=1154411#c6
Updated•10 years ago
|
Flags: needinfo?(sotaro.ikeda.g)
Flags: needinfo?(fabrice.desre)
Comment 4•10 years ago
|
||
Hi! Rex,
Please help to take a look. Thanks
--
Keven
Flags: needinfo?(rhung)
Comment 5•10 years ago
|
||
As shown in the following, closing an invalid FD triggers calling abort(). However, the reason why b2g process should be aborted for this symptom is still under checking because I cannot find noselect.c from msm8909 source tree for now. Will update this part later.
// 0be37360-0a8f-c058-60e4ff60-37e054d7.extra
740 04-14 14:48:34.538 18661 18661 E libnoselect: -1 = close(63) failed. errno=9
741 04-14 14:48:34.538 18661 18661 E Gecko : mozalloc_abort: Redirecting call to abort() to mozalloc_abort
// 0be37360-0a8f-c058-60e4ff60-37e054d7.txt
10 Thread 0 (crashed)
11 0 plugin-container!mozalloc_abort [mozalloc_abort.cpp : 37 + 0x4]
12 r0 = 0x00000046 r1 = 0xbe9b6598 r2 = 0x00000025 r3 = 0x00000000
13 r4 = 0x00000009 r5 = 0xffffffff r6 = 0xb6fe8e38 r7 = 0xb29205b8
14 r8 = 0x000014fe r9 = 0x00053e26 r10 = 0xbe9b6af0 r12 = 0xb29205b8
15 fp = 0x00000000 sp = 0xbe9b6a08 lr = 0xb6feadb3 pc = 0xb6feadb6
16 Found by: given as instruction pointer in context
17 1 libnoselect.so!close [noselect.c : 89 + 0x3]
^^^^^^^^^^ Should check here to know why an abortion should be triggered.
18 r3 = 0x2dc8b506 r4 = 0x00000009 r5 = 0xffffffff r6 = 0xb6fe8e38
19 r7 = 0xb29205b8 r8 = 0x000014fe r9 = 0x00053e26 r10 = 0xbe9b6af0
20 fp = 0x00000000 sp = 0xbe9b6a10 pc = 0xb6fd0667
21 Found by: call frame info
22 2 libmozglue.so!__wrap_close [Nuwa.cpp : 1360 + 0x3]
23 r0 = 0x0000003f r1 = 0x00000009 r4 = 0xb1a7cc20 r5 = 0x0000003f
24 r6 = 0x0000003f r7 = 0xb29205b8 r8 = 0x000014fe r9 = 0x00053e26
25 r10 = 0xbe9b6af0 fp = 0x00000000 sp = 0xbe9b6a28 pc = 0xb6f97ce1
Comment 6•10 years ago
|
||
(In reply to Rex Hung[:rhung] from comment #5)
> As shown in the following, closing an invalid FD triggers calling abort().
> However, the reason why b2g process should be aborted for this symptom is
^^^^^^^^^^^
> still under checking because I cannot find noselect.c from msm8909 source
> tree for now. Will update this part later.
Correct my comment at comment 5: It should a pre-allocated app, instead of b2g process.
Flags: needinfo?(rhung)
Assignee | ||
Comment 7•10 years ago
|
||
From the crash log, it seems dup of Bug 1154411.
Flags: needinfo?(sotaro.ikeda.g)
Reporter | ||
Comment 8•10 years ago
|
||
The libnoselect library is added in CAF internal builds to intercept calls to close() and check for invalid file descriptor, aborting if such a case is detected. This bug and bug 1154411 are two different instances where this is being caught (probably from different sources).
Updated•10 years ago
|
Whiteboard: [caf priority: p1][CR 822151] → [b2g-crash][caf-crash 621][caf priority: p1][CR 822151]
Comment 9•10 years ago
|
||
Observed on:
Device: msm8909
Gonk Version: AU_LINUX_GECKO_LF.BR.1.2.3.00.00.00.000.129
Moz BuildID: 20150409002503
Manifest: https://www.codeaurora.org/cgit/quic/lf/b2g/manifest/tree/caf_AU_LINUX_GECKO_LF.BR.1.2.3.00.00.00.000.129.xml?h=release
Gecko Version: 37.0
Gaia: http://git.mozilla.org/?p=releases/gaia.git;a=commit;h=ea735c21bfb0d78333213ff0376fce1eac89ead6
Gecko: http://git.mozilla.org/?p=releases/gecko.git;a=commit;h=ac12d858a58574b2db482b5eed61750a3e210040
Patches: bug 1150923, bug 1154439, bug 1133147, bug 1142806, bug 1150924
Comment 10•10 years ago
|
||
Comment 11•10 years ago
|
||
Comment 12•10 years ago
|
||
Observed on:
Device: msm8909
Gonk Version: AU_LINUX_GECKO_LF.BR.1.2.3.00.00.00.000.132
Moz BuildID: 20150413002502
Manifest: https://www.codeaurora.org/cgit/quic/lf/b2g/manifest/tree/caf_AU_LINUX_GECKO_LF.BR.1.2.3.00.00.00.000.132.xml?h=release
Gecko Version: 37.0
Gaia: http://git.mozilla.org/?p=releases/gaia.git;a=commit;h=cec00d643f517ffd96cde559cd3bbd43ab85816c
Gecko: http://git.mozilla.org/?p=releases/gecko.git;a=commit;h=a47c1908cca4b834ae92e9a0507f09da46993f8b
Patches: bug 1150923, bug 1154439, bug 1133147, bug 1142806, bug 1150924
Comment 13•10 years ago
|
||
Comment 14•10 years ago
|
||
Reporter | ||
Comment 15•10 years ago
|
||
(In reply to Sotaro Ikeda [:sotaro] from comment #7)
> From the crash log, it seems dup of Bug 1154411.
As mentioned in comment 8, this crash could be different root cause. Is there any evidence that the patch for bug 1154411 will fix this one too? If not, I hope there can still be analysis done on this issue.
Flags: needinfo?(sotaro.ikeda.g)
Assignee | ||
Comment 16•10 years ago
|
||
(In reply to Greg Grisco from comment #15)
> (In reply to Sotaro Ikeda [:sotaro] from comment #7)
> > From the crash log, it seems dup of Bug 1154411.
>
> As mentioned in comment 8, this crash could be different root cause. Is
> there any evidence that the patch for bug 1154411 will fix this one too? If
> not, I hope there can still be analysis done on this issue.
All crash have the following in the stack it is same to Bug 1154411 and attachment 8593613 [details] [diff] [review] is a fix to ArrayBufferBuilder::mapToFileInPackage().
> libxul.so!mozilla::ArrayBufferBuilder::mapToFileInPackage
Flags: needinfo?(sotaro.ikeda.g)
Comment 17•10 years ago
|
||
as comment 16, make duplication.
Assignee: nobody → sotaro.ikeda.g
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → DUPLICATE
Updated•10 years ago
|
No longer blocks: CAF-v2.2-metabug
You need to log in
before you can comment on or make changes to this bug.
Description
•