Bug 157107 - [PATCH] Not all debugging statements in pam_krb5 actually work with 'debug' option
Summary: [PATCH] Not all debugging statements in pam_krb5 actually work with 'debug' o...
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat Enterprise Linux 4
Classification: Red Hat
Component: pam_krb5
Version: 4.0
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
: ---
Assignee: Nalin Dahyabhai
QA Contact: Brian Brock
URL: http://www-personal.engin.umich.edu/~...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2005-05-06 21:35 UTC by wingc
Modified: 2012-06-20 16:57 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-06-20 16:57:14 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
make sure that all debug() statements are called when debugging is enabled (1.33 KB, patch)
2005-05-06 21:35 UTC, wingc
no flags Details | Diff

Description wingc 2005-05-06 21:35:51 UTC
Description of problem:

Some of the debugging statements in src/minikafs.c are wrapped inside
conditionals like this:

    if (options->debug > 1) {
        debug(....
    }

The only problem is that options->debug can only equal 0 or 1 when pam_krb5
parses its options. There is no way to get all the debugging output when you
specify 'debug' in the PAM configuration files.



Version-Release number of selected component (if applicable):

Tested on pam_krb5-2.1.2-1 from RHEL4

How reproducible:

Always

Steps to Reproduce:
1. Add 'debug' to the command line options for pam_krb5, e.g. in
/etc/pam.d/system-auth
  
Actual results:

Not all the debugging statments from src/minikafs.c generate output in the syslog.

Expected results:

Some of the missing debugging output would have been useful, such as "got error
determining file server for cell", etc.


Additional info:

A patch to fix this problem is at:

http://www-personal.engin.umich.edu/~wingc/patches/pam_krb5/pam_krb5-2.1.2-moredebug.patch

Comment 1 wingc 2005-05-06 21:35:51 UTC
Created attachment 114101 [details]
make sure that all debug() statements are called when debugging is enabled

Comment 2 Jiri Pallich 2012-06-20 16:57:14 UTC
Thank you for submitting this issue for consideration in Red Hat Enterprise Linux. The release for which you requested us to review is now End of Life. 
Please See https://access.redhat.com/support/policy/updates/errata/

If you would like Red Hat to re-consider your feature request for an active release, please re-open the request via appropriate support channels and provide additional supporting details about the importance of this issue.


Note You need to log in before you can comment on or make changes to this bug.