diff options
author | Jakub Jelen <jjelen@redhat.com> | 2019-09-24 16:04:02 +0200 |
---|---|---|
committer | Jakub Jelen <jjelen@redhat.com> | 2019-09-24 16:10:33 +0200 |
commit | e72429f205bd1a17062ae95f8850b510d71b2964 (patch) | |
tree | e416d9903a51ae6f54753b61051754f1340947a2 | |
parent | aff7c500d5721e35c998b1b3c78e450fe7ff986d (diff) | |
download | libssh-e72429f205bd1a17062ae95f8850b510d71b2964.tar.gz libssh-e72429f205bd1a17062ae95f8850b510d71b2964.tar.xz libssh-e72429f205bd1a17062ae95f8850b510d71b2964.zip |
buffer: Simplify handling of the return codes
Signed-off-by: Jakub Jelen <jjelen@redhat.com>
Reviewed-by: Andreas Schneider <asn@cryptomilk.org>
-rw-r--r-- | src/buffer.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/src/buffer.c b/src/buffer.c index d7d90d07..a2e6246a 100644 --- a/src/buffer.c +++ b/src/buffer.c @@ -1155,7 +1155,6 @@ int ssh_buffer_unpack_va(struct ssh_buffer_struct *buffer, *o.bignum = NULL; tmp_string = ssh_buffer_get_ssh_string(buffer); if (tmp_string == NULL) { - rc = SSH_ERROR; break; } *o.bignum = ssh_make_string_bn(tmp_string); @@ -1174,14 +1173,12 @@ int ssh_buffer_unpack_va(struct ssh_buffer_struct *buffer, o.cstring = va_arg(ap, char **); *o.cstring = NULL; - rc = ssh_buffer_get_u32(buffer, &u32len); - if (rc != 4){ - rc = SSH_ERROR; + rlen = ssh_buffer_get_u32(buffer, &u32len); + if (rlen != 4){ break; } len = ntohl(u32len); if (len > max_len - 1) { - rc = SSH_ERROR; break; } @@ -1237,7 +1234,6 @@ int ssh_buffer_unpack_va(struct ssh_buffer_struct *buffer, break; default: SSH_LOG(SSH_LOG_WARN, "Invalid buffer format %c", *p); - rc = SSH_ERROR; } if (rc != SSH_OK) { break; |