{"comment":{"url":"https://api.github.com/repos/OpenClovis/SAFplus-Availability-Scalability-Platform/comments/16111435","html_url":"https://github.com/OpenClovis/SAFplus-Availability-Scalability-Platform/commit/e7fbcb801f29eaa24feca467cc58af0224921df4#commitcomment-16111435","id":16111435,"user":{"login":"hoangle","id":1438611,"avatar_url":"https://avatars.githubusercontent.com/u/1438611?v=3","gravatar_id":"","url":"https://api.github.com/users/hoangle","html_url":"https://github.com/hoangle","followers_url":"https://api.github.com/users/hoangle/followers","following_url":"https://api.github.com/users/hoangle/following{/other_user}","gists_url":"https://api.github.com/users/hoangle/gists{/gist_id}","starred_url":"https://api.github.com/users/hoangle/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hoangle/subscriptions","organizations_url":"https://api.github.com/users/hoangle/orgs","repos_url":"https://api.github.com/users/hoangle/repos","events_url":"https://api.github.com/users/hoangle/events{/privacy}","received_events_url":"https://api.github.com/users/hoangle/received_events","type":"User","site_admin":false},"position":34,"line":1044,"path":"src/SAFplus/components/msg/client/clMsgApi.c","commit_id":"e7fbcb801f29eaa24feca467cc58af0224921df4","created_at":"2016-02-16T08:14:37Z","updated_at":"2016-02-16T08:14:37Z","body":"This is wrong since currentMessageId not protected out of scope mutex lock.\r\nIt should be:\r\nClUint32T temp;\r\npthread_mutex_lock(&mutexMsgId);\r\ntemp = ++currentMessageId;\r\npthread_mutex_unlock(&mutexMsgId);\r\nreturn temp;"}}