[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 2/3] pki: check ssh_buffer_pack return in ssh_pki_do_sign
[Thread Prev] | [Thread Next]
- Subject: [PATCH 2/3] pki: check ssh_buffer_pack return in ssh_pki_do_sign
- From: Jon Simons <jon@xxxxxxxxxxxxx>
- Reply-to: libssh@xxxxxxxxxx
- Date: Sat, 27 Sep 2014 03:00:49 -0700
- To: libssh@xxxxxxxxxx
From 2e7295b1c234f62184985b75375a05964d82c99e Mon Sep 17 00:00:00 2001
From: Jon Simons <jon@xxxxxxxxxxxxx>
Date: Sat, 27 Sep 2014 02:07:54 -0700
Subject: [PATCH 2/3] pki: check ssh_buffer_pack return in ssh_pki_do_sign
Check the 'ssh_buffer_pack' return in ssh_pki_do_sign for the ED25519 case.
Signed-off-by: Jon Simons <jon@xxxxxxxxxxxxx>
---
src/pki.c | 15 ++++++++++-----
1 file changed, 10 insertions(+), 5 deletions(-)
diff --git a/src/pki.c b/src/pki.c
index cde803e..af472eb 100644
--- a/src/pki.c
+++ b/src/pki.c
@@ -1453,16 +1453,21 @@ ssh_string ssh_pki_do_sign(ssh_session session,
ssh_buffer buf;
buf = ssh_buffer_new();
- if (buf == NULL){
+ if (buf == NULL) {
ssh_string_free(session_id);
return NULL;
}
ssh_buffer_set_secure(buf);
- ssh_buffer_pack(buf,
- "SP",
- session_id,
- buffer_get_rest_len(sigbuf), buffer_get_rest(sigbuf));
+ rc = ssh_buffer_pack(buf,
+ "SP",
+ session_id,
+ buffer_get_rest_len(sigbuf), buffer_get_rest(sigbuf));
+ if (rc != SSH_OK) {
+ ssh_string_free(session_id);
+ ssh_buffer_free(buf);
+ return NULL;
+ }
sig = pki_do_sign(privkey,
ssh_buffer_get_begin(buf),
--
2.1.1.602.g02f248d
Archive administrator: postmaster@lists.cynapses.org