[Libssh] Windows patch

Aris Adamantiadis aris at 0xbadc0de.be
Tue Mar 4 00:40:14 CET 2008


Hello,
Keisial a écrit :
> Waiting moderation there is a copy of this message, as well as the patch 
> which should makes libssh work on any 32-bit windows with getaddrinfo.
> 
I let it go through, I think everybody has received it.
Thanks for your patch. I added it into the bugtracker.
> Windows specific:
> There's a Makefile.Windows and a predefined config.h.win32-openssl
> If you want to use libgcrypt or have the libraries on different folders, 
> you'll need to manually edit them.
> 
That's not very clean :( but if that works, it's better than nothing

> sample.c wasn't ported due to termios.h not being available in Windows
> A dummy gai_strerror.c was copied from glibc because windows misses that 
> function.
I'd prefer finding an alternative to this. I'll take a look
> 
> General code:
> *Fixed the SIGSEGV when an unavailable cipher was requested (the return 
> of ssh_find_matching wasn't checked).
good
> *All calls to read & write have been changed into recv & send. As the 
> existence of data is previously checked, the 0 parameter shouldn't 
> change anything, but may be worth playing with it.
I know there are some semantic changes when changing from write/read to 
recv/send.
I'm writing a new socket wrapper that will make this easier next time.
> *The sockets are now closed with closesocket, #defined to close on 
> non-windows.
With the wrapper coded, there won't be any need for a macro anymore. 
it's best.
> *I removed aes256-cbc,aes192-cbc,3des-cbc from the negotiated ciphers 
> (not its code). Since they didn't work anyway, the other end will 
> hopefully support a cipher we speak.
It has been fixed this afternoon. A very silly bug.
> *A session->fd>0 fixed to >=
It will disapear with the code i'm writing now.
Thanks for your patch again.
> 
Aris



More information about the Libssh mailing list