<feed xmlns='http://www.w3.org/2005/Atom'>
<title>delta/curl.git/lib/ssh.c, branch bagder/timerfunction-not-recursive</title>
<subtitle>github.com: bagder/curl.git
</subtitle>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/curl.git/'/>
<entry>
<title>ssh: log the libssh2 error message when ssh session startup fails</title>
<updated>2019-01-17T14:03:16+00:00</updated>
<author>
<name>JDepooter</name>
<email>joel.depooter@safe.com</email>
</author>
<published>2019-01-17T01:18:20+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/curl.git/commit/?id=b095a1ca63ab4962e7256616ff02607c5a77e089'/>
<id>b095a1ca63ab4962e7256616ff02607c5a77e089</id>
<content type='text'>
When a ssh session startup fails, it is useful to know why it has
failed. This commit changes the message from:
   "Failure establishing ssh session"
to something like this, for example:
   "Failure establishing ssh session: -5, Unable to exchange encryption keys"

Closes #3481
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
When a ssh session startup fails, it is useful to know why it has
failed. This commit changes the message from:
   "Failure establishing ssh session"
to something like this, for example:
   "Failure establishing ssh session: -5, Unable to exchange encryption keys"

Closes #3481
</pre>
</div>
</content>
</entry>
<entry>
<title>snprintf: renamed and we now only use msnprintf()</title>
<updated>2018-11-23T07:26:51+00:00</updated>
<author>
<name>Daniel Stenberg</name>
<email>daniel@haxx.se</email>
</author>
<published>2018-11-22T08:01:24+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/curl.git/commit/?id=dcd6f810255785d52b89150e18460fb0899d4f7e'/>
<id>dcd6f810255785d52b89150e18460fb0899d4f7e</id>
<content type='text'>
The function does not return the same value as snprintf() normally does,
so readers may be mislead into thinking the code works differently than
it actually does. A different function name makes this easier to detect.

Reported-by: Tomas Hoger
Assisted-by: Daniel Gustafsson
Fixes #3296
Closes #3297
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The function does not return the same value as snprintf() normally does,
so readers may be mislead into thinking the code works differently than
it actually does. A different function name makes this easier to detect.

Reported-by: Tomas Hoger
Assisted-by: Daniel Gustafsson
Fixes #3296
Closes #3297
</pre>
</div>
</content>
</entry>
<entry>
<title>ssh: free the session on init failures</title>
<updated>2018-10-26T13:39:15+00:00</updated>
<author>
<name>Daniel Gustafsson</name>
<email>daniel@yesql.se</email>
</author>
<published>2018-10-26T13:39:15+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/curl.git/commit/?id=5c8c310edb1e5dbf41c5d2d68622b63f7700488e'/>
<id>5c8c310edb1e5dbf41c5d2d68622b63f7700488e</id>
<content type='text'>
Ensure to clear the session object in case the libssh2 initialization
fails.

It could be argued that the libssh2 error function should be called to
get a proper error message in this case. But since the only error path
in libssh2_knownhost_init() is memory a allocation failure it's safest
to avoid since the libssh2 error handling allocates memory.

Closes #3179
Reviewed-by: Daniel Stenberg &lt;daniel@haxx.se&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Ensure to clear the session object in case the libssh2 initialization
fails.

It could be argued that the libssh2 error function should be called to
get a proper error message in this case. But since the only error path
in libssh2_knownhost_init() is memory a allocation failure it's safest
to avoid since the libssh2 error handling allocates memory.

Closes #3179
Reviewed-by: Daniel Stenberg &lt;daniel@haxx.se&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>sftp: don't send post-qoute sequence when retrying a connection</title>
<updated>2018-09-04T09:00:17+00:00</updated>
<author>
<name>Przemysław Tomaszewski</name>
<email>p.tomaszewski.mail@gmail.com</email>
</author>
<published>2018-09-04T06:44:34+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/curl.git/commit/?id=daa3c450d066bbecd76272f7ada7e80f51e2fab5'/>
<id>daa3c450d066bbecd76272f7ada7e80f51e2fab5</id>
<content type='text'>
Fixes #2939
Closes #2940
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Fixes #2939
Closes #2940
</pre>
</div>
</content>
</entry>
<entry>
<title>all: s/int/size_t cleanup</title>
<updated>2018-09-01T08:40:42+00:00</updated>
<author>
<name>Daniel Stenberg</name>
<email>daniel@haxx.se</email>
</author>
<published>2018-08-31T08:17:40+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/curl.git/commit/?id=1a890997a47d4d22df58b5183181685e49ed6e61'/>
<id>1a890997a47d4d22df58b5183181685e49ed6e61</id>
<content type='text'>
Assisted-by: Rikard Falkeborn

