Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Coverity Scan (CID 1352361) [/plugins/http_renderer/src/httprsrv.c:srv_accept_socket] : Resource leak #229

Closed
tizonia opened this issue Mar 6, 2016 · 0 comments
Labels

Comments

@tizonia
Copy link
Collaborator

@tizonia tizonia commented Mar 6, 2016

  • CID : 1352361
  • Type : Resource leak
  • Impact : High
  • Status : New
  • First Detected : 02/29/16
  • Classification : Unclassified
  • Severity : Unspecified
  • Component : http_renderer
  • Category : Resource leaks
  • File : /plugins/http_renderer/src/httprsrv.c
  • Function : srv_accept_socket
339  bail_on_accept_error (some_error, strerror (errno));
340
341  TIZ_TRACE (p_hdl, "Accepted [%s:%u] fd [%d]", ap_ip, *ap_port,
342             accepted_sockfd);
343end:
    23. Condition some_error, taking true branch
344  if (some_error)
345    {

CID 1352361 (#1-2 of 2): Resource leak (RESOURCE_LEAK)
24. overwrite_var: Overwriting handle accepted_sockfd in accepted_sockfd = -1 leaks the handle.
346      accepted_sockfd = ICE_SOCK_ERROR;
347    }
348
349  return accepted_sockfd;
350}

@tizonia tizonia added the bug label Mar 6, 2016
@juanrubio juanrubio closed this in 18a6300 Mar 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
0 participants