Remove extraneous ERROR messages where they give no extra information

This will help to reduce the debug.log output in some cases by two unnecessary additional lines.

(debug=mempool will need to be in bitcoin.conf to ensure the error is still captured in debug.log)
Isn't the point of cleanSubVer already that it is cleaned for logging purposes? I don't see the point of doing another substr() here.