Changeset 187 in openpam


Ignore:
Timestamp:
Dec 12, 2002, 5:06:29 PM (18 years ago)
Author:
Dag-Erling Smørgrav
Message:

Instrument entry and exit points.

Location:
trunk/lib
Files:
44 edited

Legend:

Unmodified
Added
Removed

  • trunk/lib/openpam_borrow_cred.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/openpam_borrow_cred.c#2 $
     34 * $P4: //depot/projects/openpam/lib/openpam_borrow_cred.c#3 $
    3535 */
    3636
    … …  
    5858        int r;
    5959
     60        ENTER();
    6061        if (geteuid() != 0)
    61                 return (PAM_PERM_DENIED);
     62                RETURNC(PAM_PERM_DENIED);
    6263        scred = calloc(1, sizeof *scred);
    6364        if (scred == NULL)
    64                 return (PAM_BUF_ERR);
     65                RETURNC(PAM_BUF_ERR);
    6566        scred->euid = geteuid();
    6667        scred->egid = getegid();
    … …  
    6869        if (r == -1) {
    6970                free(scred);
    70                 return (PAM_SYSTEM_ERR);
     71                RETURNC(PAM_SYSTEM_ERR);
    7172        }
    7273        scred->ngroups = r;
    … …  
    7475        if (r != PAM_SUCCESS) {
    7576                free(scred);
    76                 return (r);
     77                RETURNC(r);
    7778        }
    7879        if (initgroups(pwd->pw_name, pwd->pw_gid) == -1 ||
    7980              setegid(pwd->pw_gid) == -1 || seteuid(pwd->pw_uid) == -1) {
    8081                openpam_restore_cred(pamh);
    81                 return (PAM_SYSTEM_ERR);
     82                RETURNC(PAM_SYSTEM_ERR);
    8283        }
    83         return (PAM_SUCCESS);
     84        RETURNC(PAM_SUCCESS);
    8485}
    8586

  • trunk/lib/openpam_findenv.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/openpam_findenv.c#8 $
     34 * $P4: //depot/projects/openpam/lib/openpam_findenv.c#9 $
    3535 */
    3636
    … …  
    5454        int i;
    5555
     56        ENTER();
    5657        if (pamh == NULL)
    57                 return (-1);
    58 
     58                RETURNI(-1);
    5959        for (i = 0; i < pamh->env_count; ++i)
    6060                if (strncmp(pamh->env[i], name, len) == 0 &&
    6161                    pamh->env[i][len] == '=')
    62                         return (i);
    63         return (-1);
     62                        RETURNI(i);
     63        RETURNI(-1);
    6464}
    6565

  • trunk/lib/openpam_free_data.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/openpam_free_data.c#2 $
     34 * $P4: //depot/projects/openpam/lib/openpam_free_data.c#3 $
    3535 */
    3636
    … …  
    5151openpam_free_data(pam_handle_t *pamh, void *data, int status)
    5252{
    53         /* silence compiler warnings */
    54         pamh = pamh;
    55         status = status;
     53
     54        ENTER();
     55        (void)pamh;
     56        (void)status;
    5657        free(data);
     58        RETURNV();
    5759}
    5860

  • trunk/lib/openpam_get_option.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/openpam_get_option.c#4 $
     34 * $P4: //depot/projects/openpam/lib/openpam_get_option.c#5 $
    3535 */
    3636
    … …  
    5858        int i;
    5959
     60        ENTER();
    6061        if (pamh == NULL || pamh->current == NULL || option == NULL)
    61                 return (NULL);
     62                RETURNS(NULL);
    6263        cur = pamh->current;
    6364        len = strlen(option);
    … …  
    6566                if (strncmp(cur->optv[i], option, len) == 0) {
    6667                        if (cur->optv[i][len] == '\0')
    67                                 return (&cur->optv[i][len]);
     68                                RETURNS(&cur->optv[i][len]);
    6869                        else if (cur->optv[i][len] == '=')
    69                                 return (&cur->optv[i][len + 1]);
     70                                RETURNS(&cur->optv[i][len + 1]);
    7071                }
    7172        }
    72         return (NULL);
     73        RETURNS(NULL);
    7374}
    7475

  • trunk/lib/openpam_log.c

    r175 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/openpam_log.c#16 $
     34 * $P4: //depot/projects/openpam/lib/openpam_log.c#17 $
    3535 */
    3636
    … …  
    6565#ifndef DEBUG
    6666                return;
    67 #endif
     67#else
    6868                priority = LOG_DEBUG;
    6969                break;
     70#endif
    7071        case PAM_LOG_VERBOSE:
    7172                priority = LOG_INFO;
    … …  
    110111#ifndef DEBUG
    111112                return;
    112 #endif
     113#else
    113114                priority = LOG_DEBUG;
    114115                break;
     116#endif
    115117        case PAM_LOG_VERBOSE:
    116118                priority = LOG_INFO;

  • trunk/lib/openpam_nullconv.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/openpam_nullconv.c#3 $
     34 * $P4: //depot/projects/openpam/lib/openpam_nullconv.c#4 $
    3535 */
    3636
    … …  
    3838
    3939#include <security/pam_appl.h>
    40 #include <security/openpam.h>
     40
     41#include "openpam_impl.h"
    4142
    4243/*
    … …  
    5354{
    5455
     56        ENTER();
    5557        (void)n;
    5658        (void)msg;
    5759        (void)resp;
    5860        (void)data;
    59         return (PAM_CONV_ERR);
     61        RETURNC(PAM_CONV_ERR);
    6062}
    6163

  • trunk/lib/openpam_restore_cred.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/openpam_restore_cred.c#2 $
     34 * $P4: //depot/projects/openpam/lib/openpam_restore_cred.c#3 $
    3535 */
    3636
    … …  
    5757        int r;
    5858
     59        ENTER();
    5960        r = pam_get_data(pamh, PAM_SAVED_CRED, (const void **)&scred);
    6061        if (r != PAM_SUCCESS)
    61                 return (r);
     62                RETURNC(r);
    6263        if (scred == NULL)
    63                 return (PAM_SYSTEM_ERR);
     64                RETURNC(PAM_SYSTEM_ERR);
    6465        if (seteuid(scred->euid) == -1 ||
    6566            setgroups(scred->ngroups, scred->groups) == -1 ||
    6667            setegid(scred->egid) == -1)
    67                 return (PAM_SYSTEM_ERR);
     68                RETURNC(PAM_SYSTEM_ERR);
    6869        pam_set_data(pamh, PAM_SAVED_CRED, NULL, NULL);
    69         return (PAM_SUCCESS);
     70        RETURNC(PAM_SUCCESS);
    7071}
    7172

  • trunk/lib/openpam_set_option.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/openpam_set_option.c#5 $
     34 * $P4: //depot/projects/openpam/lib/openpam_set_option.c#6 $
    3535 */
    3636
    … …  
    6262        int i;
    6363
     64        ENTER();
    6465        if (pamh == NULL || pamh->current == NULL || option == NULL)
    65                 return (PAM_SYSTEM_ERR);
     66                RETURNC(PAM_SYSTEM_ERR);
    6667        cur = pamh->current;
    6768        for (len = 0; option[len] != '\0'; ++len)
    … …  
    7677                /* remove */
    7778                if (i == cur->optc)
    78                         return (PAM_SUCCESS);
     79                        RETURNC(PAM_SUCCESS);
    7980                for (free(cur->optv[i]); i < cur->optc; ++i)
    8081                        cur->optv[i] = cur->optv[i + 1];
    8182                cur->optv[i] = NULL;
    82                 return (PAM_SUCCESS);
     83                RETURNC(PAM_SUCCESS);
    8384        }
    8485        if ((opt = malloc(len + strlen(value) + 2)) == NULL)
    85                 return (PAM_BUF_ERR);
     86                RETURNC(PAM_BUF_ERR);
    8687        sprintf(opt, "%.*s=%s", (int)len, option, value);
    8788        if (i == cur->optc) {
    … …  
    9091                if (optv == NULL) {
    9192                        free(opt);
    92                         return (PAM_BUF_ERR);
     93                        RETURNC(PAM_BUF_ERR);
    9394                }
    9495                optv[i] = opt;
    … …  
    101102                cur->optv[i] = opt;
    102103        }
    103         return (PAM_SUCCESS);
     104        RETURNC(PAM_SUCCESS);
    104105}
    105106

  • trunk/lib/openpam_ttyconv.c

    r170 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/openpam_ttyconv.c#13 $
     34 * $P4: //depot/projects/openpam/lib/openpam_ttyconv.c#14 $
    3535 */
    3636
    … …  
    4747
    4848#include <security/pam_appl.h>
    49 #include <security/openpam.h>
     49
     50#include "openpam_impl.h"
    5051
    5152int openpam_ttyconv_timeout = 0;
    … …  
    140141        int i;
    141142
    142         data = data;
     143        ENTER();
     144        (void)data;
    143145        if (n <= 0 || n > PAM_MAX_NUM_MSG)
    144                 return (PAM_CONV_ERR);
     146                RETURNC(PAM_CONV_ERR);
    145147        if ((*resp = calloc(n, sizeof **resp)) == NULL)
    146                 return (PAM_BUF_ERR);
     148                RETURNC(PAM_BUF_ERR);
    147149        for (i = 0; i < n; ++i) {
    148150                resp[i]->resp_retcode = 0;
    … …  
    175177                }
    176178        }
    177         return (PAM_SUCCESS);
     179        RETURNC(PAM_SUCCESS);
    178180 fail:
    179181        while (i)
    … …  
    181183        free(*resp);
    182184        *resp = NULL;
    183         return (PAM_CONV_ERR);
     185        RETURNC(PAM_CONV_ERR);
    184186}
    185187

  • trunk/lib/pam_acct_mgmt.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_acct_mgmt.c#9 $
     34 * $P4: //depot/projects/openpam/lib/pam_acct_mgmt.c#10 $
    3535 */
    3636
    … …  
    5252        int flags)
    5353{
     54        int pam_err;
    5455
    55         return (openpam_dispatch(pamh, PAM_SM_ACCT_MGMT, flags));
     56        ENTER();
     57        pam_err = openpam_dispatch(pamh, PAM_SM_ACCT_MGMT, flags);
     58        RETURNC(pam_err);
    5659}
    5760

  • trunk/lib/pam_authenticate.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_authenticate.c#11 $
     34 * $P4: //depot/projects/openpam/lib/pam_authenticate.c#12 $
    3535 */
    3636
    … …  
    5454        int pam_err;
    5555
     56        ENTER();
    5657        if (flags & ~(PAM_SILENT|PAM_DISALLOW_NULL_AUTHTOK))
    57                 return (PAM_SYMBOL_ERR);
     58                RETURNC(PAM_SYMBOL_ERR);
    5859        pam_err = openpam_dispatch(pamh, PAM_SM_AUTHENTICATE, flags);
    5960        pam_set_item(pamh, PAM_AUTHTOK, NULL);
    60         return (pam_err);
     61        RETURNC(pam_err);
    6162}
    6263

  • trunk/lib/pam_authenticate_secondary.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_authenticate_secondary.c#6 $
     34 * $P4: //depot/projects/openpam/lib/pam_authenticate_secondary.c#7 $
    3535 */
    3636
    … …  
    5454{
    5555
    56         return (PAM_SYSTEM_ERR);
     56        ENTER();
     57        RETURNC(PAM_SYSTEM_ERR);
    5758}
    5859

  • trunk/lib/pam_chauthtok.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_chauthtok.c#12 $
     34 * $P4: //depot/projects/openpam/lib/pam_chauthtok.c#13 $
    3535 */
    3636
    … …  
    5454        int pam_err;
    5555
     56        ENTER();
    5657        if (flags & ~(PAM_SILENT|PAM_CHANGE_EXPIRED_AUTHTOK))
    57                 return (PAM_SYMBOL_ERR);
     58                RETURNC(PAM_SYMBOL_ERR);
    5859        pam_err = openpam_dispatch(pamh, PAM_SM_CHAUTHTOK,
    5960            flags | PAM_PRELIM_CHECK);
    … …  
    6364        pam_set_item(pamh, PAM_OLDAUTHTOK, NULL);
    6465        pam_set_item(pamh, PAM_AUTHTOK, NULL);
    65         return (pam_err);
     66        RETURNC(pam_err);
    6667}
    6768

  • trunk/lib/pam_close_session.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_close_session.c#9 $
     34 * $P4: //depot/projects/openpam/lib/pam_close_session.c#10 $
    3535 */
    3636
    … …  
    5353{
    5454
     55        ENTER();
    5556        if (flags & ~(PAM_SILENT))
    56                 return (PAM_SYMBOL_ERR);
    57         return (openpam_dispatch(pamh, PAM_SM_CLOSE_SESSION, flags));
     57                RETURNC(PAM_SYMBOL_ERR);
     58        RETURNC(openpam_dispatch(pamh, PAM_SM_CLOSE_SESSION, flags));
    5859}
    5960

  • trunk/lib/pam_end.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_end.c#10 $
     34 * $P4: //depot/projects/openpam/lib/pam_end.c#11 $
    3535 */
    3636
    … …  
    5555        int i;
    5656
     57        ENTER();
    5758        if (pamh == NULL)
    58                 return (PAM_SYSTEM_ERR);
     59                RETURNC(PAM_SYSTEM_ERR);
    5960
    6061        /* clear module data */
    … …  
    8182        free(pamh);
    8283
    83         return (PAM_SUCCESS);
     84        RETURNC(PAM_SUCCESS);
    8485}
    8586

  • trunk/lib/pam_get_authtok.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_get_authtok.c#19 $
     34 * $P4: //depot/projects/openpam/lib/pam_get_authtok.c#20 $
    3535 */
    3636
    … …  
    6666        int pitem, r, style, twice;
    6767
     68        ENTER();
    6869        if (pamh == NULL || authtok == NULL)
    69                 return (PAM_SYSTEM_ERR);
    70 
     70                RETURNC(PAM_SYSTEM_ERR);
    7171        *authtok = NULL;
    7272        twice = 0;
    … …  
    8787                break;
    8888        default:
    89                 return (PAM_SYMBOL_ERR);
     89                RETURNC(PAM_SYMBOL_ERR);
    9090        }
    91 
    9291        if (openpam_get_option(pamh, "try_first_pass") ||
    9392            openpam_get_option(pamh, "use_first_pass")) {
    9493                r = pam_get_item(pamh, item, (const void **)authtok);
    9594                if (r == PAM_SUCCESS && *authtok != NULL)
    96                         return (PAM_SUCCESS);
     95                        RETURNC(PAM_SUCCESS);
    9796                else if (openpam_get_option(pamh, "use_first_pass"))
    98                         return (r == PAM_SUCCESS ? PAM_AUTH_ERR : r);
     97                        RETURNC(r == PAM_SUCCESS ? PAM_AUTH_ERR : r);
    9998        }
    10099        if (prompt == NULL) {
    … …  
    107106        r = pam_prompt(pamh, style, &resp, "%s", prompt);
    108107        if (r != PAM_SUCCESS)
    109                 return (r);
     108                RETURNC(r);
    110109        if (twice) {
    111110                r = pam_prompt(pamh, style, &resp2, "Retype %s", prompt);
    112111                if (r != PAM_SUCCESS) {
    113112                        free(resp);
    114                         return (r);
     113                        RETURNC(r);
    115114                }
    116115                if (strcmp(resp, resp2) != 0) {
    … …  
    121120        }
    122121        if (resp == NULL)
    123                 return (PAM_TRY_AGAIN);
     122                RETURNC(PAM_TRY_AGAIN);
    124123        r = pam_set_item(pamh, item, resp);
    125124        free(resp);
    126125        if (r != PAM_SUCCESS)
    127                 return (r);
    128         return (pam_get_item(pamh, item, (const void **)authtok));
     126                RETURNC(r);
     127        RETURNC(pam_get_item(pamh, item, (const void **)authtok));
    129128}
    130129

  • trunk/lib/pam_get_data.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_get_data.c#8 $
     34 * $P4: //depot/projects/openpam/lib/pam_get_data.c#9 $
    3535 */
    3636
    … …  
    5555        pam_data_t *dp;
    5656
     57        ENTER();
    5758        if (pamh == NULL)
    58                 return (PAM_SYSTEM_ERR);
    59 
     59                RETURNC(PAM_SYSTEM_ERR);
    6060        for (dp = pamh->module_data; dp != NULL; dp = dp->next)
    6161                if (strcmp(dp->name, module_data_name) == 0) {
    6262                        *data = dp->data;
    63                         return (PAM_SUCCESS);
     63                        RETURNC(PAM_SUCCESS);
    6464                }
    65 
    66         return (PAM_NO_MODULE_DATA);
     65        RETURNC(PAM_NO_MODULE_DATA);
    6766}
    6867

  • trunk/lib/pam_get_item.c

    r164 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_get_item.c#13 $
     34 * $P4: //depot/projects/openpam/lib/pam_get_item.c#14 $
    3535 */
    3636
    … …  
    5353        const void **item)
    5454{
     55
     56        ENTER();
    5557        if (pamh == NULL)
    56                 return (PAM_SYSTEM_ERR);
    57 
     58                RETURNC(PAM_SYSTEM_ERR);
    5859        switch (item_type) {
    5960        case PAM_SERVICE:
    … …  
    7071        case PAM_REPOSITORY:
    7172                *item = pamh->item[item_type];
    72                 return (PAM_SUCCESS);
     73                RETURNC(PAM_SUCCESS);
    7374        default:
    74                 return (PAM_SYMBOL_ERR);
     75                RETURNC(PAM_SYMBOL_ERR);
    7576        }
    7677}

  • trunk/lib/pam_get_mapped_authtok.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_get_mapped_authtok.c#6 $
     34 * $P4: //depot/projects/openpam/lib/pam_get_mapped_authtok.c#7 $
    3535 */
    3636
    … …  
    5353{
    5454
    55         return (PAM_SYSTEM_ERR);
     55        ENTER();
     56        RETURNC(PAM_SYSTEM_ERR);
    5657}
    5758

  • trunk/lib/pam_get_mapped_username.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_get_mapped_username.c#6 $
     34 * $P4: //depot/projects/openpam/lib/pam_get_mapped_username.c#7 $
    3535 */
    3636
    … …  
    5454{
    5555
    56         return (PAM_SYSTEM_ERR);
     56        ENTER();
     57        RETURNC(PAM_SYSTEM_ERR);
    5758}
    5859

  • trunk/lib/pam_get_user.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_get_user.c#12 $
     34 * $P4: //depot/projects/openpam/lib/pam_get_user.c#13 $
    3535 */
    3636
    … …  
    6161        int r;
    6262
     63        ENTER();
    6364        if (pamh == NULL || user == NULL)
    64                 return (PAM_SYSTEM_ERR);
    65 
     65                RETURNC(PAM_SYSTEM_ERR);
    6666        r = pam_get_item(pamh, PAM_USER, (const void **)user);
    6767        if (r == PAM_SUCCESS)
    68                 return (PAM_SUCCESS);
     68                RETURNC(PAM_SUCCESS);
    6969        if (prompt == NULL) {
    7070                r = pam_get_item(pamh, PAM_USER_PROMPT, (const void **)&prompt);
    … …  
    7474        r = pam_prompt(pamh, PAM_PROMPT_ECHO_ON, &resp, "%s", prompt);
    7575        if (r != PAM_SUCCESS)
    76                 return (r);
     76                RETURNC(r);
    7777        r = pam_set_item(pamh, PAM_USER, resp);
    7878        free(resp);
    7979        if (r != PAM_SUCCESS)
    80                 return (r);
    81         return (pam_get_item(pamh, PAM_USER, (const void **)user));
     80                RETURNC(r);
     81        RETURNC(pam_get_item(pamh, PAM_USER, (const void **)user));
    8282}
    8383

  • trunk/lib/pam_getenv.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_getenv.c#7 $
     34 * $P4: //depot/projects/openpam/lib/pam_getenv.c#8 $
    3535 */
    3636
    … …  
    5555        int i;
    5656
     57        ENTER();
    5758        if (pamh == NULL)
    58                 return (NULL);
    59 
    60         /* sanity checks */
     59                RETURNC(NULL);
    6160        if (name == NULL || strchr(name, '=') != NULL)
    62                 return (NULL);
    63 
     61                RETURNC(NULL);
    6462        if ((i = openpam_findenv(pamh, name, strlen(name))) == -1)
    65                 return (NULL);
    66         return (strdup(pamh->env[i]));
     63                RETURNC(NULL);
     64        RETURNC(strdup(pamh->env[i]));
    6765}
    6866

  • trunk/lib/pam_getenvlist.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_getenvlist.c#9 $
     34 * $P4: //depot/projects/openpam/lib/pam_getenvlist.c#10 $
    3535 */
    3636
    … …  
    5555        int i;
    5656
     57        ENTER();
    5758        if (pamh == NULL)
    58                 return (NULL);
    59 
     59                RETURNP(NULL);
    6060        envlist = malloc(sizeof(char *) * (pamh->env_count + 1));
    6161        if (envlist == NULL) {
    6262                openpam_log(PAM_LOG_ERROR, "%s",
    6363                        pam_strerror(pamh, PAM_BUF_ERR));
    64                 return (NULL);
     64                RETURNP(NULL);
    6565        }
    6666        for (i = 0; i < pamh->env_count; ++i) {
    … …  
    7171                        openpam_log(PAM_LOG_ERROR, "%s",
    7272                                pam_strerror(pamh, PAM_BUF_ERR));
    73                         return (NULL);
     73                        RETURNP(NULL);
    7474                }
    7575        }
    7676        envlist[i] = NULL;
    77         openpam_log(PAM_LOG_DEBUG, "returning %d variables\n", pamh->env_count);
    78         return (envlist);
     77        RETURNP(envlist);
    7978}
    8079

  • trunk/lib/pam_open_session.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_open_session.c#9 $
     34 * $P4: //depot/projects/openpam/lib/pam_open_session.c#10 $
    3535 */
    3636
    … …  
    5353{
    5454
     55        ENTER();
    5556        if (flags & ~(PAM_SILENT))
    56                 return (PAM_SYMBOL_ERR);
    57         return (openpam_dispatch(pamh, PAM_SM_OPEN_SESSION, flags));
     57                RETURNC(PAM_SYMBOL_ERR);
     58        RETURNC(openpam_dispatch(pamh, PAM_SM_OPEN_SESSION, flags));
    5859}
    5960

  • trunk/lib/pam_putenv.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_putenv.c#8 $
     34 * $P4: //depot/projects/openpam/lib/pam_putenv.c#9 $
    3535 */
    3636
    … …  
    5656        int i;
    5757
     58        ENTER();
    5859        if (pamh == NULL)
    59                 return (PAM_SYSTEM_ERR);
     60                RETURNC(PAM_SYSTEM_ERR);
    6061
    6162        /* sanity checks */
    6263        if (namevalue == NULL || (p = strchr(namevalue, '=')) == NULL)
    63                 return (PAM_SYSTEM_ERR);
     64                RETURNC(PAM_SYSTEM_ERR);
    6465
    6566        /* see if the variable is already in the environment */
    6667        if ((i = openpam_findenv(pamh, namevalue, p - namevalue)) != -1) {
    6768                if ((p = strdup(namevalue)) == NULL)
    68                         return (PAM_BUF_ERR);
     69                        RETURNC(PAM_BUF_ERR);
    6970                free(pamh->env[i]);
    7071                pamh->env[i] = p;
    71                 return (PAM_SUCCESS);
     72                RETURNC(PAM_SUCCESS);
    7273        }
    7374
    … …  
    7778                    sizeof(char *) * (pamh->env_size * 2 + 1));
    7879                if (env == NULL)
    79                         return (PAM_BUF_ERR);
     80                        RETURNC(PAM_BUF_ERR);
    8081                pamh->env = env;
    8182                pamh->env_size = pamh->env_size * 2 + 1;
    … …  
    8485        /* add the variable at the end */
    8586        if ((pamh->env[pamh->env_count] = strdup(namevalue)) == NULL)
    86                 return (PAM_BUF_ERR);
     87                RETURNC(PAM_BUF_ERR);
    8788        ++pamh->env_count;
    88         return (PAM_SUCCESS);
     89        RETURNC(PAM_SUCCESS);
    8990}
    9091

  • trunk/lib/pam_set_data.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_set_data.c#10 $
     34 * $P4: //depot/projects/openpam/lib/pam_set_data.c#11 $
    3535 */
    3636
    … …  
    5959        pam_data_t *dp;
    6060
     61        ENTER();
    6162        if (pamh == NULL)
    62                 return (PAM_SYSTEM_ERR);
    63 
     63                RETURNC(PAM_SYSTEM_ERR);
    6464        for (dp = pamh->module_data; dp != NULL; dp = dp->next) {
    6565                if (strcmp(dp->name, module_data_name) == 0) {
    … …  
    6868                        dp->data = data;
    6969                        dp->cleanup = cleanup;
    70                         return (PAM_SUCCESS);
     70                        RETURNC(PAM_SUCCESS);
    7171                }
    7272        }
    73 
    7473        if ((dp = malloc(sizeof *dp)) == NULL)
    75                 return (PAM_BUF_ERR);
     74                RETURNC(PAM_BUF_ERR);
    7675        if ((dp->name = strdup(module_data_name)) == NULL) {
    7776                free(dp);
    78                 return (PAM_BUF_ERR);
     77                RETURNC(PAM_BUF_ERR);
    7978        }
    8079        dp->data = data;
    … …  
    8281        dp->next = pamh->module_data;
    8382        pamh->module_data = dp;
    84         return (PAM_SUCCESS);
     83        RETURNC(PAM_SUCCESS);
    8584}
    8685

  • trunk/lib/pam_set_item.c

    r164 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_set_item.c#15 $
     34 * $P4: //depot/projects/openpam/lib/pam_set_item.c#16 $
    3535 */
    3636
    … …  
    5959        size_t nsize, osize;
    6060
     61        ENTER();
    6162        if (pamh == NULL)
    62                 return (PAM_SYSTEM_ERR);
    63 
     63                RETURNC(PAM_SYSTEM_ERR);
    6464        slot = &pamh->item[item_type];
    6565        switch (item_type) {
    … …  
    8686                break;
    8787        default:
    88                 return (PAM_SYMBOL_ERR);
     88                RETURNC(PAM_SYMBOL_ERR);
    8989        }
    9090        if (*slot != NULL) {
    … …  
    9494        if (item != NULL) {
    9595                if ((tmp = malloc(nsize)) == NULL)
    96                         return (PAM_BUF_ERR);
     96                        RETURNC(PAM_BUF_ERR);
    9797                memcpy(tmp, item, nsize);
    9898        } else {
    … …  
    100100        }
    101101        *slot = tmp;
    102         return (PAM_SUCCESS);
     102        RETURNC(PAM_SUCCESS);
    103103}
    104104

  • trunk/lib/pam_set_mapped_authtok.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_set_mapped_authtok.c#6 $
     34 * $P4: //depot/projects/openpam/lib/pam_set_mapped_authtok.c#7 $
    3535 */
    3636
    … …  
    5353{
    5454
    55         return (PAM_SYSTEM_ERR);
     55        ENTER();
     56        RETURNC(PAM_SYSTEM_ERR);
    5657}
    5758

  • trunk/lib/pam_set_mapped_username.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_set_mapped_username.c#6 $
     34 * $P4: //depot/projects/openpam/lib/pam_set_mapped_username.c#7 $
    3535 */
    3636
    … …  
    5454{
    5555
    56         return (PAM_SYSTEM_ERR);
     56        ENTER();
     57        RETURNC(PAM_SYSTEM_ERR);
    5758}
    5859

  • trunk/lib/pam_setcred.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_setcred.c#10 $
     34 * $P4: //depot/projects/openpam/lib/pam_setcred.c#11 $
    3535 */
    3636
    … …  
    5353{
    5454
     55        ENTER();
    5556        if (flags & ~(PAM_SILENT|PAM_ESTABLISH_CRED|PAM_DELETE_CRED|
    5657                PAM_REINITIALIZE_CRED|PAM_REFRESH_CRED))
    57                 return (PAM_SYMBOL_ERR);
     58                RETURNC(PAM_SYMBOL_ERR);
    5859        /* XXX enforce exclusivity */
    59         return (openpam_dispatch(pamh, PAM_SM_SETCRED, flags));
     60        RETURNC(openpam_dispatch(pamh, PAM_SM_SETCRED, flags));
    6061}
    6162

  • trunk/lib/pam_setenv.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_setenv.c#7 $
     34 * $P4: //depot/projects/openpam/lib/pam_setenv.c#8 $
    3535 */
    3636
    … …  
    5959        int r;
    6060
     61        ENTER();
    6162        if (pamh == NULL)
    62                 return (PAM_SYSTEM_ERR);
     63                RETURNC(PAM_SYSTEM_ERR);
    6364
    6465        /* sanity checks */
    6566        if (name == NULL || value == NULL || strchr(name, '=') != NULL)
    66                 return (PAM_SYSTEM_ERR);
     67                RETURNC(PAM_SYSTEM_ERR);
    6768
    6869        /* is it already there? */
    6970        if (!overwrite && openpam_findenv(pamh, name, strlen(name)) != -1)
    70                 return (PAM_SUCCESS);
     71                RETURNC(PAM_SUCCESS);
    7172
    7273        /* set it... */
    7374        if ((env = malloc(strlen(name) + strlen(value) + 2)) == NULL)
    74                 return (PAM_BUF_ERR);
     75                RETURNC(PAM_BUF_ERR);
    7576        sprintf(env, "%s=%s", name, value);
    7677        r = pam_putenv(pamh, env);
    7778        free(env);
    78         return (r);
     79        RETURNC(r);
    7980}
    8081

  • trunk/lib/pam_sm_acct_mgmt.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_sm_acct_mgmt.c#4 $
     34 * $P4: //depot/projects/openpam/lib/pam_sm_acct_mgmt.c#5 $
    3535 */
    3636
    … …  
    5454{
    5555
    56         return (PAM_SYSTEM_ERR);
     56        ENTER();
     57        RETURNC(PAM_SYSTEM_ERR);
    5758}
    5859

  • trunk/lib/pam_sm_authenticate.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_sm_authenticate.c#4 $
     34 * $P4: //depot/projects/openpam/lib/pam_sm_authenticate.c#5 $
    3535 */
    3636
    … …  
    5454{
    5555
    56         return (PAM_SYSTEM_ERR);
     56        ENTER();
     57        RETURNC(PAM_SYSTEM_ERR);
    5758}
    5859

  • trunk/lib/pam_sm_authenticate_secondary.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_sm_authenticate_secondary.c#4 $
     34 * $P4: //depot/projects/openpam/lib/pam_sm_authenticate_secondary.c#5 $
    3535 */
    3636
    … …  
    5959{
    6060
    61         return (PAM_SYSTEM_ERR);
     61        ENTER();
     62        RETURNC(PAM_SYSTEM_ERR);
    6263}
    6364

  • trunk/lib/pam_sm_chauthtok.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_sm_chauthtok.c#5 $
     34 * $P4: //depot/projects/openpam/lib/pam_sm_chauthtok.c#6 $
    3535 */
    3636
    … …  
    5454{
    5555
    56         return (PAM_SYSTEM_ERR);
     56        ENTER();
     57        RETURNC(PAM_SYSTEM_ERR);
    5758}
    5859

  • trunk/lib/pam_sm_close_session.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_sm_close_session.c#4 $
     34 * $P4: //depot/projects/openpam/lib/pam_sm_close_session.c#5 $
    3535 */
    3636
    … …  
    5454{
    5555
    56         return (PAM_SYSTEM_ERR);
     56        ENTER();
     57        RETURNC(PAM_SYSTEM_ERR);
    5758}
    5859

  • trunk/lib/pam_sm_get_mapped_authtok.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_sm_get_mapped_authtok.c#4 $
     34 * $P4: //depot/projects/openpam/lib/pam_sm_get_mapped_authtok.c#5 $
    3535 */
    3636
    … …  
    5858{
    5959
    60         return (PAM_SYSTEM_ERR);
     60        ENTER();
     61        RETURNC(PAM_SYSTEM_ERR);
    6162}
    6263

  • trunk/lib/pam_sm_get_mapped_username.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_sm_get_mapped_username.c#4 $
     34 * $P4: //depot/projects/openpam/lib/pam_sm_get_mapped_username.c#5 $
    3535 */
    3636
    … …  
    5959{
    6060
    61         return (PAM_SYSTEM_ERR);
     61        ENTER();
     62        RETURNC(PAM_SYSTEM_ERR);
    6263}
    6364

  • trunk/lib/pam_sm_open_session.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_sm_open_session.c#4 $
     34 * $P4: //depot/projects/openpam/lib/pam_sm_open_session.c#5 $
    3535 */
    3636
    … …  
    5454{
    5555
    56         return (PAM_SYSTEM_ERR);
     56        ENTER();
     57        RETURNC(PAM_SYSTEM_ERR);
    5758}
    5859

  • trunk/lib/pam_sm_set_mapped_authtok.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_sm_set_mapped_authtok.c#4 $
     34 * $P4: //depot/projects/openpam/lib/pam_sm_set_mapped_authtok.c#5 $
    3535 */
    3636
    … …  
    5858{
    5959
    60         return (PAM_SYSTEM_ERR);
     60        ENTER();
     61        RETURNC(PAM_SYSTEM_ERR);
    6162}
    6263

  • trunk/lib/pam_sm_set_mapped_username.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_sm_set_mapped_username.c#4 $
     34 * $P4: //depot/projects/openpam/lib/pam_sm_set_mapped_username.c#5 $
    3535 */
    3636
    … …  
    5656{
    5757
    58         return (PAM_SYSTEM_ERR);
     58        ENTER();
     59        RETURNC(PAM_SYSTEM_ERR);
    5960}
    6061

  • trunk/lib/pam_sm_setcred.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_sm_setcred.c#4 $
     34 * $P4: //depot/projects/openpam/lib/pam_sm_setcred.c#5 $
    3535 */
    3636
    … …  
    5454{
    5555
    56         return (PAM_SYSTEM_ERR);
     56        ENTER();
     57        RETURNC(PAM_SYSTEM_ERR);
    5758}
    5859

  • trunk/lib/pam_start.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_start.c#16 $
     34 * $P4: //depot/projects/openpam/lib/pam_start.c#17 $
    3535 */
    3636
    … …  
    5757        int r;
    5858
     59        ENTER();
    5960        if ((ph = calloc(1, sizeof *ph)) == NULL)
    60                 return (PAM_BUF_ERR);
     61                RETURNC(PAM_BUF_ERR);
    6162        if ((r = pam_set_item(ph, PAM_SERVICE, service)) != PAM_SUCCESS)
    6263                goto fail;
    … …  
    7273        *pamh = ph;
    7374        openpam_log(PAM_LOG_DEBUG, "pam_start(\"%s\") succeeded", service);
    74         return (PAM_SUCCESS);
     75        RETURNC(PAM_SUCCESS);
    7576
    7677 fail:
    7778        pam_end(ph, r);
    78         return (r);
     79        RETURNC(r);
    7980}
    8081

  • trunk/lib/pam_vprompt.c

    r157 r187  
    3232 * SUCH DAMAGE.
    3333 *
    34  * $P4: //depot/projects/openpam/lib/pam_vprompt.c#7 $
     34 * $P4: //depot/projects/openpam/lib/pam_vprompt.c#8 $
    3535 */
    3636
    … …  
    4040
    4141#include <security/pam_appl.h>
    42 #include <security/openpam.h>
     42
     43#include "openpam_impl.h"
    4344
    4445/*
    … …  
    6263        int r;
    6364
     65        ENTER();
    6466        r = pam_get_item(pamh, PAM_CONV, (const void **)&conv);
    6567        if (r != PAM_SUCCESS)
    66                 return (r);
     68                RETURNC(r);
    6769        if (conv == NULL) {
    6870                openpam_log(PAM_LOG_ERROR, "no conversation function");
    69                 return (PAM_SYSTEM_ERR);
     71                RETURNC(PAM_SYSTEM_ERR);
    7072        }
    7173        vsnprintf(msgbuf, PAM_MAX_MSG_SIZE, fmt, ap);
    … …  
    7779        *resp = rsp == NULL ? NULL : rsp->resp;
    7880        free(rsp);
    79         return (r);
     81        RETURNC(r);
    8082}
    8183
Note: See TracChangeset for help on using the changeset viewer.