From c3941698cda5a14727b97450ae00fc95dd19d855 Mon Sep 17 00:00:00 2001 From: Matt Johnston Date: Mon, 12 Feb 2007 10:44:47 +0000 Subject: [PATCH] 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 --- packet.c | 4 ---- process-packet.c | 1 - 2 files changed, 5 deletions(-) diff --git a/packet.c b/packet.c index b2c6174..9e7c67a 100644 --- a/packet.c +++ b/packet.c @@ -446,10 +446,6 @@ void encrypt_packet() { } /* 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_setlen(ses.writepayload, 0); diff --git a/process-packet.c b/process-packet.c index 07fc130..ba39d9f 100644 --- a/process-packet.c +++ b/process-packet.c @@ -119,7 +119,6 @@ void process_packet() { recv_unimplemented(); out: - buf_burn(ses.payload); /* Clear the memory to avoid swapping it out */ buf_free(ses.payload); ses.payload = NULL;