mirror of
https://github.com/clearml/dropbear
synced 2025-02-07 13:21:15 +00:00
Don't erase every buffer, it's too time consuming.
On systems where it matters encrypted swap should be enabled anyway. --HG-- extra : convert_revision : 6cb7793493d92968e09b5dea21d71ded5811d21f
This commit is contained in:
parent
47e76de56a
commit
c3941698cd
4
packet.c
4
packet.c
@ -446,10 +446,6 @@ void encrypt_packet() {
|
|||||||
}
|
}
|
||||||
|
|
||||||
/* finished with payload */
|
/* finished with payload */
|
||||||
buf_burn(ses.writepayload); /* XXX This is probably a good idea, and isn't
|
|
||||||
_that_ likely to hurt performance too badly.
|
|
||||||
Buffers can have cleartext passwords etc, or
|
|
||||||
other sensitive data */
|
|
||||||
buf_setpos(ses.writepayload, 0);
|
buf_setpos(ses.writepayload, 0);
|
||||||
buf_setlen(ses.writepayload, 0);
|
buf_setlen(ses.writepayload, 0);
|
||||||
|
|
||||||
|
@ -119,7 +119,6 @@ void process_packet() {
|
|||||||
recv_unimplemented();
|
recv_unimplemented();
|
||||||
|
|
||||||
out:
|
out:
|
||||||
buf_burn(ses.payload); /* Clear the memory to avoid swapping it out */
|
|
||||||
buf_free(ses.payload);
|
buf_free(ses.payload);
|
||||||
ses.payload = NULL;
|
ses.payload = NULL;
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user