[modbus] Fix for #8298: Avoid NPE on serial I/O error

[Wolfgang1966]

For some reason I did not analyze in detail, endpoint is null in my case when maybeResetConnection is called. As an I/O error was preceding the call, resetting the connection seems to be a good way of dealing with the situation. At least it fixes the problem in my setup.

Find patched jars here:

https://home.klimt.de/owncloud/index.php/s/7zbBpJWKPSYBrPH

[TravisBuddy]

Travis tests were successful

Hey @Wolfgang1966,
we found no major flaws with your code. Still you might want to look at this logfile, as we usually suggest some optional improvements.

[TravisBuddy]

Hey @Wolfgang1966,
Something went wrong with the build.

TravisCI finished with status errored, which means the build failed because of something unrelated to the tests, such as a problem with a dependency or the build process itself.

View build log

TravisBuddy Request Identifier: 0ce786a0-dedd-11ea-a02c-4f626b4af334

[TravisBuddy]

Hey @Wolfgang1966,
Something went wrong with the build.

TravisCI finished with status errored, which means the build failed because of something unrelated to the tests, such as a problem with a dependency or the build process itself.

View build log

TravisBuddy Request Identifier: 27c60910-dedd-11ea-a02c-4f626b4af334

[TravisBuddy]

Travis tests were successful

Hey @Wolfgang1966,
we found no major flaws with your code. Still you might want to look at this logfile, as we usually suggest some optional improvements.

[Wolfgang1966]

I somehow messed up the signoff. Do I have to open a new PR to get this sorted out?

[TravisBuddy]

Travis tests were successful

Hey @Wolfgang1966,
we found no major flaws with your code. Still you might want to look at this logfile, as we usually suggest some optional improvements.

[TravisBuddy]

Travis tests were successful

Hey @Wolfgang1966,
we found no major flaws with your code. Still you might want to look at this logfile, as we usually suggest some optional improvements.

[TravisBuddy]

Travis tests were successful

Hey @Wolfgang1966,
we found no major flaws with your code. Still you might want to look at this logfile, as we usually suggest some optional improvements.

[ssalonen]

@Wolfgang1966 I believe the issue is fixed already in #8206?

For what it's worth, the PR explains the root cause

[Wolfgang1966]

Hi @ssalonen , yes, you are right. I close this PR.

Sorry, I could have checked the existing modbus PRs first.

Have a nice weekend!

Wolfgang