7 files changed, 238 insertions, 67 deletions

diff --git a/ChangeLog b/ChangeLog
index 206e23356..837df8870 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,18 @@
+20011021
+ - (tim) [configure.in] Clean up library testing. Add optional PATH to
+         --with-pcre, --with-zlib, and --with-tcp-wrappers. Based on
+         patch by albert chin (china@thewrittenword.com)
+         Re-arange AC_CHECK_HEADERS and AC_CHECK_FUNCS for eaiser reading
+         of patches to configure.in. Replace obsolete AC_STRUCT_ST_BLKSIZE
+         with AC_CHECK_MEMBERS. Add test for broken dirname() on
+         Solaris 2.5.1 by Dan Astoorian <djast@cs.toronto.edu>
+         [acconfig.h aclocal.m4 defines.h configure.in] Better socklen_t test.
+         patch by albert chin (china@thewrittenword.com)
+         [scp.c] Replace obsolete HAVE_ST_BLKSIZE with
+         HAVE_STRUCT_STAT_ST_BLKSIZE.
+         [Makefile.in] When running make in top level, always do make
+         in openbsd-compat. patch by Dave Dykstra <dwd@bell-labs.com>
+
 20011019
  - (bal) Fixed up init.d symlink issue and piddir stuff.  Patches by
    Zoran Milojevic <Zoran.Milojevic@SS8.com> and j.petersen@msh.de 
@@ -6725,4 +6740,4 @@
  - Wrote replacements for strlcpy and mkdtemp
  - Released 1.0pre1
 
-$Id: ChangeLog,v 1.1609 2001/10/19 20:36:23 mouring Exp $
+$Id: ChangeLog,v 1.1610 2001/10/22 00:53:58 tim Exp $
diff --git a/Makefile.in b/Makefile.in
index 640fe5abd..9bf8a425c 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -1,4 +1,4 @@
-# $Id: Makefile.in,v 1.188 2001/10/12 09:15:28 djm Exp $
+# $Id: Makefile.in,v 1.189 2001/10/22 00:53:59 tim Exp $
 
 prefix=@prefix@
 exec_prefix=@exec_prefix@
@@ -86,8 +86,9 @@ $(SSHDOBJS): config.h
         $(CC) $(CFLAGS) $(CPPFLAGS) -c $<
 
 LIBCOMPAT=openbsd-compat/libopenbsd-compat.a
-$(LIBCOMPAT): config.h
+$(LIBCOMPAT): always
         (cd openbsd-compat; $(MAKE))
+always:
 
 libssh.a: $(LIBSSH_OBJS)
         $(AR) rv $@ $(LIBSSH_OBJS)
diff --git a/acconfig.h b/acconfig.h
index 0c1417885..afebfa5db 100644
--- a/acconfig.h
+++ b/acconfig.h
@@ -1,4 +1,4 @@
-/* $Id: acconfig.h,v 1.117 2001/09/20 19:43:41 stevesk Exp $ */
+/* $Id: acconfig.h,v 1.118 2001/10/22 00:53:59 tim Exp $ */
 
 #ifndef _CONFIG_H
 #define _CONFIG_H
@@ -251,7 +251,6 @@
 #undef HAVE_INT64_T
 #undef HAVE_U_INT64_T
 #undef HAVE_U_CHAR
-#undef HAVE_SOCKLEN_T
 #undef HAVE_SIZE_T
 #undef HAVE_SSIZE_T
 #undef HAVE_CLOCK_T
diff --git a/aclocal.m4 b/aclocal.m4
index d196b7585..2705a9b23 100644
--- a/aclocal.m4
+++ b/aclocal.m4
@@ -1,4 +1,4 @@
-dnl $Id: aclocal.m4,v 1.4 2000/06/26 00:20:19 djm Exp $
+dnl $Id: aclocal.m4,v 1.5 2001/10/22 00:53:59 tim Exp $
 dnl
 dnl OpenSSH-specific autoconf macros
 dnl
@@ -43,3 +43,44 @@ AC_DEFUN(OSSH_PATH_ENTROPY_PROG, [
         AC_SUBST($1)
 ])
 
