Navigation Menu

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

NullpointerException is occured during loadActualData of M3UA configuration #179

Closed
Peramuthan opened this issue Nov 30, 2016 · 4 comments
Closed

NullpointerException is occured during loadActualData of M3UA configuration #179

Peramuthan opened this issue Nov 30, 2016 · 4 comments

Comments

@Peramuthan
Copy link
Contributor

Peramuthan commented Nov 30, 2016 •
edited

Hi All,
In both jSS7 stack 7.1.1385 and 7.0.1383 versions having this issue.

file name: M3UAManagementImpl.java

private void loadActualData(XMLObjectReader reader ) throws XMLStreamException, IOException{
try {
// this.maxSequenceNumber = reader.read(MAX_SEQUENCE_NUMBER_PROP, Integer.class);
// this.maxAsForRoute = reader.read(MAX_AS_FOR_ROUTE_PROP, Integer.class);
Integer vali = reader.read(MAX_SEQUENCE_NUMBER_PROP, Integer.class);
vali = reader.read(MAX_AS_FOR_ROUTE_PROP, Integer.class);

      this.timeBetweenHeartbeat = reader.read(HEART_BEAT_TIME_PROP, Integer.class);
  } catch (java.lang.Exception e) {
      // ignore.
      // For backward compatibility we can ignore if these values are not defined
      logger.error("Errro while reading attribute", e);
  }
 --

}

Bolded statement is try to read the "maxasforroute" value from XML file. but no where writing this value in XML file.

Please correct this error in latest release.

Thanks & Regards,
Peramuthan Rameshkumar
@vetss vetss added this to the 8.0.0 milestone Nov 30, 2016
@vetss
Copy link
Contributor

vetss commented Nov 30, 2016

@Peramuthan

We need to check - if we use the parameter and it is needed to be stored we can add it.

@deruelle
Copy link
Member

deruelle commented Jan 5, 2017

@vetss did you check that ? Can @Peramuthan take ownership of this one ?

@vetss
Copy link
Contributor

vetss commented Jan 5, 2017

@deruelle

I have not checked it yet, sorry.

@vetss vetss removed this from the 8.0.0 milestone Jan 12, 2017
@vetss
Copy link
Contributor

vetss commented Jan 12, 2017

Hello @Peramuthan

Sorry for delayed response (this issue is minor). There are some parameters for m3ua stack that are not stored in xml files but is needed to configure by setters. And mentioned parameters are also not stored into xml files. What you see is some stabs that prevern of old format sml files reading (before such parameters were present in xml configs). So please do not change code, it is correct.

You can check such parameters list for stacks in the file:
https://github.com/RestComm/jss7/blob/master/Stacks_Parameters.ods
You can find info there which parameters are stored into xml and which are not.
Anyway, @Peramuthan thanks for seaching bugs in jss7 stack.

I am closing the issue.

@vetss vetss closed this as completed Jan 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@deruelle @vetss @Peramuthan