Closes #2922
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Assisted-by: Rikard Falkeborn

Closes #2922
</pre>
</div>
</content>
</entry>
<entry>
<title>curl-compilers: enable -Wimplicit-fallthrough=4 for GCC</title>
<updated>2018-08-21T16:53:45+00:00</updated>
<author>
<name>Marcel Raad</name>
<email>Marcel.Raad@teamviewer.com</email>
</author>
<published>2018-07-09T16:52:05+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/curl.git/commit/?id=15ed9f87e3a1f91bd7b4a5291edf268b843e5987'/>
<id>15ed9f87e3a1f91bd7b4a5291edf268b843e5987</id>
<content type='text'>
This enables level 4 instead of the default level 3, which of the
currently used comments only allows /* FALLTHROUGH */ to silence the
warning.

Closes https://github.com/curl/curl/pull/2747
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This enables level 4 instead of the default level 3, which of the
currently used comments only allows /* FALLTHROUGH */ to silence the
warning.

Closes https://github.com/curl/curl/pull/2747
</pre>
</div>
</content>
</entry>
<entry>
<title>Curl_debug: remove dead printhost code</title>
<updated>2018-06-12T14:00:27+00:00</updated>
<author>
<name>Daniel Stenberg</name>
<email>daniel@haxx.se</email>
</author>
<published>2018-06-11T22:10:43+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/curl.git/commit/?id=5ae22272d1506e5c0e431f4ad60f1dc446f76715'/>
<id>5ae22272d1506e5c0e431f4ad60f1dc446f76715</id>
<content type='text'>
The struct field is never set (since 5e0d9aea3) so remove the use of it
and remove the connectdata pointer from the prototype.

Reported-by: Tejas
Bug: https://curl.haxx.se/mail/lib-2018-06/0054.html
Closes #2647
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The struct field is never set (since 5e0d9aea3) so remove the use of it
and remove the connectdata pointer from the prototype.

Reported-by: Tejas
Bug: https://curl.haxx.se/mail/lib-2018-06/0054.html
Closes #2647
</pre>
</div>
</content>
</entry>
<entry>
<title>ssh: show libSSH2 error code when closing fails</title>
<updated>2018-04-16T14:46:08+00:00</updated>
<author>
<name>Christian Schmitz</name>
<email>support@monkeybreadsoftware.de</email>
</author>
<published>2018-04-15T08:49:30+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/curl.git/commit/?id=78611c745c9ebcfc8ba9f956a330f1e5dd7e4ba7'/>
<id>78611c745c9ebcfc8ba9f956a330f1e5dd7e4ba7</id>
<content type='text'>
Closes #2500
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Closes #2500
</pre>
</div>
</content>
</entry>
<entry>
<title>ssh: add two missing state names</title>
<updated>2018-02-16T08:12:42+00:00</updated>
<author>
<name>Daniel Stenberg</name>
<email>daniel@haxx.se</email>
</author>
<published>2018-02-15T12:40:35+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/curl.git/commit/?id=797b21068ecbdcbd27ebc2d1d8f1a1ea3ffa1c14'/>
<id>797b21068ecbdcbd27ebc2d1d8f1a1ea3ffa1c14</id>
<content type='text'>
The list of state names (used in debug builds) was out of sync in
relation to the list of states (used in all builds).

I now added an assert to make sure the sizes of the two lists match, to
aid in detecting this mistake better in the future.

Regression since c92d2e14cf, shipped in 7.58.0.

Reported-by: Somnath Kundu

Fixes #2312
Closes #2313
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The list of state names (used in debug builds) was out of sync in
relation to the list of states (used in all builds).

I now added an assert to make sure the sizes of the two lists match, to
aid in detecting this mistake better in the future.

Regression since c92d2e14cf, shipped in 7.58.0.

Reported-by: Somnath Kundu

Fixes #2312
Closes #2313
</pre>
</div>
</content>
</entry>
<entry>
<title>TODO fixed: Detect when called from within callbacks</title>
<updated>2018-02-15T08:36:03+00:00</updated>
<author>
<name>Björn Stenberg</name>
<email>bjorn@haxx.se</email>
</author>
<published>2018-02-10T14:13:15+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/curl.git/commit/?id=b46cfbc068ebe90f18e9777b9e877e4934c1b5e3'/>
<id>b46cfbc068ebe90f18e9777b9e877e4934c1b5e3</id>
<content type='text'>
Closes #2302
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Closes #2302
</pre>
</div>
</content>
</entry>
</feed>
