Clobber items before freeing them.
Sponsored by: DARPA, NAI Labs git-svn-id: svn+ssh://svn.openpam.org/svn/openpam/trunk@17 185d5e19-27fe-0310-9dcf-9bff6b9f3609
This commit is contained in:
parent
2a23af0e82
commit
298b19ac93
|
@ -55,11 +55,13 @@ pam_set_item(pam_handle_t *pamh,
|
||||||
int item_type,
|
int item_type,
|
||||||
const void *item)
|
const void *item)
|
||||||
{
|
{
|
||||||
void *tmp;
|
void **slot, *tmp;
|
||||||
|
size_t size;
|
||||||
|
|
||||||
if (pamh == NULL)
|
if (pamh == NULL)
|
||||||
return (PAM_SYSTEM_ERR);
|
return (PAM_SYSTEM_ERR);
|
||||||
|
|
||||||
|
slot = &pamh->item[item_type];
|
||||||
switch (item_type) {
|
switch (item_type) {
|
||||||
case PAM_SERVICE:
|
case PAM_SERVICE:
|
||||||
case PAM_USER:
|
case PAM_USER:
|
||||||
|
@ -70,26 +72,24 @@ pam_set_item(pam_handle_t *pamh,
|
||||||
case PAM_RUSER:
|
case PAM_RUSER:
|
||||||
case PAM_USER_PROMPT:
|
case PAM_USER_PROMPT:
|
||||||
case PAM_AUTHTOK_PROMPT:
|
case PAM_AUTHTOK_PROMPT:
|
||||||
if (item == NULL) {
|
size = strlen(*slot) + 1;
|
||||||
tmp = NULL;
|
if (item != NULL)
|
||||||
break;
|
tmp = strdup(item);
|
||||||
}
|
|
||||||
if ((tmp = strdup(item)) == NULL)
|
|
||||||
return (PAM_BUF_ERR);
|
|
||||||
break;
|
break;
|
||||||
case PAM_CONV:
|
case PAM_CONV:
|
||||||
if (item == NULL) {
|
size = sizeof(struct pam_conv);
|
||||||
tmp = NULL;
|
if (item != NULL)
|
||||||
break;
|
tmp = malloc(size);
|
||||||
}
|
|
||||||
if ((tmp = malloc(sizeof(struct pam_conv))) == NULL)
|
|
||||||
return (PAM_BUF_ERR);
|
|
||||||
memcpy(tmp, item, sizeof(struct pam_conv));
|
|
||||||
break;
|
break;
|
||||||
default:
|
default:
|
||||||
return (PAM_SYSTEM_ERR);
|
return (PAM_SYSTEM_ERR);
|
||||||
}
|
}
|
||||||
free(pamh->item[item_type]);
|
if (item != NULL && tmp == NULL)
|
||||||
pamh->item[item_type] = tmp;
|
return (PAM_BUF_ERR);
|
||||||
|
if (*slot != NULL) {
|
||||||
|
memset(*slot, 0xd0, size);
|
||||||
|
free(*slot);
|
||||||
|
}
|
||||||
|
*slot = tmp;
|
||||||
return (PAM_SUCCESS);
|
return (PAM_SUCCESS);
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue