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

EOFException in TConnection.startConnection() due to missing (?) maxTPDUSizeParam handling #40

Open
imario42 opened this issue Jan 8, 2024 · 0 comments

Comments

@imario42
Copy link

imario42 commented Jan 8, 2024

I use a version of the TConnection which is similar to the one you can find here

https://www.javatips.net/api/openiec61850-master/src/mms/jositransport/src/main/java/org/openmuc/jositransport/TConnection.java

In startConnection() this version does something different when it comes to the maxTPDUSizeParam handling.

int variableLength = 0; if (maxTPDUSizeParam < 16 && maxTPDUSizeParam >= 7) { variableLength += 3; }

The version in your repository simply is doing:

int variableLength = 3;

Now I wanted to switch to your version, but this difference seems to not make our S7 controller happy and I get an EOFException later on when reading the first byte in startConnection().
Seems the remote S7 simply drops the connection.

Is there something we can change in the config of our S7 or is this just missing from the version in your repository? If so, I can prepare a patch then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant