Skip to content
Snippets Groups Projects
Commit 4bf3165a authored by Deucе's avatar Deucе :ok_hand_tone4:
Browse files

Fix issue found by coverity.

parent f0883bff
Branches
Tags
1 merge request!455Update branch with changes from master
......@@ -314,10 +314,20 @@ sftp_append64(sftp_tx_pkt_t *pktp, uint64_t u64)
bool
sftp_appendstring(sftp_tx_pkt_t *pktp, sftp_str_t s)
{
uint32_t oldused;
assert(pktp);
sftp_append32(pktp, s->len);
if (!grow_tx(pktp, s->len))
if (*pktp == NULL)
oldused = 0;
else
oldused = (*pktp)->used;
if (!sftp_append32(pktp, s->len))
return false;
if (!grow_tx(pktp, s->len)) {
if (*pktp != NULL)
(*pktp)->used = oldused;
return false;
}
sftp_tx_pkt_t pkt = *pktp;
memcpy(&(&pkt->type)[pkt->used], (uint8_t *)s->c_str, s->len);
pkt->used += s->len;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment