From 62f013ae968eba77b0ee9de3c0c44a3f576e8418 Mon Sep 17 00:00:00 2001
From: Jakub Jelen <jjelen@redhat.com>
Date: Wed, 28 Jun 2023 10:18:04 +0200
Subject: [PATCH] sftpserver: Check return value ssh_buffer_get_u32

CID 1513157

Thanks coverity

Signed-off-by: Jakub Jelen <jjelen@redhat.com>
Reviewed-by: Norbert Pocs <npocs@redhat.com>
---
 src/sftpserver.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/sftpserver.c b/src/sftpserver.c
index a3760c16..8f15f39e 100644
--- a/src/sftpserver.c
+++ b/src/sftpserver.c
@@ -85,7 +85,10 @@ sftp_make_client_message(sftp_session sftp, sftp_packet packet)
     }
 
     if (msg->type != SSH_FXP_INIT) {
-        ssh_buffer_get_u32(payload, &msg->id);
+        rc = ssh_buffer_get_u32(payload, &msg->id);
+        if (rc != sizeof(uint32_t)) {
+            goto error;
+        }
     }
 
     switch (msg->type) {