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.

Reply via email to