shinys000114/libssh
1
0
Fork 0
mirror of https://git.libssh.org/projects/libssh.git synced 2026-02-04 12:20:42 +09:00
Code Issues Packages Projects Releases Wiki Activity

packet: Use consistent return codes in ssh_packet_hmac_verify()

Browse Source 
Signed-off-by: Andreas Schneider <asn@cryptomilk.org>
Reviewed-by: Jakub Jelen <jjelen@redhat.com>
This commit is contained in:
Andreas Schneider
2022-07-13 16:17:15 +02:00
parent 4a7791b784
commit e27ee9d0a4
1 changed files with 8 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
src/packet_crypt.c
View File

@@ -265,6 +265,7 @@ int ssh_packet_hmac_verify(ssh_session session,
HMACCTX ctx;
size_t hmaclen = DIGEST_MAX_LEN;
uint32_t seq;
int cmp;
/* AEAD types have no mac checking */
if (type == SSH_HMAC_AEAD_POLY1305 ||
@@ -282,7 +283,7 @@ int ssh_packet_hmac_verify(ssh_session session,
hmac_digest_len(type),
type);
if (ctx == NULL) {
return -1;
return SSH_ERROR;
}
seq = htonl(session->recv_seq);
@@ -308,11 +309,12 @@ int ssh_packet_hmac_verify(ssh_session session,
(unsigned char *)&seq,
sizeof(uint32_t));
#endif
if (secure_memcmp(mac,
hmacbuf,
hmaclen) == 0) {
return 0;
cmp = secure_memcmp(mac,
hmacbuf,
hmaclen);
if (cmp == 0) {
return SSH_OK;
}
return -1;
return SSH_ERROR;
}