Changeset 684 in openpam


Ignore:
Timestamp:
Apr 14, 2013, 3:44:32 PM (8 years ago)
Author:
Dag-Erling Smørgrav
Message:

OPENPAM_DEBUG (--enable-debug) has a double action: it enables the
debugging macros, and sets the initial value of openpam_debug to 1.
This effectively gives the user a choice between no debugging at all,
or drowning in debugging messages from every part of the system.

Assuming that the primary use case for debugging is to allow admins to
troubleshoot their policies by adding the debug option to selected
pam.conf entries, remove the initialization of openpam_debug to 1.
This allows integrators to ship OpenPAM with OPENPAM_DEBUG defined
without spamming /var/log.

Location:
trunk
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/autogen.des

    r648 r684  
    1818        --with-su \
    1919        --with-modules-dir=/usr/lib \
     20        --enable-debug \
    2021        --enable-developer-warnings \
    2122        --enable-werror \
  • trunk/configure.ac

    r660 r684  
    3232AC_ARG_ENABLE([debug],
    3333    AC_HELP_STRING([--enable-debug],
    34         [turn debugging on by default]),
    35     AC_DEFINE(OPENPAM_DEBUG, 1, [Turn debugging on by default]))
     34        [turn debugging macros on]),
     35    AC_DEFINE(OPENPAM_DEBUG, 1, [Turn debugging macros on]))
    3636
    3737AC_ARG_ENABLE([unversioned-modules],
  • trunk/lib/libpam/openpam_log.c

    r681 r684  
    5151#include "openpam_asprintf.h"
    5252
    53 #ifdef OPENPAM_DEBUG
    54 int openpam_debug = 1;
    55 #else
    5653int openpam_debug = 0;
    57 #endif
    5854
    5955#if !defined(openpam_log)
Note: See TracChangeset for help on using the changeset viewer.