[PATCH] SMPP trivial

Alexander Malysh <[email protected]>
Newsgroups gmane.comp.mobile.kannel.devel
Organization Centrium GmbH
Message-ID <[email protected]>
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hi Stipe,list

Stipe I believe you have not seen corrected patch for smpp pdu parsing ;( It 
was my fault not to resend my patch ;)
Please apply , this trivial patch will fix parsing of opt. parameters with 
length = 0...

- -- 
Best Regards / Mit besten Grüßen aus Köln

Dipl.-Ing.
Alexander Malysh
___________________________________

Centrium GmbH
Ehrenstrasse 2
50672 Köln

Fon: +49 (0221) 277 49 150
Fax: +49 (0221) 277 49 109

email: [email protected]
web: http://www.centrium.de
msn: [email protected]
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.1 (GNU/Linux)

iD8DBQE+NwflnX3e5W+uJ0ERAt9ZAJ4twVEf2iuNNDgV6SOIGiPaf1AovwCgkmzM
RgKdsBmWGaTXmT5G0Cy2Tgg=
=i40c
-----END PGP SIGNATURE-----
smpp_trivial.diff (text/x-diff, 977 B)
Index: gw/smsc/smpp_pdu.c
===================================================================
RCS file: /home/cvs/gateway/gw/smsc/smpp_pdu.c,v
retrieving revision 1.8
diff -a -u -r1.8 smpp_pdu.c
--- gw/smsc/smpp_pdu.c	28 Jan 2003 00:11:54 -0000	1.8
+++ gw/smsc/smpp_pdu.c	28 Jan 2003 22:46:47 -0000
@@ -218,7 +218,7 @@
     switch (type) {
     #define OPTIONAL_BEGIN(num_expected)                                                \
         {   /* Read optional parameters */                                              \
-            while (pos+4 < len) {                                                       \
+            while (pos+4 <= len) {                                                      \
                 unsigned long opt_tag, opt_len;                                         \
                 Octstr *opt_val = NULL;                                                 \
                 Octstr *tag_str = NULL;                                                 \
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.