[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 2/7] block/ssh: make compliant with coding gu
From: |
Richard W.M. Jones |
Subject: |
Re: [Qemu-devel] [PATCH v3 2/7] block/ssh: make compliant with coding guidelines |
Date: |
Wed, 8 Nov 2017 11:38:15 +0000 |
User-agent: |
Mutt/1.5.20 (2009-12-10) |
On Tue, Nov 07, 2017 at 05:27:19PM -0500, Jeff Cody wrote:
> Signed-off-by: Jeff Cody <address@hidden>
> Reviewed-by: Eric Blake <address@hidden>
This one is just simple coding style fixes, so:
Reviewed-by: Richard W.M. Jones <address@hidden>
Rich.
> block/ssh.c | 32 ++++++++++++++++++--------------
> 1 file changed, 18 insertions(+), 14 deletions(-)
>
> diff --git a/block/ssh.c b/block/ssh.c
> index de81ec8..39cacc1 100644
> --- a/block/ssh.c
> +++ b/block/ssh.c
> @@ -241,7 +241,7 @@ static int parse_uri(const char *filename, QDict
> *options, Error **errp)
> goto err;
> }
>
> - if(uri->user && strcmp(uri->user, "") != 0) {
> + if (uri->user && strcmp(uri->user, "") != 0) {
> qdict_put_str(options, "user", uri->user);
> }
>
> @@ -268,7 +268,7 @@ static int parse_uri(const char *filename, QDict
> *options, Error **errp)
>
> err:
> if (uri) {
> - uri_free(uri);
> + uri_free(uri);
> }
> return -EINVAL;
> }
> @@ -342,7 +342,7 @@ static int check_host_key_knownhosts(BDRVSSHState *s,
> libssh2_knownhost_readfile(knh, knh_file,
> LIBSSH2_KNOWNHOST_FILE_OPENSSH);
>
> r = libssh2_knownhost_checkp(knh, host, port, hostkey, len,
> - LIBSSH2_KNOWNHOST_TYPE_PLAIN|
> + LIBSSH2_KNOWNHOST_TYPE_PLAIN |
> LIBSSH2_KNOWNHOST_KEYENC_RAW,
> &found);
> switch (r) {
> @@ -405,15 +405,18 @@ static int compare_fingerprint(const unsigned char
> *fingerprint, size_t len,
> unsigned c;
>
> while (len > 0) {
> - while (*host_key_check == ':')
> + while (*host_key_check == ':') {
> host_key_check++;
> + }
> if (!qemu_isxdigit(host_key_check[0]) ||
> - !qemu_isxdigit(host_key_check[1]))
> + !qemu_isxdigit(host_key_check[1])) {
> return 1;
> + }
> c = hex2decimal(host_key_check[0]) * 16 +
> hex2decimal(host_key_check[1]);
> - if (c - *fingerprint != 0)
> + if (c - *fingerprint != 0) {
> return c - *fingerprint;
> + }
> fingerprint++;
> len--;
> host_key_check += 2;
> @@ -433,8 +436,8 @@ check_host_key_hash(BDRVSSHState *s, const char *hash,
> return -EINVAL;
> }
>
> - if(compare_fingerprint((unsigned char *) fingerprint, fingerprint_len,
> - hash) != 0) {
> + if (compare_fingerprint((unsigned char *) fingerprint, fingerprint_len,
> + hash) != 0) {
> error_setg(errp, "remote host key does not match host_key_check
> '%s'",
> hash);
> return -EPERM;
> @@ -507,7 +510,7 @@ static int authenticate(BDRVSSHState *s, const char
> *user, Error **errp)
> goto out;
> }
>
> - for(;;) {
> + for (;;) {
> r = libssh2_agent_get_identity(agent, &identity, prev_identity);
> if (r == 1) { /* end of list */
> break;
> @@ -860,8 +863,8 @@ static int ssh_create(const char *filename, QemuOpts
> *opts, Error **errp)
> }
>
> r = connect_to_ssh(&s, uri_options,
> - LIBSSH2_FXF_READ|LIBSSH2_FXF_WRITE|
> - LIBSSH2_FXF_CREAT|LIBSSH2_FXF_TRUNC,
> + LIBSSH2_FXF_READ | LIBSSH2_FXF_WRITE |
> + LIBSSH2_FXF_CREAT | LIBSSH2_FXF_TRUNC,
> 0644, errp);
> if (r < 0) {
> ret = r;
> @@ -869,7 +872,7 @@ static int ssh_create(const char *filename, QemuOpts
> *opts, Error **errp)
> }
>
> if (total_size > 0) {
> - libssh2_sftp_seek64(s.sftp_handle, total_size-1);
> + libssh2_sftp_seek64(s.sftp_handle, total_size - 1);
> r2 = libssh2_sftp_write(s.sftp_handle, c, 1);
> if (r2 < 0) {
> sftp_error_setg(errp, &s, "truncate failed");
> @@ -1108,7 +1111,7 @@ static int ssh_write(BDRVSSHState *s, BlockDriverState
> *bs,
> * works for me.
> */
> if (r == 0) {
> - ssh_seek(s, offset + written, SSH_SEEK_WRITE|SSH_SEEK_FORCE);
> + ssh_seek(s, offset + written, SSH_SEEK_WRITE | SSH_SEEK_FORCE);
> co_yield(s, bs);
> goto again;
> }
> @@ -1122,8 +1125,9 @@ static int ssh_write(BDRVSSHState *s, BlockDriverState
> *bs,
> end_of_vec = i->iov_base + i->iov_len;
> }
>
> - if (offset + written > s->attrs.filesize)
> + if (offset + written > s->attrs.filesize) {
> s->attrs.filesize = offset + written;
> + }
> }
>
> return 0;
> --
> 2.9.5
--
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
virt-df lists disk usage of guests without needing to install any
software inside the virtual machine. Supports Linux and Windows.
http://people.redhat.com/~rjones/virt-df/
- [Qemu-devel] [PATCH v3 0/7] Code cleanup and minor fixes (for 2.11), Jeff Cody, 2017/11/07
- [Qemu-devel] [PATCH v3 3/7] block/sheepdog: remove spurious NULL check, Jeff Cody, 2017/11/07
- [Qemu-devel] [PATCH v3 1/7] block/ssh: don't call libssh2_init() in block_init(), Jeff Cody, 2017/11/07
- [Qemu-devel] [PATCH v3 2/7] block/ssh: make compliant with coding guidelines, Jeff Cody, 2017/11/07
- [Qemu-devel] [PATCH v3 4/7] block/sheepdog: code beautification, Jeff Cody, 2017/11/07
- [Qemu-devel] [PATCH v3 7/7] block/curl: code cleanup to comply with coding style, Jeff Cody, 2017/11/07
- Re: [Qemu-devel] [Qemu-block] [PATCH v3 7/7] block/curl: code cleanup to comply with coding style, Paolo Bonzini, 2017/11/08
- Re: [Qemu-devel] [PATCH v3 7/7] block/curl: code cleanup to comply with coding style, Richard W.M. Jones, 2017/11/08
[Qemu-devel] [PATCH v3 6/7] block/curl: fix minor memory leaks, Jeff Cody, 2017/11/07