{"action":"closed","issue":{"url":"https://api.github.com/repos/MDSplus/mdsplus/issues/471","repository_url":"https://api.github.com/repos/MDSplus/mdsplus","labels_url":"https://api.github.com/repos/MDSplus/mdsplus/issues/471/labels{/name}","comments_url":"https://api.github.com/repos/MDSplus/mdsplus/issues/471/comments","events_url":"https://api.github.com/repos/MDSplus/mdsplus/issues/471/events","html_url":"https://github.com/MDSplus/mdsplus/issues/471","id":151712475,"number":471,"title":"servershr/ServerSendMessage.c: Uninitialized var used","user":{"login":"kgerickson","id":14148090,"avatar_url":"https://avatars.githubusercontent.com/u/14148090?v=3","gravatar_id":"","url":"https://api.github.com/users/kgerickson","html_url":"https://github.com/kgerickson","followers_url":"https://api.github.com/users/kgerickson/followers","following_url":"https://api.github.com/users/kgerickson/following{/other_user}","gists_url":"https://api.github.com/users/kgerickson/gists{/gist_id}","starred_url":"https://api.github.com/users/kgerickson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kgerickson/subscriptions","organizations_url":"https://api.github.com/users/kgerickson/orgs","repos_url":"https://api.github.com/users/kgerickson/repos","events_url":"https://api.github.com/users/kgerickson/events{/privacy}","received_events_url":"https://api.github.com/users/kgerickson/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/MDSplus/mdsplus/labels/bug","name":"bug","color":"fc2929"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-04-28T18:32:52Z","updated_at":"2016-04-29T18:49:30Z","closed_at":"2016-04-29T18:49:29Z","body":"In servershr/ServerSendMessage.c, ServerSendMessage(), jobid is declared at line 144 and used on line 173 in an sprintf command whether or not it has been initialized on line 170.  It is conditionally set based on the value of addr.  jobid should be defined to be an invalid value so that in the case where addr is not set, the sprintf yields defined results.  Further, better handling is needed in the case that addr is zero."}}