Re: [Openh323gk-users] 2.2.7 crash

[Grzegorz S. <sta...@ne...>]

Hi.
 Here come another one. This looks like race condition.

#0  0x40af4a57 in PArrayObjects::DestroyContents (this=0x436549e0) at 
../common/collect.cxx:320
#1  0x40afbc10 in PContainer::Destruct (this=0x436549e0) at 
../common/contain.cxx:810
#2  0x0808b9a9 in ~PASN_Sequence (this=0x436549d0) at 
/usr/local/src/openh323/../pwlib/include/ptlib/array.h:968
#3  0x40386c17 in ~H225_TransportAddress_ipAddress (this=0x436549d0) at 
/usr/local/src/h323plus/include/h225.h:3274
#4  0x409fe6e9 in PASN_Choice::operator= (this=0x8787fec, 
other=@0x425dd258) at ../../ptclib/asner.cxx:2079
#5  0x080b0b96 in EndpointRec::SetRasAddress (this=0x8787fc8, 
addr=@0x425dd258) at /usr/local/src/openh323/include/h225.h:232
#6  0x0809b3d3 in EndpointRec::Update (this=0x8787fc8, 
ras_msg=@0x426a2bc8) at RasTbl.cxx:606
#7  0x081025c1 in RegistrationRequestPDU::Process (this=0x426834b0) at 
RasSrv.cxx:1582
#8  0x080f9be3 in RasMsg::Exec (this=0x426834b0) at RasSrv.cxx:371
#9  0x08068ca8 in Jobs::Run (this=0x426834c8) at job.cxx:447
#10 0x08069cb6 in Worker::Main (this=0x82d9b10) at job.cxx:191
#11 0x40ac5e5f in PThread::PX_ThreadStart (arg=0x82d9b10) at 
tlibthrd.cxx:1361
#12 0x40be0240 in start_thread () from /lib/tls/i686/cmov/libpthread.so.0
#13 0x4104e49e in clone () from /lib/tls/i686/cmov/libc.so.6
(gdb) up 3
#3  0x40386c17 in ~H225_TransportAddress_ipAddress (this=0x436549d0) at 
/usr/local/src/h323plus/include/h225.h:3274
3274    {
(gdb) p *this
$1 = {<PASN_Sequence> = {<PASN_Object> = {<PObject> = {_vptr.PObject = 
0x81ae488}, extendable = 0,
      tagClass = PASN_Object::UniversalTagClass, tag = 16}, fields = 
{<PArrayObjects> = {<PCollection> = {<PContainer> = {<PObject> = {
              _vptr.PObject = 0x81abc58}, reference = 0x4360b010}, <No 
data fields>}, theArray = 0xffffffff}, <No data fields>},
[..]
(gdb) up 4
#7  0x081025c1 in RegistrationRequestPDU::Process (this=0x426834b0) at 
RasSrv.cxx:1582
1582                            ep->Update(m_msg->m_recvRAS);
(gdb) p ep
$1 = {pt = 0x0}
ep was not null 50 lines above,
insetring something like this before line 1582 will be enough?:
if(!ep) return 
BuildRRJ(H225_RegistrationRejectReason::e_fullRegistrationRequired);


Grzegorz Stanislawski