{"action":"created","issue":{"url":"https://api.github.com/repos/BOINC/boinc/issues/1721","repository_url":"https://api.github.com/repos/BOINC/boinc","labels_url":"https://api.github.com/repos/BOINC/boinc/issues/1721/labels{/name}","comments_url":"https://api.github.com/repos/BOINC/boinc/issues/1721/comments","events_url":"https://api.github.com/repos/BOINC/boinc/issues/1721/events","html_url":"https://github.com/BOINC/boinc/issues/1721","id":191759973,"number":1721,"title":"Remove unnecessary null pointer checks","user":{"login":"elfring","id":660477,"avatar_url":"https://avatars.githubusercontent.com/u/660477?v=3","gravatar_id":"","url":"https://api.github.com/users/elfring","html_url":"https://github.com/elfring","followers_url":"https://api.github.com/users/elfring/followers","following_url":"https://api.github.com/users/elfring/following{/other_user}","gists_url":"https://api.github.com/users/elfring/gists{/gist_id}","starred_url":"https://api.github.com/users/elfring/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/elfring/subscriptions","organizations_url":"https://api.github.com/users/elfring/orgs","repos_url":"https://api.github.com/users/elfring/repos","events_url":"https://api.github.com/users/elfring/events{/privacy}","received_events_url":"https://api.github.com/users/elfring/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2016-11-25T20:03:43Z","updated_at":"2016-12-08T00:08:47Z","closed_at":"2016-12-08T00:08:47Z","body":"[An extra null pointer check is not needed in functions](http://pubs.opengroup.org/onlinepubs/9699919799/functions/free.html \"Do I need to check for null before free p?\") like the following.\r\n- [MFILE](https://github.com/BOINC/boinc/blob/a44f7ad49d47bdc66647e34bb678ba672fe1289f/lib/mfile.cpp#L45 \"Destructor for MFILE\")\r\n- [SCHEDULER_REPLY](https://github.com/BOINC/boinc/blob/5c36ce25daae8532f44d81e9f4c5527bde5e0e51/client/scheduler_op.cpp#L566 \"Destructor for SCHEDULER_REPLY\")\r\n- [STARFIELD::build_stars](https://github.com/BOINC/boinc/blob/365ca7960f751a501fc195670abd0acfc160f8d3/api/gutil.cpp#L466 \"STARFIELD::build_stars function\")"},"comment":{"url":"https://api.github.com/repos/BOINC/boinc/issues/comments/265614020","html_url":"https://github.com/BOINC/boinc/issues/1721#issuecomment-265614020","issue_url":"https://api.github.com/repos/BOINC/boinc/issues/1721","id":265614020,"user":{"login":"davidpanderson","id":985729,"avatar_url":"https://avatars.githubusercontent.com/u/985729?v=3","gravatar_id":"","url":"https://api.github.com/users/davidpanderson","html_url":"https://github.com/davidpanderson","followers_url":"https://api.github.com/users/davidpanderson/followers","following_url":"https://api.github.com/users/davidpanderson/following{/other_user}","gists_url":"https://api.github.com/users/davidpanderson/gists{/gist_id}","starred_url":"https://api.github.com/users/davidpanderson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/davidpanderson/subscriptions","organizations_url":"https://api.github.com/users/davidpanderson/orgs","repos_url":"https://api.github.com/users/davidpanderson/repos","events_url":"https://api.github.com/users/davidpanderson/events{/privacy}","received_events_url":"https://api.github.com/users/davidpanderson/received_events","type":"User","site_admin":false},"created_at":"2016-12-08T00:08:47Z","updated_at":"2016-12-08T00:08:47Z","body":"Let's not worry about this.  A bigger concern is making the client behave deterministically (e.g. log an error message, wait 2 seconds for the GUI to get it, then exit) on a memory alloc failure.  This could be an issue on Android."}}
{"actor":{"display_login":"davidpanderson"}}