+dnl Check for socklen_t: historically on BSD it is an int, and in
+dnl POSIX 1g it is a type of its own, but some platforms use different
+dnl types for the argument to getsockopt, getpeername, etc.  So we
+dnl have to test to find something that will work.
+AC_DEFUN([TYPE_SOCKLEN_T],
+[
+   AC_CHECK_TYPE([socklen_t], ,[
+      AC_MSG_CHECKING([for socklen_t equivalent])
+      AC_CACHE_VAL([curl_cv_socklen_t_equiv],
+      [
+         # Systems have either "struct sockaddr *" or
+         # "void *" as the second argument to getpeername
+         curl_cv_socklen_t_equiv=
+         for arg2 in "struct sockaddr" void; do
+            for t in int size_t unsigned long "unsigned long"; do
+               AC_TRY_COMPILE([
+                  #include <sys/types.h>
+                  #include <sys/socket.h>
+
+                  int getpeername (int, $arg2 *, $t *);
+               ],[
+                  $t len;
+                  getpeername(0,0,&len);
+               ],[
+                  curl_cv_socklen_t_equiv="$t"
+                  break
+               ])
+            done
+         done
+
+         if test "x$curl_cv_socklen_t_equiv" = x; then
+            AC_MSG_ERROR([Cannot find a type to use in place of socklen_t])
+         fi
+      ])
+      AC_MSG_RESULT($curl_cv_socklen_t_equiv)
+      AC_DEFINE_UNQUOTED(socklen_t, $curl_cv_socklen_t_equiv,
+                        [type to use in place of socklen_t if not defined])],
+      [#include <sys/types.h>
+#include <sys/socket.h>])
+])
+
diff --git a/configure.in b/configure.in
index c07aaf2c6..95372640f 100644
--- a/configure.in
+++ b/configure.in
@@ -1,6 +1,7 @@
-# $Id: configure.in,v 1.314 2001/09/25 06:39:35 djm Exp $
+# $Id: configure.in,v 1.315 2001/10/22 00:53:59 tim Exp $
 
-AC_INIT(ssh.c)
+AC_INIT
+AC_CONFIG_SRCDIR([ssh.c])
 
 AC_CONFIG_HEADER(config.h)
 AC_PROG_CC
@@ -71,8 +72,6 @@ case "$host" in
         AC_DEFINE(IPV4_DEFAULT)
         AC_DEFINE(IP_TOS_IS_BROKEN)
         AC_DEFINE(NO_X11_UNIX_SOCKETS)
-        no_libsocket=1
-        no_libnsl=1
         ;;
 *-*-dgux*)
         AC_DEFINE(IP_TOS_IS_BROKEN)
@@ -106,8 +105,6 @@ case "$host" in
         CPPFLAGS="$CPPFLAGS -I/usr/local/include"
         LDFLAGS="$LDFLAGS"
         PATH="$PATH:/usr/etc"
-        no_libsocket=1
-        no_libnsl=1
         AC_DEFINE(BROKEN_INET_NTOA)
         ;;
 *-*-irix6*)
@@ -118,8 +115,6 @@ case "$host" in
         AC_DEFINE(WITH_IRIX_PROJECT)
         AC_DEFINE(WITH_IRIX_AUDIT)
         AC_CHECK_FUNC(jlimit_startjob, [AC_DEFINE(WITH_IRIX_JOBS)])
-        no_libsocket=1
-        no_libnsl=1
         AC_DEFINE(BROKEN_INET_NTOA)
         ;;
 *-*-linux*)
@@ -186,7 +181,7 @@ mips-sony-bsd|mips-sony-newsos4)
 *-ncr-sysv*)
         CPPFLAGS="$CPPFLAGS -I/usr/local/include"
         LDFLAGS="$LDFLAGS -L/usr/local/lib"
-        LIBS="$LIBS -lc89 -lnsl -lgen -lsocket"
+        LIBS="$LIBS -lc89"
         AC_DEFINE(HAVE_BOGUS_SYS_QUEUE_H)
         ;;
 *-sni-sysv*)
@@ -213,12 +208,11 @@ mips-sony-bsd|mips-sony-newsos4)
 *-*-sysv*)
         CPPFLAGS="$CPPFLAGS -I/usr/local/include"
         LDFLAGS="$LDFLAGS -L/usr/local/lib"
-        LIBS="$LIBS -lgen -lsocket"
         ;;
 *-*-sco3.2v4*)
         CPPFLAGS="$CPPFLAGS -Dftruncate=chsize -I/usr/local/include"
         LDFLAGS="$LDFLAGS -L/usr/local/lib"
