Coverity issues #76
Coverity issues #76
Conversation
|
|
Coverity Issue - Resource leakHandle variable "connection" going out of scope leaks the handle. High Impact, CWE-404 Issue locationThis issue was discovered outside the diff for this Pull Request. You can find it at: |
Coverity Issue - Unchecked return value from libraryCalling "send(connection, response.c_str(), response.size(), 0)" without checking return value. This library function may fail and return an error code. Medium Impact, CWE-252 Issue locationThis issue was discovered outside the diff for this Pull Request. You can find it at: |
Coverity Issue - String not null terminatedPassing unterminated string "buffer" to "basic_string", which expects a null-terminated string. [Note: The source code implementation of the function has been overridden by a builtin model.] High Impact, CWE-170 Issue locationThis issue was discovered outside the diff for this Pull Request. You can find it at: |
Coverity Issue - Resource leakHandle variable "sockfd" going out of scope leaks the handle. High Impact, CWE-404 Issue locationThis issue was discovered outside the diff for this Pull Request. You can find it at: |
| EVENT_SUBSCRIBE(0, _T("uploadComplete"), _T("org.rdk.ScreenCapture"), message); | ||
| EVENT_SUBSCRIBE_1(0, _T("uploadComplete"), _T("org.rdk.ScreenCapture"), message); | ||
|
|
||
| EXPECT_EQ(Core::ERROR_NONE, handler.Invoke(connection, _T("uploadScreenCapture"), _T("{\"url\":\"http://127.0.0.1:11111\"}"), response)); |
There was a problem hiding this comment.
Coverity issue no longer present as of: undefined
Show issue
Coverity Issue - Resource leak
Handle variable "connection" going out of scope leaks the handle.
High Impact, CWE-404
RESOURCE_LEAK
No description provided.