Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

epoch: possible reads of uninitialized data #347

Closed
ghost opened this issue Mar 22, 2019 · 1 comment
Closed

epoch: possible reads of uninitialized data #347

ghost opened this issue Mar 22, 2019 · 1 comment
Labels
bug crossbeam-epoch

Comments

@ghost
Copy link

ghost commented Mar 22, 2019

Reported in servo/servo#23077

==130127== Thread 43 WRRenderBackend#:
==130127== Conditional jump or move depends on uninitialised value(s)
==130127==    at 0x95D09AA: crossbeam_epoch::internal::Global::collect (internal.rs:189)
==130127==    by 0x957E156: crossbeam_epoch::internal::Local::pin (internal.rs:379)
==130127==    by 0x9551625: crossbeam_epoch::collector::LocalHandle::pin (collector.rs:72)
==130127==    by 0x95550E7: crossbeam_epoch::default::pin::{{closure}} (default.rs:23)
==130127==    by 0x9555097: crossbeam_epoch::default::with_handle::{{closure}} (default.rs:42)
==130127==    by 0x955668D: <std::thread::local::LocalKey<T>>::try_with (local.rs:300)
==130127==    by 0x9554F72: crossbeam_epoch::default::with_handle (default.rs:42)
==130127==    by 0x63530B9: crossbeam_epoch::default::pin (default.rs:23)
==130127==    by 0x634717E: <crossbeam_channel::flavors::list::Channel<T>>::start_send (list.rs:161)
==130127==    by 0x6347DB8: <crossbeam_channel::flavors::list::Channel<T>>::send (list.rs:381)
==130127==    by 0x634E381: <crossbeam_channel::channel::Sender<T>>::send (channel.rs:416)
==130127==    by 0x634F347: compositing::compositor_thread::CompositorProxy::send (compositor_thread.rs:31)
==130127== 
==130127== Conditional jump or move depends on uninitialised value(s)
==130127==    at 0x95D4A2F: core::ptr::real_drop_in_place (ptr.rs:193)
==130127==    by 0x95D09D0: crossbeam_epoch::internal::Global::collect (internal.rs:192)
==130127==    by 0x957E156: crossbeam_epoch::internal::Local::pin (internal.rs:379)
==130127==    by 0x9551625: crossbeam_epoch::collector::LocalHandle::pin (collector.rs:72)
==130127==    by 0x95550E7: crossbeam_epoch::default::pin::{{closure}} (default.rs:23)
==130127==    by 0x9555097: crossbeam_epoch::default::with_handle::{{closure}} (default.rs:42)
==130127==    by 0x955668D: <std::thread::local::LocalKey<T>>::try_with (local.rs:300)
==130127==    by 0x9554F72: crossbeam_epoch::default::with_handle (default.rs:42)
==130127==    by 0x63530B9: crossbeam_epoch::default::pin (default.rs:23)
==130127==    by 0x634717E: <crossbeam_channel::flavors::list::Channel<T>>::start_send (list.rs:161)
==130127==    by 0x6347DB8: <crossbeam_channel::flavors::list::Channel<T>>::send (list.rs:381)
==130127==    by 0x634E381: <crossbeam_channel::channel::Sender<T>>::send (channel.rs:416)
@jonhoo jonhoo mentioned this issue Feb 17, 2020
Merged
bors bot added a commit that referenced this issue Feb 17, 2020
@bors @jonhoo
Merge #474
556ebbc 
474: Patch release for all crossbeam crates. r=jeehoonkang a=jonhoo

This bumps the patch version for all sub-crates and the top-level crate and updates the changelogs with references to PRs where applicable. As far as I can tell, there are no breaking changes. The top-level crate has not change at all, though I bumped `rand` to `0.7` in all the crates, which I guess technically requires a patch bump if we do another release.

Fixes #473.
Closes #472.
Closes #468.
Closes #409.
Not sure if this also solves #347 by virtue of #458?

Co-authored-by: Jon Gjengset <jon@thesquareplanet.com>
@jonhoo jonhoo mentioned this issue Feb 17, 2020
Open
bors bot added a commit that referenced this issue Feb 17, 2020
@bors @jonhoo
Merge #474
0210b05 
474: Patch release for all crossbeam crates. r=jeehoonkang a=jonhoo

This bumps the patch version for all sub-crates and the top-level crate and updates the changelogs with references to PRs where applicable. As far as I can tell, there are no breaking changes. The top-level crate has not change at all, though I bumped `rand` to `0.7` in all the crates, which I guess technically requires a patch bump if we do another release.

Fixes #473.
Closes #472.
Closes #468.
Closes #409.
Not sure if this also solves #347 by virtue of #458?

Co-authored-by: Jon Gjengset <jon@thesquareplanet.com>
bors bot added a commit that referenced this issue Feb 22, 2020
@bors @jonhoo
Merge #474
6d6591a 
474: Patch release for all crossbeam crates. r=jeehoonkang a=jonhoo

This bumps the patch version for all sub-crates and the top-level crate and updates the changelogs with references to PRs where applicable. As far as I can tell, there are no breaking changes. The top-level crate has not change at all, though I bumped `rand` to `0.7` in all the crates, which I guess technically requires a patch bump if we do another release.

Fixes #473.
Closes #472.
Closes #468.
Closes #409.
Not sure if this also solves #347 by virtue of #458?

Co-authored-by: Jon Gjengset <jon@thesquareplanet.com>
@taiki-e
Copy link
Member

taiki-e commented Feb 17, 2022

Closing - There is no way to reproduce the original problem with the latest crossbeam, and it is very likely that the relevant code in epoch has been fixed in #458.

@taiki-e taiki-e closed this as completed Feb 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug crossbeam-epoch
Milestone
No milestone
Development

No branches or pull requests
1 participant
@taiki-e