Jeremy Cross
6697f85b50
sftp: ensure sftp_packet_read recognizes channel EOF to avoid infinite loop
...
Signed-off-by: Jeremy Cross <jcross@bomgar.com >
Reviewed-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit dbf72ffba2 )
2016-09-09 11:40:07 +02:00
Travers Carter
67fe6f56ea
Make "Host" pattern list handling consistent with OpenSSH
...
https://red.libssh.org/issues/187
Signed-off-by: Travers Carter <tcarter@noggin.com.au >
Reviewed-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit bc2db86d1c )
2016-05-03 17:57:45 +02:00
Andreas Schneider
b5ce15eefa
priv: Fix client banner specification for libssh
...
BUG: https://red.libssh.org/issues/231
Signed-off-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit 4f7be0dbb2 )
2016-05-03 10:58:47 +02:00
Andreas Schneider
a3688ada1a
client: If SSHv1 is disabled send the banner immediately
...
This saves a round-trip with SSHv2 connecting to the server. See RFC
4253 section 5.2 (New Client, Old Server).
Thanks to Yang Yubo <yang@yangyubo.com > for the suggestion.
Signed-off-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit 1da5c94b44 )
2016-05-03 10:58:36 +02:00
Andreas Schneider
219d0bba42
client: Fix ssh_send_banner() to confirm with RFC 4253
...
Signed-off-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit 3d1edffe77 )
2016-05-03 10:58:27 +02:00
Andreas Schneider
bf3d8f3ad4
client: Fix maximum banner length
...
According to RFC 4253 the max banner length is 255.
Thanks to Saju Panikulam <spanikulam@ipswitch.com > for the report.
Signed-off-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit cb52ed7b12 )
2016-05-03 10:58:18 +02:00
Andreas Schneider
04a5d5bd74
client: Reformat callback_receive_banner()
...
The function is hard to read as the indentation is not correctly
applied.
Signed-off-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit 22799b107d )
2016-05-03 10:58:06 +02:00
Ken Reister
2957aaf9f0
client: Receive the banner correctly
...
Comply with RFC 4253 compliance section 4.2.
Allow data other than "SSH-" to be sent across prior to the actual
version striong.
Signed-off-by: Ken Reister <reister.kenneth@CIMCOR.COM >
Reviewed-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit c3ce3d5cc3 )
2016-05-03 10:57:50 +02:00
Kohei Suzuki
8360139506
Add id_ed25519 to the default identity list
...
Signed-off-by: Kohei Suzuki <eagletmt@gmail.com >
Reviewed-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit c092101e01 )
2016-05-02 15:34:32 +02:00
David Kedves
0bf78b0b8b
channels: Bugfix for a possible invalid pointer usage (channel->session) in various places
...
BUG: https://red.libssh.org/issues/230
Signed-off-by: David Kedves <kedazo@severalnines.com >
Reviewed-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit d7df4429eb )
2016-05-02 15:32:16 +02:00
Stef Walter
faca78f547
auth: Cleanup memory leak when using SSH agent
...
In Cockpit we've seen this memory leak:
at 0x4C2A9C7: calloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
by 0x5B76B03: ssh_userauth_agent (auth.c:778)
by 0x40DD5A: cockpit_ssh_authenticate (cockpitsshtransport.c:327)
BUG: https://red.libssh.org/issues/208
Signed-off-by: Stef Walter <stefw@redhat.com >
Reviewed-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit ffe8b98cc2 )
2016-05-02 15:29:01 +02:00
Andreas Schneider
7da587ba6c
auth1: Fix non-blocking SSHv1 auth
...
BUG: https://red.libssh.org/issues/232
Thanks to Fengyu Gao.
TODO: Add SSHv1 tests to our testsuite.
Signed-off-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit 33ecaaac01 )
2016-05-02 12:31:53 +02:00
Andreas Schneider
c7aa51240d
Update the README
...
(cherry picked from commit e8b28f978e )
2016-05-02 12:11:02 +02:00
Andreas Schneider
cdf7690e03
Bump version to 0.7.3
...
Signed-off-by: Andreas Schneider <asn@cryptomilk.org >
libssh-0.7.3
2016-02-23 08:23:31 +01:00
Andreas Schneider
7b19719022
Update ChangeLog
2016-02-23 08:22:09 +01:00
Aris Adamantiadis
f8d0026c65
dh: Fix CVE-2016-0739
...
Due to a byte/bit confusion, the DH secret was too short. This file was
completely reworked and will be commited in a future version.
Signed-off-by: Aris Adamantiadis <aris@0xbadc0de.be >
Reviewed-by: Andreas Schneider <asn@cryptomilk.org >
2016-02-23 08:17:43 +01:00
Hani Benhabiles
6b608e70ee
options: Fix documentation typo
...
Signed-off-by: Hani Benhabiles <hani@linux.com >
Reviewed-by: Andreas Schneider <asn@samba.org >
(cherry picked from commit f8bde7156f )
2016-02-23 08:15:26 +01:00
Younes Serraj
a69a1af568
pki: Fixed documentation about return value.
...
Documentation now is congruent with the code:
- SSH_OK is returned on success,
- SSH_ERROR is returned on error.
Signed-off-by: Younes Serraj <younes.serraj@gmail.com >
Reviewed-by: Andreas Schneider <asn@samba.org >
(cherry picked from commit 66c6ae1a55 )
2016-02-23 08:15:24 +01:00
pouete
32b72555ee
cmake: use check_symbol for (v)snprintf
...
Updated how snprintf and vsnprintf are discovered by cmake. Visual studio
2015 now include it in the file stdio.h.
More information here :
https://msdn.microsoft.com/en-us/library/bb531344.aspx
Reviewed-By: Aris Adamantiadis <aris@0xbadc0de.be >
2015-12-30 19:51:26 +01:00
Sebastián Peyrott
32af6a2390
CMake: include CheckIncludeFiles for calls to check_include_files.
2015-12-24 13:01:56 +01:00
Fabiano Fidêncio
b470dd943f
Fix a bunch of -Wmaybe-uninitialized
...
Reviewed-By: Aris Adamantiadis <aris@0xbadc0de.be >
2015-12-17 15:02:01 +01:00
Dirk Neukirchen
69ca977aed
headers: fix missing mode_t (2nd)
...
Reviewed-By: Aris Adamantiadis <aris@0xbadc0de.be >
2015-11-10 18:39:59 +01:00
Aris Adamantiadis
728a6349b7
Revert "headers: fix missing mode_t"
...
I commited a patch file *headdesk*
This reverts commit 378fcccc0a .
2015-11-10 18:39:52 +01:00
Dirk Neukirchen
ec32174abc
headers: fix missing mode_t
...
Signed-off-by: Dirk Neukirchen <dirkneukirchen@web.de >
Reviewed-by: Aris Adamantiadis <aris@0xbadc0de.be >
2015-11-10 18:28:45 +01:00
Andreas Schneider
2172cd234a
Ignore all build and obj* directories
...
Signed-off-by: Andreas Schneider <asn@cryptomilk.org >
2015-10-28 09:28:42 +01:00
Andreas Schneider
0425ac9ad0
agent: Fix agent auth on big endian machines
...
BUG: https://red.libssh.org/issues/204
Signed-off-by: Andreas Schneider <asn@cryptomilk.org >
2015-10-20 17:17:17 +02:00
Aris Adamantiadis
367558bb21
crypto: fix potential memory leak in ECDH
2015-09-21 15:03:08 +02:00
Andreas Schneider
186e7b5ca4
kex: Fix zlib compression
...
Signed-off-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit 747e7d05db )
libssh-0.7.2
2015-09-16 08:34:58 +02:00
Andreas Schneider
2197704693
Bump version to 0.7.2
2015-09-15 15:17:35 +02:00
Andreas Schneider
229eb8715d
cmake: Use tar.xz source package generator
...
Signed-off-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit 1f3a9105ff )
2015-09-15 15:17:35 +02:00
Andreas Schneider
1b18a06f8c
kex: Prefer sha2 over sha1
...
Signed-off-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit b0f22fde62 )
2015-09-15 15:09:21 +02:00
Andreas Schneider
91b513798e
cmake: Handle libssh threas library correctly
...
This should fix the build on Windows and would not install pkg files.
Signed-off-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit 5b586fdfec )
2015-09-08 17:32:57 +02:00
Michael Wilder
25234e510a
bignum: Fix OpenSSL crash in SAFE_FREE
...
Signed-off-by: Michael Wilder <wilder.michael@cimcor.com >
Reviewed-by: Andreas Schneider <asn@samba.org >
(cherry picked from commit 2f193b5cbb )
2015-09-08 17:32:40 +02:00
Andreas Schneider
d16eac5704
server: Fix return code check of ssh_buffer_pack()
...
Thanks to Andreas Gutschick <andreas.gutschick@mitel.com >
Signed-off-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit 36d9b62f1f )
2015-08-18 09:12:47 +02:00
Andreas Schneider
46bff47975
doc: Fix typos in sftp tutorial
...
Thanks to Anthony Baker <AnthonyBaker@fico.com >
Signed-off-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit da4bebbe1e )
2015-08-18 09:05:45 +02:00
Andreas Schneider
f718b50b3f
tests: Add checks for ssh_key_is_private()
...
Signed-off-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit d54a1ab798 )
2015-08-10 13:58:51 +02:00
Andreas Schneider
58b7d0f5d2
pki: Fix return values of ssh_key_is_(public|private)
...
Thanks to Kevin Haake <khaake@red-cocoa.com >
Signed-off-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit e1081796af )
2015-08-10 13:58:50 +02:00
Tilo Eckert
30d4581be5
sftp: Fix incorrect handling of received length fields
...
Signed-off-by: Tilo Eckert <tilo.eckert@flam.de >
2015-08-01 10:52:48 +03:00
Peter Volpe
83387f957f
auth: Fix return status for ssh_userauth_agent()
...
BUG: https://red.libssh.org/issues/201
Return SSH_AUTH_DENIED instead of SSH_AUTH_ERROR when the provided agent
offers no public keys.
Signed-off-by: Peter Volpe <pvolpe@redhat.com >
Reviewed-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit dc9c4d22ab )
2015-07-30 10:52:11 +02:00
Andreas Schneider
f3620bbbad
cmake: Fix zlib include directory
...
Signed-off-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit 728c2fbd01 )
2015-07-03 12:36:53 +02:00
Andreas Schneider
b45933d30d
cmake: Fix OpenSSL detection in non-standard path
...
This should fix the detection on Windows.
Signed-off-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit 613b71b385 )
2015-07-03 11:40:04 +02:00
Andreas Schneider
1613ed556d
cmake: Fail if can't find OpenSSL aes and des headers
...
Signed-off-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit 70cc11618a )
2015-07-03 10:52:56 +02:00
Andreas Schneider
8f5b7b65eb
include: Add stdarg.h so we can check for va_copy macro
...
Signed-off-by: Andreas Schneider <asn@cryptomilk.org >
2015-06-30 09:59:21 +02:00
Andreas Schneider
053f72c671
Bump version to 0.7.1
...
Signed-off-by: Andreas Schneider <asn@cryptomilk.org >
libssh-0.7.1
2015-06-30 09:34:28 +02:00
Tilo Eckert
63a8f333b8
SSH_AUTH_PARTIAL is now correctly passed to the caller of ssh_userauth_publickey_auto().
...
Implicitly fixed unsafe return code handling that could result in use-after-free.
Signed-off-by: Tilo Eckert <tilo.eckert@flam.de >
Reviewed-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit 0423057424 )
2015-06-29 11:11:26 +02:00
Tilo Eckert
57fd8e3187
available auth_methods must be reset on partial authentication
...
Signed-off-by: Tilo Eckert <tilo.eckert@flam.de >
Reviewed-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit cc25d747d4 )
2015-06-29 11:11:25 +02:00
Peter Volpe
03972b16c9
channels: Fix exit-signal data unpacking
...
Signed-off-by: Peter Volpe <pvolpe@redhat.com >
Reviewed-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit 7637351065 )
2015-06-29 09:50:28 +02:00
Peter Volpe
ac7ed82585
agent: Add ssh_set_agent_socket
...
Allow callers to specify their own socket
for an ssh agent.
Signed-off-by: Peter Volpe <pvolpe@redhat.com >
Reviewed-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit 7aeba71a92 )
2015-06-29 09:47:35 +02:00
Seb Boving
196c2e9c1f
Don't allocate a new identity list in the new session's options.
...
The previous list is not freed. Since the new session just got
created, an identity list is already allocated and empty.
Signed-off-by: Sebastien Boving <seb@google.com >
Reviewed-by: Andreas Schneider <asn@cryptomilk.org >
(cherry picked from commit e020dd8d59 )
2015-06-24 18:36:10 +02:00
Douglas Heriot
1accbcb98b
cmake: Do not use CMAKE_(SOURCE|BINARY)_DIR
...
(cherry picked from commit a65af1b3b8 )
2015-06-24 18:36:08 +02:00