post-1.16.1 fixes
authorDenys Vlasenko <vda.linux@googlemail.com>
Sat, 12 Jun 2010 13:51:29 +0000 (15:51 +0200)
committerDenys Vlasenko <vda.linux@googlemail.com>
Sat, 12 Jun 2010 13:51:29 +0000 (15:51 +0200)
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
archival/cpio.c
editors/sed.c
include/platform.h
networking/dnsd.c
networking/httpd_indexcgi.c
networking/udhcp/leases.c
shell/ash.c
shell/hush.c
testsuite/cpio.tests
testsuite/sed.tests
util-linux/hwclock.c

index 858e59b306732357a7c7a2f596e6d5182cbc84ce..2698f0791cb7db4ee1232542ecabd161ea1fc619 100644 (file)
@@ -424,7 +424,7 @@ int cpio_main(int argc UNUSED_PARAM, char **argv)
        if (archive_handle->cpio__blocks != (off_t)-1
         && !(opt & CPIO_OPT_QUIET)
        ) {
-               printf("%"OFF_FMT"u blocks\n", archive_handle->cpio__blocks);
+               fprintf(stderr, "%"OFF_FMT"u blocks\n", archive_handle->cpio__blocks);
        }
 
        return EXIT_SUCCESS;
index fd9dd1be6fcceb6723a2b98b86be1c007b1404e4..4289aa6468fc3521ee1583c4cf48a3e4c24cf96a 100644 (file)
@@ -487,7 +487,7 @@ static const char *parse_cmd_args(sed_cmd_t *sed_cmd, const char *cmdstr)
 static void add_cmd(const char *cmdstr)
 {
        sed_cmd_t *sed_cmd;
-       int temp;
+       unsigned len, n;
 
        /* Append this line to any unfinished line from last time. */
        if (G.add_cmd_line) {
@@ -496,12 +496,14 @@ static void add_cmd(const char *cmdstr)
                cmdstr = G.add_cmd_line = tp;
        }
 
-       /* If this line ends with backslash, request next line. */
-       temp = strlen(cmdstr);
-       if (temp && cmdstr[--temp] == '\\') {
+       /* If this line ends with unescaped backslash, request next line. */
+       n = len = strlen(cmdstr);
+       while (n && cmdstr[n-1] == '\\')
+               n--;
+       if ((len - n) & 1) { /* if odd number of trailing backslashes */
                if (!G.add_cmd_line)
                        G.add_cmd_line = xstrdup(cmdstr);
-               G.add_cmd_line[temp] = '\0';
+               G.add_cmd_line[len-1] = '\0';
                return;
        }
 
@@ -936,7 +938,15 @@ static void process_files(void)
                /* Skip blocks of commands we didn't match */
                if (sed_cmd->cmd == '{') {
                        if (sed_cmd->invert ? matched : !matched) {
-                               while (sed_cmd->cmd != '}') {
+                               unsigned nest_cnt = 0;
+                               while (1) {
+                                       if (sed_cmd->cmd == '{')
+                                               nest_cnt++;
+                                       if (sed_cmd->cmd == '}') {
+                                               nest_cnt--;
+                                               if (nest_cnt == 0)
+                                                       break;
+                                       }
                                        sed_cmd = sed_cmd->next;
                                        if (!sed_cmd)
                                                bb_error_msg_and_die("unterminated {");
@@ -1031,7 +1041,7 @@ static void process_files(void)
                case 'c':
                        /* Only triggers on last line of a matching range. */
                        if (!sed_cmd->in_match)
-                               sed_puts(sed_cmd->string, NO_EOL_CHAR);
+                               sed_puts(sed_cmd->string, '\n');
                        goto discard_line;
 
                /* Read file, append contents to output */
index a41daa08cd0cbe81ea5779b623a080c39abb0b03..804fab4e1e7e516d57fb14fbb9d92d3afda7b306 100644 (file)
@@ -291,10 +291,12 @@ typedef unsigned smalluint;
 #if 1 /* if needed: !defined(arch1) && !defined(arch2) */
 # define ALIGN1 __attribute__((aligned(1)))
 # define ALIGN2 __attribute__((aligned(2)))
+# define ALIGN4 __attribute__((aligned(4)))
 #else
 /* Arches which MUST have 2 or 4 byte alignment for everything are here */
 # define ALIGN1
 # define ALIGN2
+# define ALIGN4
 #endif
 
 
index 56ede3fca2ccf1e2223860451a22545a4868b670..1a4ccb911d79395afa13742156e157773e0a7387 100644 (file)
@@ -44,10 +44,15 @@ struct dns_head {
        uint16_t nauth;
        uint16_t nadd;
 };
+/* Structure used to access type and class fields.
+ * They are totally unaligned, but gcc 4.3.4 thinks that pointer of type uint16_t*
+ * is 16-bit aligned and replaces 16-bit memcpy (in move_from_unaligned16 macro)
+ * with aligned halfword access on arm920t!
+ * Oh well. Slapping PACKED everywhere seems to help: */
 struct dns_prop {
-       uint16_t type;
-       uint16_t class;
-};
+       uint16_t type PACKED;
+       uint16_t class PACKED;
+} PACKED;
 /* element of known name, ip address and reversed ip address */
 struct dns_entry {
        struct dns_entry *next;
@@ -459,7 +464,8 @@ int dnsd_main(int argc UNUSED_PARAM, char **argv)
        unsigned lsa_size;
        int udps, opts;
        uint16_t port = 53;
-       uint8_t buf[MAX_PACK_LEN + 1];
+       /* Ensure buf is 32bit aligned (we need 16bit, but 32bit can't hurt) */
+       uint8_t buf[MAX_PACK_LEN + 1] ALIGN4;
 
        opts = getopt32(argv, "vi:c:t:p:d", &listen_interface, &fileconf, &sttl, &sport);
        //if (opts & 0x1) // -v
index 9fa7c74812b743adb474ebfd027fbb91ff6ca9ba..af4338006e431a522503e57a0a45389cb0347bd0 100644 (file)
@@ -315,7 +315,7 @@ int main(int argc, char *argv[])
                if (S_ISREG(cdir->dl_mode))
                        fmt_ull(cdir->dl_size);
                fmt_str("<td class=dt>");
-               tm = gmtime(&cdir->dl_mtime);
+               ptm = gmtime(&cdir->dl_mtime);
                fmt_04u(1900 + ptm->tm_year); *dst++ = '-';
                fmt_02u(ptm->tm_mon + 1); *dst++ = '-';
                fmt_02u(ptm->tm_mday); *dst++ = ' ';
index 78b0d0ade12ae537833aeb3f882c22a546125411..7b6bb2066e0e80b6a72da6038531b4128aabfc24 100644 (file)
@@ -64,6 +64,8 @@ struct dyn_lease* FAST_FUNC add_lease(
                oldest->hostname[0] = '\0';
                if (hostname) {
                        char *p;
+
+                       hostname_len++; /* include NUL */
                        if (hostname_len > sizeof(oldest->hostname))
                                hostname_len = sizeof(oldest->hostname);
                        p = safe_strncpy(oldest->hostname, hostname, hostname_len);
index 34f70ecfc303ce111a8e74098a8af1f9fddc549e..e018e877be3ed380767606eee0892e341be162fb 100644 (file)
@@ -5059,7 +5059,7 @@ static int is_hidden_fd(struct redirtab *rp, int fd)
                return 0;
        pf = g_parsefile;
        while (pf) {
-               if (fd == pf->fd) {
+               if (pf->fd > 0 && fd == pf->fd) {
                        return 1;
                }
                pf = pf->prev;
@@ -5424,7 +5424,11 @@ rmescapes(char *str, int flag)
                size_t fulllen = len + strlen(p) + 1;
 
                if (flag & RMESCAPE_GROW) {
+                       int strloc = str - (char *)stackblock();
                        r = makestrspace(fulllen, expdest);
+                       /* p and str may be invalidated by makestrspace */
+                       str = (char *)stackblock() + strloc;
+                       p = str + len;
                } else if (flag & RMESCAPE_HEAP) {
                        r = ckmalloc(fulllen);
                } else {
index 2aeb8e4406620394946e9e72e8c1ede4c7af07c6..78f0f7940d916dac9676ecdb60b073e567a56493 100644 (file)
@@ -6944,7 +6944,7 @@ int hush_main(int argc, char **argv)
                                /* -c 'script' (no params): prevent empty $0 */
                                G.global_argv--; /* points to argv[i] of 'script' */
                                G.global_argv[0] = argv[0];
-                               G.global_argc--;
+                               G.global_argc++;
                        } /* else -c 'script' ARG0 [ARG1...]: $0 is ARG0 */
                        init_sigmasks();
                        parse_and_run_string(optarg);
index e53ade9254b69e9afa43cb698c29480f1ba95bbc..325664d71e3dc67d28147dec5333c808b963eb32 100755 (executable)
@@ -32,7 +32,7 @@ rm -rf cpio.testdir cpio.testdir2 2>/dev/null
 # testing "test name" "command" "expected result" "file input" "stdin"
 
 testing "cpio extracts zero-sized hardlinks" \
-"$ECHO -ne '$hexdump' | bzcat | cpio -i; echo \$?;
+"$ECHO -ne '$hexdump' | bzcat | cpio -i 2>&1; echo \$?;
 ls -ln cpio.testdir | $FILTER_LS" \
 "\
 1 blocks
@@ -45,7 +45,7 @@ ls -ln cpio.testdir | $FILTER_LS" \
 
 test x"$SKIP_KNOWN_BUGS" = x"" && {
 # Currently fails. Numerous buglets: "1 blocks" versus "1 block",
-# "1 block" must go to stderr, does not list cpio.testdir/x and cpio.testdir/y
+# does not list cpio.testdir/x and cpio.testdir/y
 testing "cpio lists hardlinks" \
 "$ECHO -ne '$hexdump' | bzcat | cpio -t 2>&1; echo \$?" \
 "\
@@ -70,7 +70,7 @@ ln cpio.testdir/nonempty cpio.testdir/nonempty1
 mkdir cpio.testdir2
 
 testing "cpio extracts zero-sized hardlinks 2" \
-"find cpio.testdir | cpio -H newc --create | (cd cpio.testdir2 && cpio -i); echo \$?;
+"find cpio.testdir | cpio -H newc --create | (cd cpio.testdir2 && cpio -i 2>&1); echo \$?;
 ls -ln cpio.testdir2/cpio.testdir | $FILTER_LS" \
 "\
 2 blocks
@@ -87,7 +87,7 @@ ls -ln cpio.testdir2/cpio.testdir | $FILTER_LS" \
 # Was trying to create "/usr/bin", correct is "usr/bin".
 rm -rf cpio.testdir
 testing "cpio -p with absolute paths" \
-"echo /usr/bin | cpio -dp cpio.testdir; echo \$?;
+"echo /usr/bin | cpio -dp cpio.testdir 2>&1; echo \$?;
 ls cpio.testdir" \
 "\
 1 blocks
index 875c946be80dc810cf4c25ffc625de34b8b4a456..5b0750cac0cb880e12ec600a6f55021089537248 100755 (executable)
@@ -248,4 +248,28 @@ testing "sed beginning (^) matches only once" \
        ">/usr</>lib<\n" "" \
        "/usr/lib\n"
 
+testing "sed c" \
+       "sed 'crepl'" \
+       "repl\nrepl\n" "" \
+       "first\nsecond\n"
+
+testing "sed nested {}s" \
+       "sed '/asd/ { p; /s/ { s/s/c/ }; p; q }'" \
+       "qwe\nasd\nacd\nacd\n" "" \
+       "qwe\nasd\nzxc\n"
+
+testing "sed a cmd ended by double backslash" \
+       "sed -e '/| one /a \\
+       | three \\\\' -e '/| one-/a \\
+       | three-* \\\\'" \
+'      | one \\
+       | three \\
+       | two \\
+' '' \
+'      | one \\
+       | two \\
+'
+
+# testing "description" "arguments" "result" "infile" "stdin"
+
 exit $FAILCOUNT
index b8300570ef112d0a0750677fd241dc1798e7a2e5..416271b31561e78118d3dd4985d574e9bdb1eb23 100644 (file)
@@ -109,10 +109,53 @@ static void to_sys_clock(const char **pp_rtcname, int utc)
 
 static void from_sys_clock(const char **pp_rtcname, int utc)
 {
-#define TWEAK_USEC 200
-       struct tm tm_time;
+#if 1
        struct timeval tv;
+       struct tm tm_time;
+       int rtc;
+
+       rtc = rtc_xopen(pp_rtcname, O_WRONLY);
+       gettimeofday(&tv, NULL);
+       /* Prepare tm_time */
+       if (sizeof(time_t) == sizeof(tv.tv_sec)) {
+               if (utc)
+                       gmtime_r((time_t*)&tv.tv_sec, &tm_time);
+               else
+                       localtime_r((time_t*)&tv.tv_sec, &tm_time);
+       } else {
+               time_t t = tv.tv_sec;
+               if (utc)
+                       gmtime_r(&t, &tm_time);
+               else
+                       localtime_r(&t, &tm_time);
+       }
+#else
+/* Bloated code which tries to set hw clock with better precision.
+ * On x86, even though code does set hw clock within <1ms of exact
+ * whole seconds, apparently hw clock (at least on some machines)
+ * doesn't reset internal fractional seconds to 0,
+ * making all this a pointless excercise.
+ */
+       /* If we see that we are N usec away from whole second,
+        * we'll sleep for N-ADJ usecs. ADJ corrects for the fact
+        * that CPU is not infinitely fast.
+        * On infinitely fast CPU, next wakeup would be
+        * on (exactly_next_whole_second - ADJ). On real CPUs,
+        * this difference between current time and whole second
+        * is less than ADJ (assuming system isn't heavily loaded).
+        */
+       /* Small value of 256us gives very precise sync for 2+ GHz CPUs.
+        * Slower CPUs will fail to sync and will go to bigger
+        * ADJ values. qemu-emulated armv4tl with ~100 MHz
+        * performance ends up using ADJ ~= 4*1024 and it takes
+        * 2+ secs (2 tries with successively larger ADJ)
+        * to sync. Even straced one on the same qemu (very slow)
+        * takes only 4 tries.
+        */
+#define TWEAK_USEC 256
        unsigned adj = TWEAK_USEC;
+       struct tm tm_time;
+       struct timeval tv;
        int rtc = rtc_xopen(pp_rtcname, O_WRONLY);
 
        /* Try to catch the moment when whole second is close */
@@ -124,55 +167,64 @@ static void from_sys_clock(const char **pp_rtcname, int utc)
 
                t = tv.tv_sec;
                rem_usec = 1000000 - tv.tv_usec;
-               if (rem_usec < 1024) {
-                       /* Less than 1ms to next second. Good enough */
+               if (rem_usec < adj) {
+                       /* Close enough */
  small_rem:
                        t++;
                }
 
-               /* Prepare tm */
+               /* Prepare tm_time from t */
                if (utc)
                        gmtime_r(&t, &tm_time); /* may read /etc/xxx (it takes time) */
                else
                        localtime_r(&t, &tm_time); /* same */
-               tm_time.tm_isdst = 0;
+
+               if (adj >= 32*1024) {
+                       break; /* 32 ms diff and still no luck?? give up trying to sync */
+               }
 
                /* gmtime/localtime took some time, re-get cur time */
                gettimeofday(&tv, NULL);
 
-               if (tv.tv_sec < t /* may happen if rem_usec was < 1024 */
-                || (tv.tv_sec == t && tv.tv_usec < 1024)
+               if (tv.tv_sec < t /* we are still in old second */
+                || (tv.tv_sec == t && tv.tv_usec < adj) /* not too far into next second */
                ) {
-                       /* We are not too far into next second. Good. */
-                       break;
-               }
-               adj += 32; /* 2^(10-5) = 2^5 = 32 iterations max */
-               if (adj >= 1024) {
-                       /* Give up trying to sync */
-                       break;
+                       break; /* good, we are in sync! */
                }
 
-               /* Try to sync up by sleeping */
                rem_usec = 1000000 - tv.tv_usec;
-               if (rem_usec < 1024) {
-                       goto small_rem; /* already close, don't sleep */
+               if (rem_usec < adj) {
+                       t = tv.tv_sec;
+                       goto small_rem; /* already close to next sec, don't sleep */
                }
-               /* Need to sleep.
-                * Note that small adj on slow processors can make us
-                * to always overshoot tv.tv_usec < 1024 check on next
-                * iteration. That's why adj is increased on each iteration.
-                * This also allows it to be reused as a loop limiter.
-                */
-               usleep(rem_usec - adj);
-       }
 
-       xioctl(rtc, RTC_SET_TIME, &tm_time);
+               /* Try to sync up by sleeping */
+               usleep(rem_usec - adj);
 
-       /* Debug aid to find "good" TWEAK_USEC.
+               /* Jump to 1ms diff, then increase fast (x2): EVERY loop
+                * takes ~1 sec, people won't like slowly converging code here!
+                */
+       //bb_error_msg("adj:%d tv.tv_usec:%d", adj, (int)tv.tv_usec);
+               if (adj < 512)
+                       adj = 512;
+               /* ... and if last "overshoot" does not look insanely big,
+                * just use it as adj increment. This makes convergence faster.
+                */
+               if (tv.tv_usec < adj * 8) {
+                       adj += tv.tv_usec;
+                       continue;
+               }
+               adj *= 2;
+       }
+       /* Debug aid to find "optimal" TWEAK_USEC with nearly exact sync.
         * Look for a value which makes tv_usec close to 999999 or 0.
-        * for 2.20GHz Intel Core 2: TWEAK_USEC ~= 200
+        * For 2.20GHz Intel Core 2: optimal TWEAK_USEC ~= 200
         */
-       //bb_error_msg("tv.tv_usec:%d adj:%d", (int)tv.tv_usec, adj);
+       //bb_error_msg("tv.tv_usec:%d", (int)tv.tv_usec);
+#endif
+
+       tm_time.tm_isdst = 0;
+       xioctl(rtc, RTC_SET_TIME, &tm_time);
 
        if (ENABLE_FEATURE_CLEAN_UP)
                close(rtc);