d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Set ses_ictx and ses_octx to NULL after freeing them, so we won't free

Browse Source 
them twice.
This is possible for example in situation when session is used in
authentication context, then freed and then used in encryption context
and freed - in encryption context ses_ictx and ses_octx are not touched
at newsession time, but padlock_freesession could still try to free them
when they are not NULL.
This commit is contained in:
Pawel Jakub Dawidek 2006-07-22 10:04:47 +00:00
parent 87b4dfd5b2
commit 5362e27b6d
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
sys/crypto/via/padlock.c
View File

@ -489,10 +489,12 @@ padlock_freesession(void *arg __unused, uint64_t tid)
if (ses->ses_ictx != NULL) {
bzero(ses->ses_ictx, sizeof(ses->ses_ictx));
free(ses->ses_ictx, M_CRYPTO_DATA);
ses->ses_ictx = NULL;
}
if (ses->ses_octx != NULL) {
bzero(ses->ses_octx, sizeof(ses->ses_octx));
free(ses->ses_octx, M_CRYPTO_DATA);
ses->ses_octx = NULL;
}
bzero(ses, sizeof(ses));
ses->ses_used = 0;