diff options
author | Rainer Gerhards <rgerhards@adiscon.com> | 2012-01-09 09:01:44 +0100 |
---|---|---|
committer | Rainer Gerhards <rgerhards@adiscon.com> | 2012-01-09 09:01:44 +0100 |
commit | 5438784b3ff566dc9efd863151ff782e2b9f4e45 (patch) | |
tree | 8621981d7b326368c130e94b9d757930e0f3901b /plugins | |
parent | 6ea4cb7f5383f8c367be810fdc66650eeb9cc787 (diff) | |
download | rsyslog-5438784b3ff566dc9efd863151ff782e2b9f4e45.tar.gz rsyslog-5438784b3ff566dc9efd863151ff782e2b9f4e45.tar.xz rsyslog-5438784b3ff566dc9efd863151ff782e2b9f4e45.zip |
imudp refactor: fixed socket leak
Diffstat (limited to 'plugins')
-rw-r--r-- | plugins/imudp/imudp.c | 14 |
1 files changed, 5 insertions, 9 deletions
diff --git a/plugins/imudp/imudp.c b/plugins/imudp/imudp.c index 6a6ab498..f5a8a08d 100644 --- a/plugins/imudp/imudp.c +++ b/plugins/imudp/imudp.c @@ -596,18 +596,14 @@ ENDwillRun BEGINafterRun + int i; CODESTARTafterRun /* do cleanup here */ net.clearAllowedSenders((uchar*)"UDP"); -#warning UDP listen socks must be cloesed! also select must be supported! -#if 0 - if(lcnfinfo.udpLstnSocks != NULL) { - net.closeUDPListenSockets(lcnfinfo.udpLstnSocks); - lcnfinfo.udpLstnSocks = NULL; - free(lcnfinfo.udpRulesets); - lcnfinfo.udpRulesets = NULL; - } -#endif + for (i = 0; i < nLstn ; i++) + close(lcnfinfo[i].udpLstnSocks); + free(lcnfinfo); + lcnfinfo = NULL; if(pRcvBuf != NULL) { free(pRcvBuf); pRcvBuf = NULL; |