Project

General

Profile

Actions
Copy link

Bug #1875

closed

fix possible fd leak on errors

Added by neels over 7 years ago. Updated about 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
neels
Target version:
-
Start date:
12/13/2016
Due date:
% Done:

100%

Spec Reference:

Description

Here is a patch that came from a coverity warning:
https://gerrit.osmocom.org/1320

I've abandoned it because I didn't get around to testing it yet and don't want it to take up space in the gerrit queue all the time.

Resurrect the patch, test it and merge it to libosmo-netif master.
Also attached as patch file.

Files

0001-osmo_stream_srv_fd_cb-don-t-leak-socket-FDs-on-error.patch 0001-osmo_stream_srv_fd_cb-don-t-leak-socket-FDs-on-error.patch 1.8 KB neels, 12/13/2016 12:46 AM
Actions
Copy link
 #1

Updated by neels about 7 years ago

still not finding time for this :/

Actions
Copy link
 #2

Updated by laforge about 7 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

I've reviewed the patch, extended it to some recent changes in the osmo_stream_srv_fd_cb() and merged it now.

Actions
Copy link
 #3

Updated by laforge about 7 years ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)