1790587 - Crash in [@ core::option::expect_failed | l10nregistry::registry::synchronous::<T>::bundle_from_order]

Status: RESOLVED FIXED

(Core :: Internationalization: Localization, defect)

Description

[merlino37]

Crash report: https://crash-stats.mozilla.org/report/index/d6b23963-8e81-47a0-8d82-a7d080220912

MOZ_CRASH Reason: Index out-of-range

Top 10 frames of crashing thread:

0 libxul.so RustMozCrash mozglue/static/rust/wrappers.cpp:18
1 libxul.so mozglue_static::panic_hook mozglue/static/rust/lib.rs:91
2 libxul.so core::ops::function::Fn::call library/core/src/ops/function.rs:77
3 libxul.so std::panicking::rust_panic_with_hook library/std/src/panicking.rs:702
4 libxul.so std::panicking::begin_panic_handler::{{closure}} library/std/src/panicking.rs:588
5 libxul.so std::sys_common::backtrace::__rust_end_short_backtrace library/std/src/sys_common/backtrace.rs:138
6 libxul.so rust_begin_unwind library/std/src/panicking.rs:584
7 libxul.so core::panicking::panic_fmt library/core/src/panicking.rs:142
8 libxul.so core::panicking::panic_display library/core/src/panicking.rs:72
9 libxul.so core::panicking::panic_str library/core/src/panicking.rs:56

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The bug has a crash signature, thus the bug will be considered confirmed.

Comment 2

[Marco Bonardo [:mak]]

Apparently it's a panic oom crash in l10n registry

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:eemeli, could you have a look please?

For more information, please visit auto_nag documentation.

Comment 4

[Eemeli Aro [:eemeli]]

Erik, could you take a look at this?

Comment 5

[Greg Tatum [:gregtatum]]

I think this is the same metasources bug as Bug 1773733. The reason is still the index panic message.

Comment 6

[Erik Nordin [:nordzilla]]

I suspected as well that this may be the same root issue as Bug 1773733, which I know Greg has already looked into some.

I will continue to collaborate and communicate with Greg in seeking resolution for this, and hopefully it is the same root cause.

Comment 7

[Eemeli Aro [:eemeli]]

Greg, do you know if this has been fixed by the bug 1773733 fix?

Comment 8

[Greg Tatum [:gregtatum]]

Yes, it should be fixed, but I don't have any crash stats data on it yet, since it's Nightly only, and the Beta uplift was denied. I'm going to re-request uplift to Release if it looks stable once it rides to Beta.

Comment 9

[Greg Tatum [:gregtatum]]

Also, this crash has been around since fluent-rs was initially merged, but its crash signature has changed over time making it seem like many different bugs, but it's all because of a core issue of mutating the list of sources while async iterating over them.

Comment 10

[Eemeli Aro [:eemeli]]

Closing as fixed, based on this comment.