From bcd1485075aa72ba9418003f5cc27af2b049c51b Mon Sep 17 00:00:00 2001 From: Damien Miller Date: Sat, 10 Jun 2017 23:41:25 +1000 Subject: [PATCH] portability for sftp globbed ls sort by mtime Include replacement timespeccmp() for systems that lack it. Support time_t struct stat->st_mtime in addition to timespec stat->st_mtim, as well as unsorted fallback. --- configure.ac | 2 ++ defines.h | 7 +++++++ includes.h | 3 +++ sftp.c | 10 ++++++++-- 4 files changed, 20 insertions(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac index 46f7d4957..18079acba 100644 --- a/configure.ac +++ b/configure.ac @@ -3814,6 +3814,8 @@ 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_CHECK_MEMBERS([struct stat.st_blksize]) +AC_CHECK_MEMBERS([struct stat.st_mtim]) +AC_CHECK_MEMBERS([struct stat.st_mtime]) AC_CHECK_MEMBERS([struct passwd.pw_gecos, struct passwd.pw_class, struct passwd.pw_change, struct passwd.pw_expire], [], [], [[ diff --git a/defines.h b/defines.h index 0420a7e8e..f1662edcf 100644 --- a/defines.h +++ b/defines.h @@ -519,6 +519,13 @@ struct winsize { } #endif +#ifndef timespeccmp +#define timespeccmp(tsp, usp, cmp) \ + (((tsp)->tv_sec == (usp)->tv_sec) ? \ + ((tsp)->tv_nsec cmp (usp)->tv_nsec) : \ + ((tsp)->tv_sec cmp (usp)->tv_sec)) +#endif + #ifndef __P # define __P(x) x #endif diff --git a/includes.h b/includes.h index 497a038b2..0fd71792e 100644 --- a/includes.h +++ b/includes.h @@ -93,6 +93,9 @@ #ifdef HAVE_SYS_SYSMACROS_H # include /* For MIN, MAX, etc */ #endif +#ifdef HAVE_SYS_TIME_H +# include /* for timespeccmp if present */ +#endif #ifdef HAVE_SYS_MMAN_H #include /* for MAP_ANONYMOUS */ #endif diff --git a/sftp.c b/sftp.c index 001c6ed2d..67110f738 100644 --- a/sftp.c +++ b/sftp.c @@ -894,9 +894,15 @@ sglob_comp(const void *aa, const void *bb) #define NCMP(a,b) (a == b ? 0 : (a < b ? 1 : -1)) if (sort_flag & LS_NAME_SORT) return (rmul * strcmp(ap, bp)); - else if (sort_flag & LS_TIME_SORT) + else if (sort_flag & LS_TIME_SORT) { +#if defined(HAVE_STRUCT_STAT_ST_MTIM) return (rmul * timespeccmp(&as->st_mtim, &bs->st_mtim, <)); - else if (sort_flag & LS_SIZE_SORT) +#elif defined(HAVE_STRUCT_STAT_ST_MTIME) + return (rmul * NCMP(as->st_mtime, bs->st_mtime)); +#else + return rmul * 1; +#endif + } else if (sort_flag & LS_SIZE_SORT) return (rmul * NCMP(as->st_size, bs->st_size)); fatal("Unknown ls sort type");