diff options
author | Andreas Schneider <asn@cryptomilk.org> | 2020-06-03 10:04:09 +0200 |
---|---|---|
committer | Andreas Schneider <asn@cryptomilk.org> | 2020-06-08 09:15:20 +0200 |
commit | e631ebb3e2247dd25e9678e6827c20dc73b73238 (patch) | |
tree | e63d38c1b04ec1caf5bc76fbc0e583a08077867d | |
parent | dcc0b9d7aa092bd4bc9f50273e40a3777f9f8c5e (diff) | |
download | libssh-e631ebb3e2247dd25e9678e6827c20dc73b73238.tar.gz libssh-e631ebb3e2247dd25e9678e6827c20dc73b73238.tar.xz libssh-e631ebb3e2247dd25e9678e6827c20dc73b73238.zip |
sftpserver: Add missing NULL check for ssh_buffer_new()
Thanks to Ramin Farajpour Cami for spotting this.
Fixes T232
Signed-off-by: Andreas Schneider <asn@cryptomilk.org>
Reviewed-by: Anderson Toshiyuki Sasaki <ansasaki@redhat.com>
Reviewed-by: Jakub Jelen <jjelen@redhat.com>
(cherry picked from commit 533d881b0f4b24c72b35ecc97fa35d295d063e53)
-rw-r--r-- | src/sftpserver.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/src/sftpserver.c b/src/sftpserver.c index 5a2110e5..b639a2ce 100644 --- a/src/sftpserver.c +++ b/src/sftpserver.c @@ -67,6 +67,12 @@ sftp_client_message sftp_get_client_message(sftp_session sftp) { /* take a copy of the whole packet */ msg->complete_message = ssh_buffer_new(); + if (msg->complete_message == NULL) { + ssh_set_error_oom(session); + sftp_client_message_free(msg); + return NULL; + } + ssh_buffer_add_data(msg->complete_message, ssh_buffer_get(payload), ssh_buffer_get_len(payload)); |