You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "James E. King, III (JIRA)" <ji...@apache.org> on 2016/10/06 11:58:22 UTC

[jira] [Updated] (THRIFT-3943) Coverity Scan identified some high severity defects

     [ https://issues.apache.org/jira/browse/THRIFT-3943?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

James E. King, III updated THRIFT-3943:
---------------------------------------
    Description: 
Coverity Scan identified 9 issues of high severity.
I dismissed 4 of them as false positives; coverity lost track of the handling of socket file descriptors across multiple layers of calls; this left 5 issues, and I took care of a number of insignificant issues as well:

1295822 - memory leak in ThreadFactoryTests
1216842 - uninitialized rfds fd_set is passed to select if mode is not WAIT_MODE_C (R+W)
1216841 - uninitialized wfds fd_set is passed to select if mode is not WAIT_MODE_C (R+W)
1216840 - getsockname is always passed uninitialized addrlen
1295810 - uninitialized variables in test
1295808 - uninitialized variable in test
1295804 - structurally dead code in processor test event log - changed to use environment variable
excuded:
1174563 - memory leak in compiler class handling functions
1174671 - uninitialized variable in FunctionRunner (intervalMs_)
1174669, 1174763, 1295806, 1295807, 1295809 - uninitialized variable in TSocket (peerPort_)

  was:
Coverity Scan identified 9 issues of high severity.
I dismissed 4 of them as false positives; coverity lost track of the handling of socket file descriptors across multiple layers of calls; this left 5 issues:

1295822 - memory leak in ThreadFactoryTests
1216842 - uninitialized rfds fd_set is passed to select if mode is not WAIT_MODE_C (R+W)
1216841 - uninitialized wfds fd_set is passed to select if mode is not WAIT_MODE_C (R+W)
1216840 - getsockname is always passed uninitialized addrlen
1174563 - memory leak in compiler class handling functions


> Coverity Scan identified some high severity defects
> ---------------------------------------------------
>
>                 Key: THRIFT-3943
>                 URL: https://issues.apache.org/jira/browse/THRIFT-3943
>             Project: Thrift
>          Issue Type: Bug
>          Components: C++ - Library, Lua - Library
>    Affects Versions: 0.9.3
>         Environment: https://scan.coverity.com/projects/thrift
>            Reporter: James E. King, III
>            Assignee: James E. King, III
>            Priority: Critical
>
> Coverity Scan identified 9 issues of high severity.
> I dismissed 4 of them as false positives; coverity lost track of the handling of socket file descriptors across multiple layers of calls; this left 5 issues, and I took care of a number of insignificant issues as well:
> 1295822 - memory leak in ThreadFactoryTests
> 1216842 - uninitialized rfds fd_set is passed to select if mode is not WAIT_MODE_C (R+W)
> 1216841 - uninitialized wfds fd_set is passed to select if mode is not WAIT_MODE_C (R+W)
> 1216840 - getsockname is always passed uninitialized addrlen
> 1295810 - uninitialized variables in test
> 1295808 - uninitialized variable in test
> 1295804 - structurally dead code in processor test event log - changed to use environment variable
> excuded:
> 1174563 - memory leak in compiler class handling functions
> 1174671 - uninitialized variable in FunctionRunner (intervalMs_)
> 1174669, 1174763, 1295806, 1295807, 1295809 - uninitialized variable in TSocket (peerPort_)



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)