{"action":"created","issue":{"url":"https://api.github.com/repos/rtrlib/rtrlib/issues/98","repository_url":"https://api.github.com/repos/rtrlib/rtrlib","labels_url":"https://api.github.com/repos/rtrlib/rtrlib/issues/98/labels{/name}","comments_url":"https://api.github.com/repos/rtrlib/rtrlib/issues/98/comments","events_url":"https://api.github.com/repos/rtrlib/rtrlib/issues/98/events","html_url":"https://github.com/rtrlib/rtrlib/pull/98","id":185910162,"number":98,"title":"rtrlib/rtr: add range checks for interval values in packets.c","user":{"login":"ColinBS","id":1913262,"avatar_url":"https://avatars.githubusercontent.com/u/1913262?v=3","gravatar_id":"","url":"https://api.github.com/users/ColinBS","html_url":"https://github.com/ColinBS","followers_url":"https://api.github.com/users/ColinBS/followers","following_url":"https://api.github.com/users/ColinBS/following{/other_user}","gists_url":"https://api.github.com/users/ColinBS/gists{/gist_id}","starred_url":"https://api.github.com/users/ColinBS/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ColinBS/subscriptions","organizations_url":"https://api.github.com/users/ColinBS/orgs","repos_url":"https://api.github.com/users/ColinBS/repos","events_url":"https://api.github.com/users/ColinBS/events{/privacy}","received_events_url":"https://api.github.com/users/ColinBS/received_events","type":"User","site_admin":false},"labels":[{"id":459438505,"url":"https://api.github.com/repos/rtrlib/rtrlib/labels/WIP","name":"WIP","color":"fbca04","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-10-28T12:22:47Z","updated_at":"2016-11-01T19:46:46Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/rtrlib/rtrlib/pulls/98","html_url":"https://github.com/rtrlib/rtrlib/pull/98","diff_url":"https://github.com/rtrlib/rtrlib/pull/98.diff","patch_url":"https://github.com/rtrlib/rtrlib/pull/98.patch"},"body":"Closes issue #81\nUses same if-statements used in rtr.c\nThere still are return values for failed checks needed, suggestions are welcome.\n"},"comment":{"url":"https://api.github.com/repos/rtrlib/rtrlib/issues/comments/257673834","html_url":"https://github.com/rtrlib/rtrlib/pull/98#issuecomment-257673834","issue_url":"https://api.github.com/repos/rtrlib/rtrlib/issues/98","id":257673834,"user":{"login":"smlng","id":10532675,"avatar_url":"https://avatars.githubusercontent.com/u/10532675?v=3","gravatar_id":"","url":"https://api.github.com/users/smlng","html_url":"https://github.com/smlng","followers_url":"https://api.github.com/users/smlng/followers","following_url":"https://api.github.com/users/smlng/following{/other_user}","gists_url":"https://api.github.com/users/smlng/gists{/gist_id}","starred_url":"https://api.github.com/users/smlng/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/smlng/subscriptions","organizations_url":"https://api.github.com/users/smlng/orgs","repos_url":"https://api.github.com/users/smlng/repos","events_url":"https://api.github.com/users/smlng/events{/privacy}","received_events_url":"https://api.github.com/users/smlng/received_events","type":"User","site_admin":false},"created_at":"2016-11-01T19:46:46Z","updated_at":"2016-11-01T19:46:46Z","body":"@ColinBS, I agree with @waehlisch, before we merge we need _real_ error handling. Further, though haven't had the time for a careful review, I think those values have to be passed through `ntohl` see [here](https://github.com/ColinBS/rtrlib/blob/a77d9a6263e551829cec9bf936fc55601b203611/rtrlib/rtr/packets.c#L235)."}}
{"actor":{"display_login":"smlng"}}