From c02a062db47468207a1db17a47ca19568bb8e3c5 Mon Sep 17 00:00:00 2001
From: Gerhard Rieger <gerhard@dest-unreach.org>
Date: Sun, 22 Jul 2012 16:12:05 +0200
Subject: [PATCH] UNIX-CONNECT did not support half-close

---
 CHANGES    |  3 +++
 test.sh    | 55 +++++++++++++++++++++++++++++++++++++++---------------
 xio-unix.c |  2 ++
 3 files changed, 45 insertions(+), 15 deletions(-)

diff --git a/CHANGES b/CHANGES
index 6575bb1..c5e7afa 100644
--- a/CHANGES
+++ b/CHANGES
@@ -41,6 +41,9 @@ corrections:
 	UDP-LISTEN would alway set SO_REUSEADDR even without fork option and
 	when user set it to 0. Thanks to Michal Svoboda for reporting this bug.
 
+	UNIX-CONNECT did not support half-close. Thanks to Greg Hughes who
+	pointed me to that bug
+
 	docu mentions option so-bindtodev but correct name is so-bindtodevice. 
 	Thanks to Jim Zimmerman for reporting.
 
diff --git a/test.sh b/test.sh
index ef3c2ea..b388b70 100755
--- a/test.sh
+++ b/test.sh
@@ -3927,32 +3927,42 @@ esac
 PORT=$((PORT+1))
 N=$((N+1))
 
-# does our OpenSSL implementation support halfclose?
-NAME=OPENSSLEOF
+
+while read NAMEKEYW FEAT RUNS TESTTMPL PEERTMPL WAITTMPL; do
+if [ -z "$NAMEKEYW" ] || [[ "$NAMEKEYW" == \#* ]]; then continue; fi
+
+export ts="$td/test$N.socket"
+WAITTMPL="$(echo "$WAITTMPL" |sed -e 's/\040/ /g')"
+TESTADDR=$(eval echo $TESTTMPL)
+PEERADDR=$(eval echo $PEERTMPL)
+WAITCMD=$(eval echo $WAITTMPL)
+TESTKEYW=${TESTADDR%%:*}
+
+# does our address implementation support halfclose?
+NAME=${NAMEKEYW}_HALFCLOSE
 case "$TESTS" in
-*%functions%*|*%openssl%*|*%tcp%*|*%tcp4%*|*%ip4%*|*%$NAME%*)
-TEST="$NAME: openssl half close"
-# have an SSL server that executes "$OD_C" and see if EOF on the SSL client
-# brings the result of od to the client
+*%functions%*|*%socket%*|*%halfclose%*|*%$NAME%*)
+TEST="$NAME: $TESTKEYW half close"
+# have a "peer" socat "peer" that executes "$OD_C" and see if EOF on the
+# connecting socat  brings the result of od
 if ! eval $NUMCOND; then :;
-elif ! testaddrs openssl >/dev/null; then
-    $PRINTF "test $F_n $TEST... ${YELLOW}OPENSSL not available${NORMAL}\n" $N
-    numCANT=$((numCANT+1))
-elif ! testaddrs listen tcp ip4 >/dev/null || ! runsip4 >/dev/null; then
-    $PRINTF "test $F_n $TEST... ${YELLOW}TCP/IPv4 not available${NORMAL}\n" $N
+elif [ "$FEAT" != ',' ] && ! testaddrs "$FEAT" >/dev/null; then
+    $PRINTF "test $F_n $TEST... ${YELLOW}$FEAT not available${NORMAL}\n" $N
     numCANT=$((numCANT+1))
+#elif ! runs$RUNS; then
+#    $PRINTF "test $F_n $TEST... ${YELLOW}$RUNS not available${NORMAL}\n" $N
+#    numCANT=$((numCANT+1))
 else
-gentestcert testsrv
 tf="$td/test$N.stdout"
 te="$td/test$N.stderr"
 tdiff="$td/test$N.diff"
 da="test$N $(date) $RANDOM"