-        LIBS="$LIBS -lgen -lsocket -los -lprot -lx -ltinfo -lm"
+        LIBS="$LIBS -los -lprot -lx -ltinfo -lm"
         rsh_path="/usr/bin/rcmd"
         RANLIB=true
         no_dev_ptmx=1
@@ -230,6 +224,7 @@ mips-sony-bsd|mips-sony-newsos4)
         AC_DEFINE(BROKEN_SAVED_UIDS)
         AC_CHECK_FUNCS(getluid setluid)
         MANTYPE=man
+        do_sco3_extra_lib_check=yes
         ;;
 *-*-sco3.2v5*)
         CPPFLAGS="$CPPFLAGS -I/usr/local/include"
@@ -319,34 +314,92 @@ AC_ARG_WITH(libs,
 )
 
 AC_ARG_WITH(pcre,
-        [  --with-pcre             Override built in regex library with pcre],
+        [  --with-pcre[[=PATH]]      Override built in regex library with pcre
+                            (optionally in PATH)],
         [
+                case "$withval" in
+                no) ;;
+                *)
+                        if test "x$withval" != "xyes"; then
+                                if test -d "$withval/lib"; then
+                                        if test -n "${need_dash_r}"; then
+                                                LDFLAGS="${LDFLAGS} -L$withval/lib -R$withval/lib"
+                                        else
+                                                LDFLAGS="${LDFLAGS} -L$withval/lib"
+                                        fi
+                                else
+                                        if test -n "${need_dash_r}"; then
+                                                LDFLAGS="${LDFLAGS} -L$withval -R$withval"
+                                        else
+                                                LDFLAGS="${LDFLAGS} -L$withval"
+                                        fi
+                                fi
+                                if test -d "$withval/include"; then
+                                        CPPFLAGS="${CPPFLAGS} -I$withval/include"
+                                else
+                                        CPPFLAGS="${CPPFLAGS} -I$withval"
+                                fi
+                        fi
 
-                AC_CHECK_LIB(pcre, pcre_info, 
-                        [ 
-                                AC_DEFINE(HAVE_LIBPCRE) 
-                                LIBS="$LIBS -lpcreposix -lpcre" 
-                                no_comp_check="yes"
-                        ],
-                        [ AC_MSG_ERROR([*** Can not locate pcre libraries.]) ]
-                )
+                        AC_CHECK_HEADER(pcreposix.h,
+                                AC_CHECK_LIB(pcre, pcre_info,[
+                                        AC_DEFINE(HAVE_LIBPCRE)
+                                        LIBS="$LIBS -lpcreposix -lpcre"
+                                        no_comp_check=yes],
+                                        AC_MSG_ERROR([*** unable to locate pcre library ***])),
+                                AC_MSG_ERROR([*** unable to locate pcreposix.h include file ***]))
+                        ;;
+                esac
         ]       
 )
 
 # Checks for libraries.
-if test -z "$no_libnsl" ; then
-        AC_CHECK_LIB(nsl, yp_match, , )
-fi
-if test -z "$no_libsocket" ; then
-        AC_CHECK_LIB(socket, main, , )
-fi
+AC_CHECK_FUNC(yp_match, , AC_CHECK_LIB(nsl, yp_match))
+AC_CHECK_FUNC(setsockopt, , AC_CHECK_LIB(socket, setsockopt))
 
 dnl SCO OS3 needs this for libwrap
