{"action":"created","issue":{"url":"https://api.github.com/repos/floodlight/indigo/issues/191","labels_url":"https://api.github.com/repos/floodlight/indigo/issues/191/labels{/name}","comments_url":"https://api.github.com/repos/floodlight/indigo/issues/191/comments","events_url":"https://api.github.com/repos/floodlight/indigo/issues/191/events","html_url":"https://github.com/floodlight/indigo/pull/191","id":59331348,"number":191,"title":"OFConnectionManager: add asserts to silence static analysis warning","user":{"login":"rlane","id":28525,"avatar_url":"https://avatars.githubusercontent.com/u/28525?v=3","gravatar_id":"","url":"https://api.github.com/users/rlane","html_url":"https://github.com/rlane","followers_url":"https://api.github.com/users/rlane/followers","following_url":"https://api.github.com/users/rlane/following{/other_user}","gists_url":"https://api.github.com/users/rlane/gists{/gist_id}","starred_url":"https://api.github.com/users/rlane/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rlane/subscriptions","organizations_url":"https://api.github.com/users/rlane/orgs","repos_url":"https://api.github.com/users/rlane/repos","events_url":"https://api.github.com/users/rlane/events{/privacy}","received_events_url":"https://api.github.com/users/rlane/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-02-28T07:48:15Z","updated_at":"2015-02-28T08:17:29Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/floodlight/indigo/pulls/191","html_url":"https://github.com/floodlight/indigo/pull/191","diff_url":"https://github.com/floodlight/indigo/pull/191.diff","patch_url":"https://github.com/floodlight/indigo/pull/191.patch"},"body":"Reviewer: trivial\r\n\r\nThe cxn_id_to_connection function can usually return null, which would lead to\r\na null dereference in ind_cxn_accepts_async_message. However, we maintain the\r\naux_id_to_cxn_id map so that none of the connection IDs are ever stale, so\r\n*cxn will never be NULL.\r\n\r\nI don't think this needs to be ported to OFConnectionManager2 since it stores\r\npointers in aux_id_to_cxn."},"comment":{"url":"https://api.github.com/repos/floodlight/indigo/issues/comments/76516725","html_url":"https://github.com/floodlight/indigo/pull/191#issuecomment-76516725","issue_url":"https://api.github.com/repos/floodlight/indigo/issues/191","id":76516725,"user":{"login":"bsn-abat","id":1144962,"avatar_url":"https://avatars.githubusercontent.com/u/1144962?v=3","gravatar_id":"","url":"https://api.github.com/users/bsn-abat","html_url":"https://github.com/bsn-abat","followers_url":"https://api.github.com/users/bsn-abat/followers","following_url":"https://api.github.com/users/bsn-abat/following{/other_user}","gists_url":"https://api.github.com/users/bsn-abat/gists{/gist_id}","starred_url":"https://api.github.com/users/bsn-abat/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bsn-abat/subscriptions","organizations_url":"https://api.github.com/users/bsn-abat/orgs","repos_url":"https://api.github.com/users/bsn-abat/repos","events_url":"https://api.github.com/users/bsn-abat/events{/privacy}","received_events_url":"https://api.github.com/users/bsn-abat/received_events","type":"User","site_admin":false},"created_at":"2015-02-28T08:17:29Z","updated_at":"2015-02-28T08:17:29Z","body":"ABAT: START_MERGE:  Log file at http://vbs22/abat/2015.02.28.0817-m.indigo.master/abat.log"}}