Bug 17427 – std.concurrency internal errors on uninitialised mailbox
Status
RESOLVED
Resolution
FIXED
Severity
normal
Priority
P1
Component
phobos
Product
D
Version
D2
Platform
All
OS
All
Creation time
2017-05-24T17:47:33Z
Last change time
2019-12-30T08:48:43Z
Keywords
pull
Assigned to
No Owner
Creator
John Colvin
Comments
Comment #0 by john.loughran.colvin — 2017-05-24T17:47:33Z
there are a bunch of functions in std.concurrency that take Tid as a parameter, none(?) of which check the validity of that Tid (specifically the Mailbox inside), resulting in mysterious segfaults in private functions if a user accidentally calls those functions with an uninitialised Tid.
assert(tid.mbox !is null);
in the implementation (or ideally contracts) of these public API functions would help.
Comment #1 by dlang-bot — 2019-12-14T10:11:16Z
@berni44 created dlang/phobos pull request #7319 "Fix Issue 17427 - std.concurrency internal errors on uninitialised mailbox" fixing this issue:
- Fix Issue 17427 - std.concurrency internal errors on uninitialised mailbox
https://github.com/dlang/phobos/pull/7319
Comment #2 by dlang-bot — 2019-12-30T08:48:43Z
dlang/phobos pull request #7319 "Fix Issue 17427 - std.concurrency internal errors on uninitialised mailbox" was merged into master:
- 63a7f855e075f0f7c46040ebd92a9fe3c403d24b by Bernhard Seckinger:
Fix Issue 17427 - std.concurrency internal errors on uninitialised mailbox
https://github.com/dlang/phobos/pull/7319