Thanks again, I have now merged the patch. I had some problems with git-am, probably due to some mangling by the mailing list processor. So I think it would be best to get further patches as attachments. This time, I simply merged it manually ;)
Rainer On Fri, Oct 25, 2013 at 11:15 AM, Rainer Gerhards <[email protected]>wrote: > On Fri, Oct 25, 2013 at 10:59 AM, Radu Gheorghe > <[email protected]>wrote: > >> Dude, you're a blessing to this list/project. > > > Fully ack! I really, really appreciate Pavel's contributions. > > >> I didn't even know that this >> module existed! >> >> > It's not an official module in the sense that I do not support it (don't > have env or know how to do so). I have begun to think about moving these > modules to something like a ./contrib tree, which would make it clearer, > but have not yet given this a shot. > > In general, many of these modules are poorly visible, as the authors > usually contribute no doc. With me struggling with my own doc, you probably > guessed right that I don't write any doc for those packages ;) > > Just FYI, > Rainer > >> >> 2013/10/25 Pavel Levshin <[email protected]> >> >> > >> > Here is the patch to a memory leak in omhiredis. >> > >> > >> > -- >> > Pavel Levshin >> > >> > From: Pavel Levshin <[email protected]> >> > Date: Fri, 25 Oct 2013 12:08:09 +0400 >> > Subject: [PATCH] omhiredis: remove memory leak >> > >> > --- >> > plugins/omhiredis/omhiredis.c | 2 +- >> > 1 files changed, 1 insertions(+), 1 deletions(-) >> > >> > diff --git a/plugins/omhiredis/omhiredis.**c >> > b/plugins/omhiredis/omhiredis.**c index 051ac0b..cb4e608 100644 >> > --- a/plugins/omhiredis/omhiredis.**c >> > +++ b/plugins/omhiredis/omhiredis.**c >> > @@ -196,7 +196,7 @@ CODESTARTendTransaction >> > for ( i = 0; i < pData->count; i++ ) { >> > redisGetReply ( pData->conn, (void *)&pData->replies[i] ); >> > /* TODO: add error checking here! */ >> > - free ( pData->replies[i] ); >> > + freeReplyObject ( pData->replies[i] ); >> > } >> > free ( pData->replies ); >> > pData->count = 0; >> > -- >> > 1.7.1 >> > >> > >> > >> > ______________________________**_________________ >> > rsyslog mailing list >> > http://lists.adiscon.net/**mailman/listinfo/rsyslog< >> http://lists.adiscon.net/mailman/listinfo/rsyslog> >> > http://www.rsyslog.com/**professional-services/< >> http://www.rsyslog.com/professional-services/> >> > What's up with rsyslog? Follow https://twitter.com/rgerhards >> > NOTE WELL: This is a PUBLIC mailing list, posts are ARCHIVED by a myriad >> > of sites beyond our control. PLEASE UNSUBSCRIBE and DO NOT POST if you >> > DON'T LIKE THAT. >> > >> _______________________________________________ >> rsyslog mailing list >> http://lists.adiscon.net/mailman/listinfo/rsyslog >> http://www.rsyslog.com/professional-services/ >> What's up with rsyslog? Follow https://twitter.com/rgerhards >> NOTE WELL: This is a PUBLIC mailing list, posts are ARCHIVED by a myriad >> of sites beyond our control. PLEASE UNSUBSCRIBE and DO NOT POST if you >> DON'T LIKE THAT. >> > > _______________________________________________ rsyslog mailing list http://lists.adiscon.net/mailman/listinfo/rsyslog http://www.rsyslog.com/professional-services/ What's up with rsyslog? Follow https://twitter.com/rgerhards NOTE WELL: This is a PUBLIC mailing list, posts are ARCHIVED by a myriad of sites beyond our control. PLEASE UNSUBSCRIBE and DO NOT POST if you DON'T LIKE THAT.
