Changeset 487 in openpam for trunk


Ignore:
Timestamp:
Nov 5, 2011, 8:00:46 PM (7 years ago)
Author:
Dag-Erling Smørgrav
Message:

The count was never used, so ditch it and return plain PAM error codes
instead.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/lib/openpam_configure.c

    r484 r487  
    374374{
    375375        pam_chain_t *this, **next;
    376         int count, lineno;
    377376        pam_facility_t fclt;
    378377        pam_control_t ctlf;
    379378        char *line, *str, *name;
    380379        char *option, **optv;
    381         int len, ret;
     380        int len, lineno, ret;
    382381        FILE *f;
    383382
     
    385384                openpam_log(errno == ENOENT ? PAM_LOG_DEBUG : PAM_LOG_NOTICE,
    386385                    "%s: %m", filename);
    387                 return (0);
     386                return (PAM_SUCCESS);
    388387        }
    389388        this = NULL;
    390389        name = NULL;
    391         count = lineno = 0;
     390        lineno = 0;
    392391        while ((line = openpam_readline(f, &lineno, NULL)) != NULL) {
    393392                /* get service name if necessary */
     
    436435                        ret = openpam_load_chain(pamh, name, fclt);
    437436                        FREE(name);
    438                         if (ret < 0)
     437                        if (ret != PAM_SUCCESS)
    439438                                goto fail;
    440                         count += ret;
    441439                        FREE(line);
    442440                        continue;
     
    497495                *next = this;
    498496                this = NULL;
    499                 ++count;
    500497
    501498                /* next please... */
     
    505502                goto syserr;
    506503        fclose(f);
    507         return (count);
     504        return (PAM_SUCCESS);
    508505syserr:
    509506        openpam_log(PAM_LOG_ERROR, "%s: %m", filename);
     
    518515        FREE(name);
    519516        fclose(f);
    520         return (-1);
     517        return (PAM_SYSTEM_ERR);
    521518}
    522519
     
    541538        char *filename;
    542539        size_t len;
    543         int r;
     540        int ret;
    544541
    545542        for (path = openpam_policy_path; *path != NULL; ++path) {
     
    548545                        if (asprintf(&filename, "%s%s", *path, service) < 0) {
    549546                                openpam_log(PAM_LOG_ERROR, "asprintf(): %m");
    550                                 return (-PAM_BUF_ERR);
     547                                return (PAM_BUF_ERR);
    551548                        }
    552                         r = openpam_parse_chain(pamh, service, facility,
     549                        ret = openpam_parse_chain(pamh, service, facility,
    553550                            filename, pam_d_style);
    554551                        FREE(filename);
    555552                } else {
    556                         r = openpam_parse_chain(pamh, service, facility,
     553                        ret = openpam_parse_chain(pamh, service, facility,
    557554                            *path, pam_conf_style);
    558555                }
    559                 if (r != 0)
    560                         return (r);
    561         }
    562         return (0);
     556                if (ret != PAM_SUCCESS)
     557                        return (ret);
     558        }
     559        return (PAM_SUCCESS);
    563560}
    564561
     
    575572        pam_facility_t fclt;
    576573
    577         if (openpam_load_chain(pamh, service, PAM_FACILITY_ANY) < 0)
     574        if (openpam_load_chain(pamh, service, PAM_FACILITY_ANY) != PAM_SUCCESS)
    578575                goto load_err;
    579576
     
    581578                if (pamh->chains[fclt] != NULL)
    582579                        continue;
    583                 if (openpam_load_chain(pamh, PAM_OTHER, fclt) < 0)
     580                if (openpam_load_chain(pamh, PAM_OTHER, fclt) != PAM_SUCCESS)
    584581                        goto load_err;
    585582        }
Note: See TracChangeset for help on using the changeset viewer.