-CMD2="$SOCAT $opts OPENSSL-LISTEN:$PORT,pf=ip4,reuseaddr,$SOCAT_EGD,cert=testsrv.crt,key=testsrv.key,verify=0 exec:'$OD_C'"
-CMD="$SOCAT -T1 $OPTS - openssl:$LOCALHOST:$PORT,verify=0,$SOCAT_EGD"
+CMD2="$SOCAT $opts \"$PEERADDR\" EXEC:'$OD_C'"
+CMD="$SOCAT -T1 $opts - $TESTADDR"
 printf "test $F_n $TEST... " $N
 eval "$CMD2 2>\"${te}1\" &"
 pid=$!	# background process id
-waittcp4port $PORT
+$WAITCMD
 echo "$da" |$CMD >$tf 2>"${te}2"
 if ! echo "$da" |$OD_C |diff - "$tf" >"$tdiff"; then
     $PRINTF "$FAILED: $SOCAT:\n"
@@ -3974,6 +3984,21 @@ esac
 PORT=$((PORT+1))
 N=$((N+1))
 
+done <<<"
+UNIXCONNECT      ,       unix UNIX-CONNECT:\$ts UNIX-LISTEN:\$ts waitfile\040\$ts
+UNIXCLIENT       ,       unix UNIX-CLIENT:\$ts UNIX-LISTEN:\$ts waitfile\040\$ts
+GOPEN_UNIXSTREAM ,       unix GOPEN:\$ts UNIX-LISTEN:\$ts waitfile\040\$ts
+UNIXLISTEN       ,       unix UNIX-LISTEN:\$ts UNIX-CONNECT:\$ts,retry=3 sleep\040\1
+TCP4CONNECT      ,       tcp4 TCP4-CONNECT:\$LOCALHOST:\$PORT TCP4-LISTEN:\$PORT waittcp4port\040\$PORT
+TCP4LISTEN       ,       tcp4 TCP4-LISTEN:\$PORT TCP4-CONNECT:\$LOCALHOST:\$PORT,retry=3
+TCP6CONNECT      ,       tcp6 TCP6-CONNECT:\$LOCALHOST6:\$PORT TCP6-LISTEN:\$PORT waittcp6port\040\$PORT
+TCP6LISTEN       ,       tcp6 TCP6-LISTEN:\$PORT TCP6-CONNECT:\$LOCALHOST6:\$PORT,retry=3
+OPENSSL4CLIENT   OPENSSL tcp4 OPENSSL:\$LOCALHOST:\$PORT,verify=0 OPENSSL-LISTEN:\$PORT,$SOCAT_EGD,cert=testsrv.crt,key=testsrv.key,verify=0 waittcp4port\040\$PORT
+OPENSSL4SERVER   OPENSSL tcp4 OPENSSL-LISTEN:\$PORT,$SOCAT_EGD,cert=testsrv.crt,key=testsrv.key,verify=0 OPENSSL:\$LOCALHOST:\$PORT,verify=0,retry=3
+OPENSSL6CLIENT   OPENSSL tcp6 OPENSSL:\$LOCALHOST6:\$PORT,pf=ip6,verify=0 OPENSSL-LISTEN:\$PORT,pf=ip6,$SOCAT_EGD,cert=testsrv.crt,key=testsrv.key,verify=0 waittcp6port\040\$PORT
+OPENSSL6SERVER   OPENSSL tcp6 OPENSSL-LISTEN:\$PORT,pf=ip6,$SOCAT_EGD,cert=testsrv.crt,key=testsrv.key,verify=0 OPENSSL:\$LOCALHOST6:\$PORT,pf=ip6,verify=0,retry=3
+"
+
 
 NAME=OPENSSL_SERVERAUTH
 case "$TESTS" in
diff --git a/xio-unix.c b/xio-unix.c
index 27c6451..2026dd7 100644
--- a/xio-unix.c
+++ b/xio-unix.c
@@ -231,6 +231,8 @@ static int xioopen_unix_connect(int argc, const char *argv[], struct opt *opts,
       return STAT_NORETRY;
    }
 
+   xfd->howtoshut = XIOSHUT_DOWN;
+
    name = argv[1];
    retropt_socket_pf(opts, &pf);
    retropt_bool(opts, OPT_UNIX_TIGHTSOCKLEN, &tight);