[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Ccrtp-devel] missing memory desallocation
From: |
Federico Montesino Pouzols |
Subject: |
Re: [Ccrtp-devel] missing memory desallocation |
Date: |
Sun, 6 Apr 2003 16:36:02 +0200 |
User-agent: |
Mutt/1.5.3i |
It is fixed in CVS. I have added a new data member in which
the string returned by getHostname is stored.
On Wed, Apr 02, 2003 at 02:10:34PM +0200, glodas wrote:
> Hi,
>
> we've found another problem in source.cpp:212 in the function
> RTPApplication::findCNAME you call const char
> *InetAddress::getHostname(void) const
> (inaddr.cpp from package commonc++ 1.0.8) which is buggy
>
> inaddr.cpp :
> 382 if(hp)
> 383 return hp->h_name
> hp points on the stack variable hb which content points on the stack
> array hbuf.
> hp->h_name is removed from the stack at the end of the function, so the
> pointer is no more valid,
>
> As you seem to be involved in common c++ too... ;)
>
> Guillaume Fraysse,
> Guillaume Glodas
>
>
>
>
> Le mer 02/04/2003 à 13:37, David Sugar a écrit :
> > We can do a 1.0.1 this week if nessisary.
> >
> > Federico Montesino Pouzols wrote:
> >
> > > Hi,
> > >
> > > it is fixed in CVS. ccRTP 1.0 has been released the last week,
> > >so the leak will be fixed in 1.0.1. Note that 1.0 comes with an
> > >important bug fix with respect to 1.0pre2, so I recommend you to
> > >upgrade.
> > >
> > >On Wed, Apr 02, 2003 at 10:48:08AM +0200, Guillaume FRAYSSE wrote:
> > >
> > >
> > >>Hello,
> > >>
> > >>While lokking for memory leaks with valgrind in an application we've
> > >>written we found one in ccRTP, in the class TRTPSessionBase in rtp.h,
> > >>the dso and cso object are never desallocated.
> > >>
> > >>We fixed this in the endSocket method like this :
> > >>
> > >>inline void
> > >>endSocket()
> > >>{
> > >> dso->endSocket();
> > >> cso->endSocket();
> > >> if (dso) delete dso; dso = NULL;
> > >> if(cso) delete cso; cso = NULL;
> > >>}
> > >>
> > >>We use the pre2 but I didn't see a fix in the CVS repository neither.
> > >>Will this be fixed in a later release ?
> > >>
> > >>Hope this helps,
> > >>Cheers,
> > >>Guillaume
> > >>
> > >>
> > >>_______________________________________________
> > >>Ccrtp-devel mailing list
> > >>address@hidden
> > >>http://mail.gnu.org/mailman/listinfo/ccrtp-devel
> > >>
> > >>
> > >
> > >
> > >_______________________________________________
> > >Ccrtp-devel mailing list
> > >address@hidden
> > >http://mail.gnu.org/mailman/listinfo/ccrtp-devel
> > >
> > >
> >
> >
>
>