Aditi16 / jss7

Automatically exported from code.google.com/p/jss7
0 stars 0 forks source link

Error in ExtForwInfoImpl.java #291

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1.receive subscriberManagement map message with forwardingFeatureList
2. in log there is somethins like this: MAPParsingComponentException when 
parsing components: MistypedParameter - Error while decoding ExtForwInfo: 
Parameter forwardingFeatureList size must be from 1 to 32, found: 3

What is the expected output? What do you see instead?
There is no error in message

What version of the product are you using? On what operating system?
commit ba6f8af25eb1db43f3be4cf7a214dbbda20eb5db
Author: amit.bhayani@gmail.com <amit.bhayani@gmail.com>
Date:   Thu Jun 13 21:41:08 2013 +0530

Please provide any additional information below.
--- 
a/map/map-impl/src/main/java/org/mobicents/protocols/ss7/map/service/mobility/su
bscriberManagement/ExtForwInfoImpl.java
+++ 
b/map/map-impl/src/main/java/org/mobicents/protocols/ss7/map/service/mobility/su
bscriberManagement/ExtForwInfoImpl.java
@@ -128,7 +128,7 @@ public class ExtForwInfoImpl extends SequenceBase 
implements ExtForwInfo {
                         this.forwardingFeatureList.add(featureItem);
                     }

-                    if (this.forwardingFeatureList.size() > 1 || 
this.forwardingFeatureList.size() > 32) {
+                    if (this.forwardingFeatureList.size() < 1 || 
this.forwardingFeatureList.size() > 32) {
                         throw new MAPParsingComponentException("Error while decoding " + _PrimitiveName
                                 + ": Parameter forwardingFeatureList size must be from 1 to 32, found: "
                                 + this.forwardingFeatureList.size(), MAPParsingComponentExceptionReason.MistypedParameter)

Original issue reported on code.google.com by avilov.v...@gmail.com on 14 Jun 2013 at 12:59

GoogleCodeExporter commented 9 years ago
Hello, 

thanks for a bug report.

Original comment by serg.vet...@gmail.com on 18 Jun 2013 at 7:34

GoogleCodeExporter commented 9 years ago
Hello!

I have committed an update into a master branch:

https://code.google.com/p/jss7/source/detail?r=767e602d9e0405c3a47ad264a157e69b2
7354119

Original comment by serg.vet...@gmail.com on 18 Jun 2013 at 8:20

GoogleCodeExporter commented 9 years ago

Original comment by serg.vet...@gmail.com on 7 Sep 2013 at 10:08