]> git.ipfire.org Git - thirdparty/krb5.git/commitdiff
Eliminate qualname.plin
authorGreg Hudson <ghudson@mit.edu>
Wed, 31 Jul 2019 03:59:27 +0000 (23:59 -0400)
committerGreg Hudson <ghudson@mit.edu>
Mon, 9 Dec 2019 22:02:52 +0000 (17:02 -0500)
Rewriting the qualname Perl script to use getaddrinfo created an
unchecked dependency on Perl 5.14.  Instead, remove the script and use
the C program in tests/resolve for the kadmin and gssrpc test suites.

(cherry picked from commit 2a852b1f3fb00c7d19f4dbfba76f5ff7ebccd2c7)

src/kadmin/testing/scripts/Makefile.in
src/kadmin/testing/scripts/env-setup.shin
src/kadmin/testing/scripts/init_db
src/kadmin/testing/scripts/qualname.plin [deleted file]
src/kadmin/testing/scripts/start_servers
src/kadmin/testing/scripts/start_servers_local
src/lib/rpc/unit-test/lib/helpers.exp
src/lib/rpc/unit-test/rpc_test_setup.sh

index 8c0d2362a12dc72320dc0a3829418700b299643c..4b3c75c9adab844b67edf9dd589fab896b10dd90 100644 (file)
@@ -5,7 +5,7 @@ PERL_PATH=@PERL_PATH@
 .SUFFIXES: .plin .pl
 
 GEN_SCRIPTS = compare_dump.pl make-host-keytab.pl \
-       simple_dump.pl verify_xrunner_report.pl qualname.pl
+       simple_dump.pl verify_xrunner_report.pl
 
 all: env-setup.sh $(GEN_SCRIPTS)
 
index c8d866f153f0666e1fb957894e8efccb3ef206ae..273cf6954172390040133d8cd468ed2a0db7e31e 100755 (executable)
@@ -69,12 +69,14 @@ MAKE_KEYTAB=$TESTDIR/scripts/make-host-keytab.pl; export MAKE_KEYTAB
 LOCAL_MAKE_KEYTAB=$TESTDIR/scripts/make-host-keytab.pl
 export LOCAL_MAKE_KEYTAB
 SIMPLE_DUMP=$TESTDIR/scripts/simple_dump.pl; export SIMPLE_DUMP
-QUALNAME=$TESTDIR/scripts/qualname.pl; export QUALNAME
 TCLUTIL=$STESTDIR/tcl/util.t; export TCLUTIL
 BSDDB_DUMP=$TESTDIR/util/bsddb_dump; export BSDDB_DUMP
 CLNTTCL=$TESTDIR/util/kadm5_clnt_tcl; export CLNTTCL
 SRVTCL=$TESTDIR/util/kadm5_srv_tcl; export SRVTCL
 
+QUALNAME=`$BUILDTOP/tests/resolve/resolve -q | tr '[A-Z]' '[a-z]'`
+export QUALNAME
+
 KRB5_CONFIG=$K5ROOT/krb5.conf; export KRB5_CONFIG
 KRB5_KDC_PROFILE=$K5ROOT/kdc.conf; export KRB5_KDC_PROFILE
 KRB5_KTNAME=$K5ROOT/ovsec_adm.srvtab; export KRB5_KTNAME
index cd71656288cbc5fc476779d96798ec04145bfb9a..c41d290d16c40c385ba3f9912bad963c40c3d197 100755 (executable)
@@ -77,11 +77,9 @@ fi
 #      esac
 # done
 
-qualname=`$QUALNAME`
-
 sed -e "s/__REALM__/$REALM/g" -e "s#__K5ROOT__#$K5ROOT#g" \
-       -e "s/__KDCHOST__/$qualname/g" \
-       -e "s/__LOCALHOST__/$qualname/g" \
+       -e "s/__KDCHOST__/$QUALNAME/g" \
+       -e "s/__LOCALHOST__/$QUALNAME/g" \
        -e "s#__MODDIR__#$MODDIR#g" \
        < $STESTDIR/proto/krb5.conf.proto > $K5ROOT/krb5.conf
 sed -e "s/__REALM__/$REALM/g" -e "s#__K5ROOT__#$K5ROOT#g" \
