Changeset 18 in openpam for trunk

Timestamp:

Feb 2, 2002, 5:58:58 PM (19 years ago)

Author:

Dag-Erling Smørgrav

Message:

Clear module data.
Clear environment.
Use pam_set_item() rather than free() to clear items so they get
properly clobbered.

Sponsored by: DARPA, NAI Labs

File:

• trunk/lib/pam_end.c (modified) (2 diffs)

trunk/lib/pam_end.c

@@ -51 +51 @@
 int
 pam_end(pam_handle_t *pamh,
-        int status __unused)
+        int status)
 {
         pam_chain_t *module;
+        pam_data_t *dp;
         int i;
 
         if (pamh == NULL)
                 return (PAM_SYSTEM_ERR);
+
+        /* clear module data */
+        while ((dp = pamh->module_data) != NULL) {
+                if (dp->cleanup)
+                        (dp->cleanup)(pamh, dp->data, status);
+                pamh->module_data = dp->next;
+                free(dp->name);
+                free(dp);
+        }
+
+        /* clear environment */
+        while (pamh->env_count)
+                free(pamh->env[--pamh->env_count]);
+        free(pamh->env);
 
         /* clear chains */
@@ -73 +88 @@
         /* clear items */
         for (i = 0; i < PAM_NUM_ITEMS; ++i)
-                free(pamh->item[i]);
+                pam_set_item(pamh, i, NULL);
 
         free(pamh);