Bug 157107

Summary: [PATCH] Not all debugging statements in pam_krb5 actually work with 'debug' option
Product: Red Hat Enterprise Linux 4 Reporter: wingc
Component: pam_krb5Assignee: Nalin Dahyabhai <nalin>
Status: CLOSED WONTFIX QA Contact: Brian Brock <bbrock>
Severity: medium Docs Contact:
Priority: medium    
Version: 4.0CC: mattdm
Target Milestone: ---Keywords: Patch
Target Release: ---   
Hardware: All   
OS: Linux   
URL: http://www-personal.engin.umich.edu/~wingc/patches/pam_krb5/pam_krb5-2.1.2-moredebug.patch
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-06-20 16:57:14 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
make sure that all debug() statements are called when debugging is enabled none

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.