Committed to svn.

 

From: [email protected] [mailto:[email protected]] On Behalf
Of Rene Kluwen
Sent: Thursday, 03 November, 2011 15:12
To: 'Alexander Malysh'
Cc: [email protected]; [email protected]
Subject: RE: This patch fixes...

 

Attached, the simplified version with the modification you mentioned.

 

From: Alexander Malysh [mailto:[email protected]] On Behalf Of
Alexander Malysh
Sent: Thursday, 03 November, 2011 10:05
To: Rene Kluwen
Cc: [email protected]; [email protected]
Subject: Re: This patch fixes...

 

Hi Rene,

 

SMPP rely on the fact that tlv dict exist and I find it should be here. You
can simplify your code because dict_destroy already checks for

Dict to be NULL and tlv dict is always initialized.

 

Alex

 

Am 02.11.2011 um 19:47 schrieb Rene Kluwen:

 

.a memory leak in opensmppbox whilst using v3.4 tlv values in incoming
messages.

 

Rather than using this patch, I don't want to allocate a dict in
smpp_pdu_create at all. But I am not sure about how it will affect the rest
of the smpp code.

 

== Rene

<tlv_memory_leak.diff>

 

Reply via email to