← Back to index | Original Bugzilla link

Bug 14135 – std.uuid.randomUUID breaks @safety

Status
RESOLVED
Resolution
FIXED
Severity
major
Priority
P1
Component
dlang.org
Product
D
Version
D2
Platform
x86_64
OS
All
Creation time
2015-02-06T23:24:00Z
Last change time
2017-07-19T17:42:31Z
Keywords
safe
Assigned to
monarchdodra
Creator
code

Comments

Comment #0 by code — 2015-02-06T23:24:50Z
The @trusted UUID randomUUID(RNG)(ref RNG randomGen) if(isIntegral!(typeof(RNG.front))) overload is incorrectly marked as @trusted, as the user might pass a custom RNG with @system range primitives. Seems like it was introduced in https://github.com/D-Programming-Language/phobos/pull/1708.
Comment #1 by code — 2015-02-06T23:29:46Z
Source permalink for your convenience: https://github.com/D-Programming-Language/phobos/blob/041db2d8fac2b1cf5169be7188ea537a01f27586/std/uuid.d#L1061-L1086
Comment #2 by monarchdodra — 2015-02-09T13:03:41Z
https://github.com/D-Programming-Language/phobos/pull/2971
Comment #3 by github-bugzilla — 2015-03-12T12:32:35Z
Commit pushed to master at https://github.com/D-Programming-Language/phobos https://github.com/D-Programming-Language/phobos/commit/9010ecd4b0942e0eb0c37c073d4d53f1984f7872 Merge pull request #2971 from monarchdodra/14135 Issue 14135 - std.uuid.randomUUID breaks @safety
Comment #4 by github-bugzilla — 2017-07-19T17:42:31Z
Commit pushed to dmd-cxx at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/9010ecd4b0942e0eb0c37c073d4d53f1984f7872 Merge pull request #2971 from monarchdodra/14135