Bug 452577

Summary: Actual & placeholder funcs have differing param counts
Product: Red Hat Enterprise Linux 5 Reporter: Steve Snyder <swsnyder>
Component: kernelAssignee: Prarit Bhargava <prarit>
Status: CLOSED ERRATA QA Contact: Martin Jenner <mjenner>
Severity: low Docs Contact:
Priority: low    
Version: 5.2CC: dzickus
Target Milestone: rc   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-01-20 20:25:41 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
Change 4-param macros to match 3-param functions
none
RHEL5 fix for this issue none

Description Steve Snyder 2008-06-23 20:29:41 UTC
Description of problem:

Function xfrm_audit_state_replay(a, b, c) in ~/include/net/xfrm.h is replaced bu
a macro of the same name if the Auditd System Calls feature is not enabled.  The
probalem is that the macro is defined as having 4, not 3, parameters.  The code
that invokes this func/macro only provides 3 parameters.

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

Kernel 2.6.18-92.el5.i686

How reproducible:

Always.

Steps to Reproduce:
1. Disable the Auditd System Calls feature in the kernel
2.  Attempt to rebuild the kernel
3.
  
Actual results:

  CC      net/ipv4/xfrm4_input.o
net/ipv4/xfrm4_input.c:98:39: error: macro "xfrm_audit_state_replay" requires 4
arguments, but only 3 given
net/ipv4/xfrm4_input.c: In function ‘xfrm4_rcv_encap’:
net/ipv4/xfrm4_input.c:98: error: ‘xfrm_audit_state_replay’ undeclared (first
use in this function)
net/ipv4/xfrm4_input.c:98: error: (Each undeclared identifier is reported only once
net/ipv4/xfrm4_input.c:98: error: for each function it appears in.)
make[2]: *** [net/ipv4/xfrm4_input.o] Error 1
make[1]: *** [net/ipv4] Error 2
make: *** [net] Error 2

Expected results:

Placeholder macro should be defined as having the same number of parameters as
the function it is replacing.

Additional info:

Same param count mismatch is also seen with function/macro
xfrm_naudit_state_replay() in same header file.

Comment 1 Steve Snyder 2008-06-23 20:29:41 UTC
Created attachment 310071 [details]
Change 4-param macros to match 3-param functions

Comment 2 Prarit Bhargava 2008-06-24 14:47:59 UTC
Red Hat does not support custom kernel building.  However, custom kernels should
still build :)

Thanks Steve -- I'll see what I can do about getting this into 5.3.

P.

Comment 3 Prarit Bhargava 2008-06-24 16:01:57 UTC
Created attachment 310158 [details]
RHEL5 fix for this issue

Loosely based on original patch -- now fixes compile errors and warning.

Comment 4 RHEL Program Management 2008-06-24 18:05:48 UTC
This request was evaluated by Red Hat Product Management for inclusion in a Red
Hat Enterprise Linux maintenance release.  Product Management has requested
further review of this request by Red Hat Engineering, for potential
inclusion in a Red Hat Enterprise Linux Update release for currently deployed
products.  This request is not yet committed for inclusion in an Update
release.

Comment 6 Don Zickus 2008-09-13 01:47:35 UTC
in kernel-2.6.18-113.el5
You can download this test kernel from http://people.redhat.com/dzickus/el5

Comment 10 errata-xmlrpc 2009-01-20 20:25:41 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHSA-2009-0225.html