-AC_CHECK_LIB(rpc, innetgr, LIBS="-lrpc -lyp -lrpc $LIBS" , , -lyp -lrpc)
+if test "x$with_tcp_wrappers" != "xno" ; then
+    if test "x$do_sco3_extra_lib_check" = "xyes" ; then
+        AC_CHECK_LIB(rpc, innetgr, LIBS="-lrpc -lyp -lrpc $LIBS" , , -lyp -lrpc)
+    fi
+fi
+
+AC_CHECK_FUNC(getspnam, ,
+        AC_CHECK_LIB(gen, getspnam, LIBS="$LIBS -lgen"))
+AC_CHECK_FUNC(login, ,
+        AC_CHECK_LIB(util, login,
+                AC_DEFINE(HAVE_LIBUTIL_LOGIN) LIBS="$LIBS -lutil"))
+
+dnl zlib is required
+AC_ARG_WITH(zlib,
+        [  --with-zlib=PATH        Use zlib in PATH],
+        [
+                if test -d "$withval/lib"; then
+                        if test -n "${need_dash_r}"; then
+                                LDFLAGS="${LDFLAGS} -L$withval/lib -R$withval/lib"
+                        else
+                                LDFLAGS="${LDFLAGS} -L$withval/lib"
+                        fi
+                else
+                        if test -n "${need_dash_r}"; then
+                                LDFLAGS="${LDFLAGS} -L$withval -R$withval"
+                        else
+                                LDFLAGS="${LDFLAGS} -L$withval"
+                        fi
+                fi
+                if test -d "$withval/include"; then
+                        CPPFLAGS="${CPPFLAGS} -I$withval/include"
+                else
+                        CPPFLAGS="${CPPFLAGS} -I$withval"
+                fi
+        ]
+)
+
+AC_CHECK_HEADER(zlib.h,
+        AC_CHECK_LIB(z, gzread, ,
+                AC_MSG_ERROR([*** zlib missing. install first or check config.log ***])),
+        AC_MSG_ERROR([*** zlib missing. install first or check config.log ***]))
 
-AC_CHECK_LIB(gen, getspnam, LIBS="$LIBS -lgen")
-AC_CHECK_LIB(z, deflate, ,AC_MSG_ERROR([*** zlib missing - please install first or check config.log ***]))
-AC_CHECK_LIB(util, login, AC_DEFINE(HAVE_LIBUTIL_LOGIN) LIBS="$LIBS -lutil")
 
 # We don't want to check if we did an pcre override.
 if test -z "$no_comp_check" ; then
