cvs commit: src/sys/net if_gre.c src/sys/netinet ip_gre.c ip_gre.h src/sys/netipsec keysock.c keysock.h src/sys/netkey keysock.c keysock.h

[Bjoern A. Zeeb]

bz          2006-01-28 14:37:35 UTC

  FreeBSD src repository

  Modified files:        (Branch: RELENG_5)
    sys/net              if_gre.c 
    sys/netinet          ip_gre.c ip_gre.h 
    sys/netipsec         keysock.c keysock.h 
    sys/netkey           keysock.c keysock.h 
  Log:
  MFC:
    rev. 1.39 sys/net/if_gre.c
    rev. 1.21 sys/netinet/ip_gre.c      rev. 1.4  sys/netinet/ip_gre.h
    rev. 1.12 sys/netipsec/keysock.c    rev. 1.3  sys/netipsec/keysock.h
    rev. 1.32 sys/netkey/keysock.c      rev. 1.9  sys/netkey/keysock.h
  
    Fix stack corruptions on amd64.
  
    Vararg functions have a different calling convention than regular
    functions on amd64. Casting a varag function to a regular one to
    match the function pointer declaration will hide the varargs from
    the caller and we will end up with an incorrectly setup stack.
  
    Entirely remove the varargs from these functions and change the
    functions to match the declaration of the function pointers.
    Remove the now unnecessary casts.
  
    Lots of explanations and help from:     peter
    PR:                                     amd64/89261
  
  Revision  Changes    Path
  1.28.2.3  +2 -2      src/sys/net/if_gre.c
  1.17.2.3  +3 -25     src/sys/netinet/ip_gre.c
  1.2.10.2  +2 -2      src/sys/netinet/ip_gre.h
  1.8.2.2   +2 -14     src/sys/netipsec/keysock.c
  1.1.10.2  +1 -1      src/sys/netipsec/keysock.h
  1.27.2.3  +2 -14     src/sys/netkey/keysock.c
  1.7.4.2   +1 -1      src/sys/netkey/keysock.h