{"action":"created","issue":{"url":"https://api.github.com/repos/dotnet/corefx/issues/30748","repository_url":"https://api.github.com/repos/dotnet/corefx","labels_url":"https://api.github.com/repos/dotnet/corefx/issues/30748/labels{/name}","comments_url":"https://api.github.com/repos/dotnet/corefx/issues/30748/comments","events_url":"https://api.github.com/repos/dotnet/corefx/issues/30748/events","html_url":"https://github.com/dotnet/corefx/issues/30748","id":336841050,"node_id":"MDU6SXNzdWUzMzY4NDEwNTA=","number":30748,"title":"Marvin.ComputeHash has an invalid (unaligned) cast","user":{"login":"GrabYourPitchforks","id":1746272,"node_id":"MDQ6VXNlcjE3NDYyNzI=","avatar_url":"https://avatars0.githubusercontent.com/u/1746272?v=4","gravatar_id":"","url":"https://api.github.com/users/GrabYourPitchforks","html_url":"https://github.com/GrabYourPitchforks","followers_url":"https://api.github.com/users/GrabYourPitchforks/followers","following_url":"https://api.github.com/users/GrabYourPitchforks/following{/other_user}","gists_url":"https://api.github.com/users/GrabYourPitchforks/gists{/gist_id}","starred_url":"https://api.github.com/users/GrabYourPitchforks/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/GrabYourPitchforks/subscriptions","organizations_url":"https://api.github.com/users/GrabYourPitchforks/orgs","repos_url":"https://api.github.com/users/GrabYourPitchforks/repos","events_url":"https://api.github.com/users/GrabYourPitchforks/events{/privacy}","received_events_url":"https://api.github.com/users/GrabYourPitchforks/received_events","type":"User","site_admin":false},"labels":[{"id":158885797,"node_id":"MDU6TGFiZWwxNTg4ODU3OTc=","url":"https://api.github.com/repos/dotnet/corefx/labels/area-Meta","name":"area-Meta","color":"d4c5f9","default":false},{"id":145399955,"node_id":"MDU6TGFiZWwxNDUzOTk5NTU=","url":"https://api.github.com/repos/dotnet/corefx/labels/bug","name":"bug","color":"207de5","default":true}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-06-29T02:26:39Z","updated_at":"2018-07-19T16:54:03Z","closed_at":null,"author_association":"MEMBER","body":"https://github.com/dotnet/corefx/blob/540a99ce273265ba62a25b48c696d2d506c88ab4/src/Common/src/System/Marvin.cs#L34\r\n\r\nIt is invalid to reinterpret cast a `ref byte` to a `ref uint` without first checking the alignment of the input buffer. This could cause problems on future architectures or IOT devices where unaligned reads incur faults. This code should pin and check alignment or should use `Unsafe.ReadUnaligned` instead."},"comment":{"url":"https://api.github.com/repos/dotnet/corefx/issues/comments/406344447","html_url":"https://github.com/dotnet/corefx/issues/30748#issuecomment-406344447","issue_url":"https://api.github.com/repos/dotnet/corefx/issues/30748","id":406344447,"node_id":"MDEyOklzc3VlQ29tbWVudDQwNjM0NDQ0Nw==","user":{"login":"danmosemsft","id":6385855,"node_id":"MDQ6VXNlcjYzODU4NTU=","avatar_url":"https://avatars3.githubusercontent.com/u/6385855?v=4","gravatar_id":"","url":"https://api.github.com/users/danmosemsft","html_url":"https://github.com/danmosemsft","followers_url":"https://api.github.com/users/danmosemsft/followers","following_url":"https://api.github.com/users/danmosemsft/following{/other_user}","gists_url":"https://api.github.com/users/danmosemsft/gists{/gist_id}","starred_url":"https://api.github.com/users/danmosemsft/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/danmosemsft/subscriptions","organizations_url":"https://api.github.com/users/danmosemsft/orgs","repos_url":"https://api.github.com/users/danmosemsft/repos","events_url":"https://api.github.com/users/danmosemsft/events{/privacy}","received_events_url":"https://api.github.com/users/danmosemsft/received_events","type":"User","site_admin":false},"created_at":"2018-07-19T16:54:03Z","updated_at":"2018-07-19T16:54:03Z","author_association":"MEMBER","body":"The Corefx version has `Marvin.ComputeHash()` which returns a `long`, the Corelib version does not. Although perhaps it can be removed as it is only used by tests."}}
{"actor":{"display_login":"danmosemsft"}}