Re: [Qemu-devel] [PATCH 05/18] block/ssh: Drop superfluous libssh2_session_last_errno() calls

["Richard W.M. Jones" <rjones@redhat.com>]

On Tue, May 13, 2014 at 06:02:39PM +0200, Markus Armbruster wrote:
> libssh2_session_last_error() already returns the error code.
> 
> Cc: "Richard W.M. Jones" <rjones@redhat.com>
> Signed-off-by: Markus Armbruster <armbru@redhat.com>

Reviewed-by: Richard W.M. Jones <rjones@redhat.com>

>  block/ssh.c | 9 ++++-----
>  1 file changed, 4 insertions(+), 5 deletions(-)
> 
> diff --git a/block/ssh.c b/block/ssh.c
> index aa63c9d..e38d232 100644
> --- a/block/ssh.c
> +++ b/block/ssh.c
> @@ -121,10 +121,9 @@ session_error_report(BDRVSSHState *s, const char *fs, ...)
>          char *ssh_err;
>          int ssh_err_code;
>  
> -        libssh2_session_last_error((s)->session, &ssh_err, NULL, 0);
>          /* This is not an errno.  See <libssh2.h>. */
> -        ssh_err_code = libssh2_session_last_errno((s)->session);
> -
> +        ssh_err_code = libssh2_session_last_error(s->session,
> +                                                  &ssh_err, NULL, 0);
>          error_printf(": %s (libssh2 error code: %d)", ssh_err, ssh_err_code);
>      }
>  
> @@ -145,9 +144,9 @@ sftp_error_report(BDRVSSHState *s, const char *fs, ...)
>          int ssh_err_code;
>          unsigned long sftp_err_code;
>  
> -        libssh2_session_last_error((s)->session, &ssh_err, NULL, 0);
>          /* This is not an errno.  See <libssh2.h>. */
> -        ssh_err_code = libssh2_session_last_errno((s)->session);
> +        ssh_err_code = libssh2_session_last_error(s->session,
> +                                                  &ssh_err, NULL, 0);
>          /* See <libssh2_sftp.h>. */
>          sftp_err_code = libssh2_sftp_last_error((s)->sftp);
>  
> -- 
> 1.8.1.4

-- 
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
libguestfs lets you edit virtual machines.  Supports shell scripting,
bindings from many languages.  http://libguestfs.org