| Summary: | ASTERISK-02556: [patch] Memory leak fixes for chan_sip.c with realtime() functions | ||
| Reporter: | Rob Gagnon (rgagnon) | Labels: | |
| Date Opened: | 2004-10-07 13:12:55 | Date Closed: | 2004-10-07 22:45:39 |
| Priority: | Minor | Regression? | No |
| Status: | Closed/Complete | Components: | Channels/chan_sip/DatabaseSupport |
| Versions: | Frequency of Occurrence | ||
| Related Issues: | |||
| Environment: | Attachments: | ( 0) chan_sip.c.patch.txt ( 1) chan_sip.c.patch.txt | |
| Description: | In some places, where a call is made to find_user(), the "temponly" value was not checked before exitting the function. Also, destroy_peer() was missing a call to ast_free_ha() to free any ACL that might be hanging on the peer. ****** ADDITIONAL INFORMATION ****** [disclaimed] | ||
| Comments: | By: Rob Gagnon (rgagnon) 2004-10-07 13:15:51 hang on before patching this. I found another, and am about to re-do this patch By: Rob Gagnon (rgagnon) 2004-10-07 13:20:59 OK Now, "chan_sip.c.patch.txt (3,385 bytes) 10-07-04 13:20" fixes 7 instances where a memory leak would occur if find_peer() or find_user() used the realtime() functions to locate a user or peer dynamically. By: Mark Spencer (markster) 2004-10-07 14:23:16 Fixed, thanks! By: Russell Bryant (russell) 2004-10-07 22:45:39 only applies to CVS head | ||