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

Padding byte interpreted as signed integer, should be unsigned #57

Open
briandfoy opened this issue Jul 10, 2023 · 1 comment
Open

Padding byte interpreted as signed integer, should be unsigned #57

briandfoy opened this issue Jul 10, 2023 · 1 comment
Labels
imported from rt.cpan.org RT: checked Verified the RT import Status: has fix in RT The original RT queue has a fix

Comments

@briandfoy
Copy link
Owner

This ticket was imported from rt.cpan.org 118437

  • Requestor: xan@biblionix.com
  • Date: Mon Jun 12 02:45:00 2017
  • Original subject: Padding byte interpreted as signed integer, should be unsigned
  • Original link: rt.cpan.org 118437

Hello,

I've just run across a problem connecting to some servers, which I
believe to be the problem described here:
https://fogbugz.bitvise.com/default.asp?WinSSHD.1.11700.2

Basically, Net::SSH::Perl works if the padding happens to be 127 or
below, but it should work all the way up to 255. The symptom is for
example "Corrupted padlen -102 on input".

The solution seems to be to change line 241 (in version 2.01) to 'unpack
"C"' instead of 'unpack "c"'.

Thanks!

-Xan

@briandfoy
Copy link
Owner Author

from schwigon@cpan.org


Hi,

thanks for your report?

Is there any chance that you create a Pull Request on Lance Kinley's repo on github https://github.com/lkinley/Net-SSH-Perl so he can have a look? He is the main author behind current development.

Thanks,
Steffen

On Wed Oct 19 11:38:52 2016, xan@biblionix.com wrote:

Hello,

I've just run across a problem connecting to some servers, which I
believe to be the problem described here:
https://fogbugz.bitvise.com/default.asp?WinSSHD.1.11700.2

Basically, Net::SSH::Perl works if the padding happens to be 127 or
below, but it should work all the way up to 255. The symptom is for
example "Corrupted padlen -102 on input".

The solution seems to be to change line 241 (in version 2.01) to 'unpack
"C"' instead of 'unpack "c"'.

Thanks!

-Xan

@briandfoy briandfoy added RT: checked Verified the RT import Status: has fix in RT The original RT queue has a fix labels Jul 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
imported from rt.cpan.org RT: checked Verified the RT import Status: has fix in RT The original RT queue has a fix
Projects
None yet
Development

No branches or pull requests

1 participant