@@ -376,7 +429,16 @@ AC_CHECK_FUNC(utimes,
 AC_FUNC_STRFTIME
 
 # Checks for header files.
-AC_CHECK_HEADERS(bstring.h crypt.h endian.h floatingpoint.h getopt.h glob.h lastlog.h libgen.h limits.h login.h login_cap.h maillock.h netdb.h netgroup.h netinet/in_systm.h paths.h poll.h pty.h regex.h shadow.h security/pam_appl.h stdint.h strings.h sys/bitypes.h sys/bsdtty.h sys/cdefs.h sys/poll.h sys/queue.h sys/select.h sys/stat.h sys/stropts.h sys/sysmacros.h sys/time.h sys/ttcompat.h sys/un.h stddef.h time.h ttyent.h usersec.h util.h utime.h utmp.h utmpx.h)
+AC_CHECK_HEADERS(bstring.h crypt.h endian.h floatingpoint.h \
+        getopt.h glob.h lastlog.h libgen.h limits.h login.h \
+        login_cap.h maillock.h netdb.h netgroup.h \
+        netinet/in_systm.h paths.h poll.h pty.h regex.h \
+        security/pam_appl.h shadow.h stddef.h stdint.h \
+        strings.h sys/bitypes.h sys/bsdtty.h sys/cdefs.h \
+        sys/poll.h sys/queue.h sys/select.h sys/stat.h \
+        sys/stropts.h sys/sysmacros.h sys/time.h \
+        sys/ttcompat.h sys/un.h time.h ttyent.h usersec.h \
+        util.h utime.h utmp.h utmpx.h)
 
 # Check for ALTDIRFUNC glob() extension
 AC_MSG_CHECKING(for GLOB_ALTDIRFUNC support)
@@ -429,7 +491,8 @@ int main(void){struct dirent d;return(sizeof(d.d_name)<=sizeof(char));}
 # Check whether user wants S/Key support
 SKEY_MSG="no" 
 AC_ARG_WITH(skey,
-        [  --with-skey=PATH         Enable S/Key support],
+        [  --with-skey[[=PATH]]      Enable S/Key support
+                            (optionally in PATH)],
         [
                 if test "x$withval" != "xno" ; then
 
@@ -452,12 +515,36 @@ AC_ARG_WITH(skey,
 )
 
 # Check whether user wants TCP wrappers support
-TCPW_MSG="no" 
+TCPW_MSG="no"
 AC_ARG_WITH(tcp-wrappers,
-        [  --with-tcp-wrappers     Enable tcpwrappers support],
+        [  --with-tcp-wrappers[[=PATH]]      Enable tcpwrappers support
+                            (optionally in PATH)],
         [
                 if test "x$withval" != "xno" ; then
                         saved_LIBS="$LIBS"
+                        saved_LDFLAGS="$LDFLAGS"
+                        saved_CPPFLAGS="$CPPFLAGS"
+                        if test -n "${withval}" -a "${withval}" != "yes"; then
+                                if test -d "${withval}/lib"; then
+                                        if test -n "${need_dash_r}"; then
+                                                LDFLAGS="$LDFLAGS -L${withval}/lib -R${withval}/lib"
+                                        else
+                                                LDFLAGS="$LDFLAGS -L${withval}/lib"
+                                        fi
+                                else
+                                        if test -n "${need_dash_r}"; then
+                                                LDFLAGS="$LDFLAGS -L${withval} -R${withval}"
+                                        else
+                                                LDFLAGS="$LDFLAGS -L${withval}"
+                                        fi
+                                fi
+                                if test -d "${withval}/include"; then
+                                        CPPFLAGS="$CPPFLAGS -I${withval}/include"
+                                else
+                                        CPPFLAGS="$CPPFLAGS -I${withval}"
+                                fi
+                                        TCPW_MSG="yes"
+                        fi
                         LIBS="-lwrap $LIBS"
                         AC_MSG_CHECKING(for libwrap)
                         AC_TRY_LINK(
@@ -469,7 +556,7 @@ AC_ARG_WITH(tcp-wrappers,
                                 [
                                         AC_MSG_RESULT(yes)
                                         AC_DEFINE(LIBWRAP)
-                                        TCPW_MSG="yes" 
+                                        TCPW_MSG="yes"
                                 ],
                                 [
                                         AC_MSG_ERROR([*** libwrap missing])
@@ -480,7 +567,51 @@ AC_ARG_WITH(tcp-wrappers,
 )
 
 dnl    Checks for library functions.
-AC_CHECK_FUNCS(arc4random atexit b64_ntop bcopy bindresvport_sa clock dirname fchown fchmod freeaddrinfo futimes gai_strerror getcwd getaddrinfo getgrouplist getopt getnameinfo getrlimit getrusage getttyent glob inet_aton inet_ntoa inet_ntop innetgr login_getcapbool md5_crypt memmove mkdtemp on_exit openpty readpassphrase realpath rresvport_af setdtablesize setenv setegid seteuid setlogin setproctitle setresgid setreuid setrlimit setsid setvbuf sigaction sigvec snprintf strerror strlcat strlcpy strmode strsep sysconf tcgetpgrp utimes vsnprintf vhangup waitpid _getpty __b64_ntop)
+AC_CHECK_FUNCS(arc4random atexit b64_ntop bcopy bindresvport_sa \
+        clock fchmod fchown freeaddrinfo futimes gai_strerror \
+        getaddrinfo getcwd getgrouplist getnameinfo getopt \
+        getrlimit getrusage getttyent glob inet_aton inet_ntoa \
+        inet_ntop innetgr login_getcapbool md5_crypt memmove \
+        mkdtemp on_exit openpty readpassphrase realpath \
+        rresvport_af setdtablesize setegid setenv seteuid \
+        setlogin setproctitle setresgid setreuid setrlimit \
+        setsid setvbuf sigaction sigvec snprintf strerror \
+        strlcat strlcpy strmode strsep sysconf tcgetpgrp utimes \
+        vhangup vsnprintf waitpid __b64_ntop _getpty)
+
+dnl IRIX and Solaris 2.5.1 have dirname() in libgen
+AC_CHECK_FUNCS(dirname, ,[
+        AC_CHECK_LIB(gen, dirname,[
+                AC_CACHE_CHECK([for broken dirname],
+                        ac_cv_have_broken_dirname, [
+                        AC_TRY_RUN(
+                                [
+#include <libgen.h>
+#include <string.h>
+
+int main(int argc, char **argv) {
+    char *s, buf[32];
+
+    strncpy(buf,"/etc", 32);
+    s = dirname(buf);
+    if (s && s[0] == '\0') {
+        exit(1);
+    } else {
+        exit(0);
+    }
+}
+                                ],
+                                [ ac_cv_have_broken_dirname="no" ],
+                                [ ac_cv_have_broken_dirname="yes" ]
+                        )
+                ])
+                if test "x$ac_cv_have_getopt_optreset" = "xno" ; then
+                        LIBS="$LIBS -lgen"
+                        AC_DEFINE(HAVE_DIRNAME)
+                fi
+        ])
+])
+
 dnl    Checks for time functions
 AC_CHECK_FUNCS(gettimeofday time)
 dnl    Checks for libutil functions
@@ -928,20 +1059,7 @@ if test "x$ac_cv_have_u_char" = "xyes" ; then
         AC_DEFINE(HAVE_U_CHAR)
 fi
 
-AC_CACHE_CHECK([for socklen_t], ac_cv_have_socklen_t, [
-        AC_TRY_COMPILE(
-                [
-#include <sys/types.h>
-#include <sys/socket.h>
-                ],
-                [socklen_t foo; foo = 1235;],
-                [ ac_cv_have_socklen_t="yes" ],
-                [ ac_cv_have_socklen_t="no" ]
-        )
-])
-if test "x$ac_cv_have_socklen_t" = "xyes" ; then
-        AC_DEFINE(HAVE_SOCKLEN_T)
-fi
+TYPE_SOCKLEN_T
 
 AC_CACHE_CHECK([for size_t], ac_cv_have_size_t, [
         AC_TRY_COMPILE(
@@ -1168,7 +1286,8 @@ OSSH_CHECK_HEADER_FOR_FIELD(ut_exit, utmp.h, HAVE_EXIT_IN_UTMP)
 OSSH_CHECK_HEADER_FOR_FIELD(ut_time, utmp.h, HAVE_TIME_IN_UTMP)
 OSSH_CHECK_HEADER_FOR_FIELD(ut_time, utmpx.h, HAVE_TIME_IN_UTMPX)
 OSSH_CHECK_HEADER_FOR_FIELD(ut_tv, utmpx.h, HAVE_TV_IN_UTMPX)
-AC_STRUCT_ST_BLKSIZE
+
+AC_CHECK_MEMBERS([struct stat.st_blksize])
 
 AC_CACHE_CHECK([for ss_family field in struct sockaddr_storage],
                 ac_cv_have_ss_family_in_struct_ss, [
@@ -2045,7 +2164,8 @@ fi
 
 AC_EXEEXT
 
-AC_OUTPUT(Makefile openbsd-compat/Makefile scard/Makefile ssh_prng_cmds)
+AC_CONFIG_FILES([Makefile openbsd-compat/Makefile scard/Makefile ssh_prng_cmds])
+AC_OUTPUT
 
 # Print summary of options
 
diff --git a/defines.h b/defines.h
index 52cd41125..886ad9408 100644
--- a/defines.h
+++ b/defines.h
@@ -1,7 +1,7 @@
 #ifndef _DEFINES_H
 #define _DEFINES_H
 
-/* $Id: defines.h,v 1.72 2001/09/20 19:43:41 stevesk Exp $ */
+/* $Id: defines.h,v 1.73 2001/10/22 00:53:59 tim Exp $ */
 
 /* Necessary headers */
 
@@ -243,11 +243,6 @@ typedef unsigned char u_char;
 # define HAVE_U_CHAR
 #endif /* HAVE_U_CHAR */
 
-#ifndef HAVE_SOCKLEN_T
-typedef unsigned int socklen_t;
-# define HAVE_SOCKLEN_T
-#endif /* HAVE_SOCKLEN_T */
-
 #ifndef HAVE_SIZE_T
 typedef unsigned int size_t;
 # define HAVE_SIZE_T
diff --git a/scp.c b/scp.c
index b5cb541b4..fb4d3096c 100644
--- a/scp.c
+++ b/scp.c
@@ -1040,7 +1040,7 @@ allocbuf(bp, fd, blksize)
         int fd, blksize;
 {
         size_t size;
-#ifdef HAVE_ST_BLKSIZE
+#ifdef HAVE_STRUCT_STAT_ST_BLKSIZE
         struct stat stb;
 
         if (fstat(fd, &stb) < 0) {
@@ -1052,9 +1052,9 @@ allocbuf(bp, fd, blksize)
         else
                 size = blksize + (stb.st_blksize - blksize % stb.st_blksize) %
                     stb.st_blksize;
-#else /* HAVE_ST_BLKSIZE */
+#else /* HAVE_STRUCT_STAT_ST_BLKSIZE */
         size = blksize;
-#endif /* HAVE_ST_BLKSIZE */
+#endif /* HAVE_STRUCT_STAT_ST_BLKSIZE */
         if (bp->cnt >= size)
                 return (bp);
         if (bp->buf == NULL)