You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@mesos.apache.org by "Joseph Wu (JIRA)" <ji...@apache.org> on 2016/09/19 20:46:20 UTC
[jira] [Updated] (MESOS-6152) Resource leak in
libevent_ssl_socket.cpp.
[ https://issues.apache.org/jira/browse/MESOS-6152?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Joseph Wu updated MESOS-6152:
-----------------------------
Fix Version/s: 1.0.2
0.27.4
0.28.3
> Resource leak in libevent_ssl_socket.cpp.
> -----------------------------------------
>
> Key: MESOS-6152
> URL: https://issues.apache.org/jira/browse/MESOS-6152
> Project: Mesos
> Issue Type: Bug
> Reporter: Joerg Schad
> Assignee: Benjamin Bannier
> Labels: coverity
> Fix For: 0.28.3, 0.27.4, 1.1.0, 1.0.2
>
>
> Coverity detected the following resource leak.
> IMO {code} if (fd == -1) {code} should be {code} if (owned_fd == -1) {code}.
> {code}
> // Duplicate the file descriptor because Libevent will take ownership
> 754 // and control the lifecycle separately.
> 755 //
> 756 // TODO(josephw): We can avoid duplicating the file descriptor in
> 757 // future versions of Libevent. In Libevent versions 2.1.2 and later,
> 758 // we may use `evbuffer_file_segment_new` and `evbuffer_add_file_segment`
> 759 // instead of `evbuffer_add_file`.
> 3. open_fn: Returning handle opened by dup.
> 4. var_assign: Assigning: owned_fd = handle returned from dup(fd).
> 760 int owned_fd = dup(fd);
> CID 1372873: Argument cannot be negative (REVERSE_NEGATIVE) [select issue]
> 5. Condition fd == -1, taking true branch.
> 761 if (fd == -1) {
>
> CID 1372872 (#1 of 1): Resource leak (RESOURCE_LEAK)
> 6. leaked_handle: Handle variable owned_fd going out of scope leaks the handle.
> 762 return Failure(ErrnoError("Failed to duplicate file descriptor"));
> 763 }
> {code}
> https://scan5.coverity.com/reports.htm#v39597/p10429/fileInstanceId=98881747&defectInstanceId=28450468
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)