[krbdev.mit.edu #8608] git commit

Greg Hudson via RT rt-comment at krbdev.mit.edu
Mon Sep 11 11:32:55 EDT 2017


Fix AIX build issues

In k5-platform.h, only test for SHARED and define the finalizer as
static if we don't expect linker options to be used for finalizers.
SHARED is not a robust test (it isn't defined when building objects
for shared libraries on AIX, OSF/1, or sometimes IRIX because they
don't use separate PIC objects), and as linker finalizer options are
only applied when shared libraries are created, we don't have to worry
about finalizers happening for static libraries.

In expand_path.c, remove the unnecessary structure tag from "struct
token" as it conflicts with <net/if_arp.h> on AIX.

In localaddr.c, initialize output parameters at the beginning of
get_ifreq_array().  Otherwise, gcc cannot be sure that they are always
set when get_ifreq_array() returns 0, because we use errno as a return
value in one case.  Also remove two unused variables.

Use socklen_t instead of int for socket lengths in sim_client.c and
sim_client.h.

Based on patches from Tony Reix.

https://github.com/krb5/krb5/commit/795bcbfab036d9ffdafddc48b6159afd90c1b92e
Author: Greg Hudson <ghudson at mit.edu>
Commit: 795bcbfab036d9ffdafddc48b6159afd90c1b92e
Branch: master
 src/appl/simple/client/sim_client.c |    2 +-
 src/appl/simple/server/sim_server.c |    3 ++-
 src/include/k5-platform.h           |   20 ++++++++++----------
 src/lib/krb5/os/expand_path.c       |    2 +-
 src/lib/krb5/os/localaddr.c         |    7 +++++--
 5 files changed, 19 insertions(+), 15 deletions(-)



More information about the krb5-bugs mailing list