diff --git a/src/kadmin/testing/scripts/qualname.plin b/src/kadmin/testing/scripts/qualname.plin
deleted file mode 100755 (executable)
index ccc57e3..0000000
+++ /dev/null
@@ -1,20 +0,0 @@
-#!/usr/bin/perl
-use Socket qw(:addrinfo);
-use strict;
-
-my $hostname;
-if ($#ARGV == -1) {
-    chop($hostname = `hostname`);
-} else {
-    $hostname = $ARGV[0];
-}
-
-my ($gaerr, @addrs) = getaddrinfo($hostname, "", {flags => AI_CANONNAME});
-die "No such host: $hostname ($gaerr)" if $gaerr;
-my ($canonname, $addr) = ($addrs[0]->{canonname}, $addrs[0]->{addr});
-
-my ($gnerr, $name, $servicename) = getnameinfo($addr, NI_NAMEREQD);
-my $qualname = $gnerr ? $name : $name;
-
-$qualname =~ tr/A-Z/a-z/;      # lowercase our name for keytab use.
-print "$qualname\n";
index 87bd52fc0ba6890d5a3c3fd7545aaa08f15d987f..f23df06828f8347dde5d288284f5a59c78fceedf 100755 (executable)
@@ -35,10 +35,9 @@ fi
 if [ $local = 0 ]; then
 
        # Fix up the local krb5.conf to point to the remote 
-       localname=`$QUALNAME`
        sed -e "s/__REALM__/$REALM/g" -e "s#__K5ROOT__#$K5ROOT#g" \
                -e "s/__KDCHOST__/$hostname/g" \
-               -e "s/__LOCALHOST__/$localname/g" \
+               -e "s/__LOCALHOST__/$QUALNAME/g" \
                -e "s#__MODDIR__#$TOP/../plugins/kdb#g"\
                -e "s#__PLUGIN_DIR__#$TOP/../plugins#g"\
                < $STESTDIR/proto/krb5.conf.proto > $K5ROOT/krb5.conf
index 0cbed462d1adbcb5d17a97136c607f22c7ef944b..f34444ee8d1683f1477ae2004dec45340113847e 100755 (executable)
@@ -73,8 +73,6 @@ fi
 
 # allow admin to krlogin as root (for cleanup)
 DUMMY=${REALM=SECURE-TEST.OV.COM}; export REALM
-hostname=`hostname`
-QUALNAME=`$TOP/testing/scripts/qualname.pl $hostname`; export QUALNAME
 
 cat - > /tmp/start_servers_local$$ <<\EOF
 if { [catch {
index a1b078374af54ba73395f58bc22a330571ef9289..a7f89f636a47d5f5577255450f21229f844ab3b5 100644 (file)
@@ -119,7 +119,7 @@ proc setup_database {} {
 }
 
 if ![info exists CANON_HOST] {
-    set CANON_HOST [exec $env(QUALNAME)]
+    set CANON_HOST $env(QUALNAME)
     setup_database
     file delete $env(RPC_TEST_SRVTAB)
     exec $env(MAKE_KEYTAB) -princ "server/$CANON_HOST" $env(RPC_TEST_SRVTAB)
index 968f52a67045735053fc16a5e26120e4507c02fe..d147a337e01edd55cea038f4ca7c895b79d95915 100755 (executable)
@@ -20,12 +20,9 @@ fi
 
 PATH=$TOP/install/admin:$PATH; export PATH
 
-CANON_HOST=`$QUALNAME`
-export CANON_HOST
-
 cat - > /tmp/rpc_test_setup$$ <<\EOF
 source $env(TCLUTIL)
-set h $env(CANON_HOST)
+set h $env(QUALNAME)
 puts stdout [kadm5_init admin admin $KADM5_ADMIN_SERVICE null $KADM5_STRUCT_VERSION $KADM5_API_VERSION_2 server_handle]
 if ![info exists server_handle] { exit 1 }
 puts stdout [kadm5_create_principal $server_handle [simple_principal server/$h] {KADM5_PRINCIPAL} admin]