Scrub things and ditch uint64_t and use a custom type name

to avoid C lib compatibility problems.

2 files changed, 26 insertions, 14 deletions

diff --git a/coreutils/dos2unix.c b/coreutils/dos2unix.c
index e110680c3..8b65d05de 100644
--- a/coreutils/dos2unix.c
+++ b/coreutils/dos2unix.c
@@ -30,14 +30,18 @@
 #include <string.h>
 #include <getopt.h>
 #include <unistd.h>
+#include <stdint.h>
 #include <fcntl.h>
 #include <sys/time.h>
 #include "busybox.h"
 
-/* Teach older glibc and libc5 what a uint64_t is */
-#if (__GLIBC__ <= 2) && (__GLIBC_MINOR__ < 3)
-typedef unsigned long int       uint64_t;
-#endif
+/* We are making a lame pseudo-random string generator here.  in
+ * convert(), each pass through the while loop will add more and more
+ * stuff into value, which is _supposed_ to wrap.  We don't care about
+ * it being accurate.  We care about it being messy, since we then mod
+ * it by the sizeof(letters) and then use that as an index into letters
+ * to pick a random letter to add to out temporary file. */
+typedef unsigned long int bb_uint64_t;
 
 static const char letters[] = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789";
 
@@ -47,7 +51,7 @@ static int convert(char *fn, int ConvType)
         int c, fd;
         struct timeval tv;
         char tempFn[BUFSIZ];
-        static uint64_t value=0;
+        static bb_uint64_t value=0;
         FILE *in = stdin, *out = stdout;
 
         if (fn != NULL) {
@@ -64,7 +68,7 @@ static int convert(char *fn, int ConvType)
                      * random filename based (and in the same dir as)
                      * the input file... */
                     gettimeofday (&tv, NULL);
-                    value += ((uint64_t) tv.tv_usec << 16) ^ tv.tv_sec ^ getpid ();
+                    value += ((bb_uint64_t) tv.tv_usec << 16) ^ tv.tv_sec ^ getpid ();
                     tempFn[++c] = letters[value % 62];
                     tempFn[c+1] = '\0';
                     value /= 62;
@@ -138,7 +142,9 @@ static int convert(char *fn, int ConvType)
                 return -2;
             }
 
-            /* Assume they are both on the same filesystem */
+            /* Assume they are both on the same filesystem (which
+             * should be true since we put them into the same directory
+             * so we _should_ be ok, but you never know... */
             if (rename(tempFn, fn) < 0) {
                 perror_msg("unable to rename '%s' as '%s'", tempFn, fn);
                 return -1;
diff --git a/dos2unix.c b/dos2unix.c
index e110680c3..8b65d05de 100644
--- a/dos2unix.c
+++ b/dos2unix.c
@@ -30,14 +30,18 @@
 #include <string.h>
 #include <getopt.h>
 #include <unistd.h>
+#include <stdint.h>
 #include <fcntl.h>
 #include <sys/time.h>
 #include "busybox.h"
 
-/* Teach older glibc and libc5 what a uint64_t is */
-#if (__GLIBC__ <= 2) && (__GLIBC_MINOR__ < 3)
-typedef unsigned long int       uint64_t;
-#endif
+/* We are making a lame pseudo-random string generator here.  in
+ * convert(), each pass through the while loop will add more and more
+ * stuff into value, which is _supposed_ to wrap.  We don't care about
+ * it being accurate.  We care about it being messy, since we then mod
+ * it by the sizeof(letters) and then use that as an index into letters
+ * to pick a random letter to add to out temporary file. */
+typedef unsigned long int bb_uint64_t;
 
 static const char letters[] = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789";
 
@@ -47,7 +51,7 @@ static int convert(char *fn, int ConvType)
         int c, fd;
         struct timeval tv;
         char tempFn[BUFSIZ];
-        static uint64_t value=0;
+        static bb_uint64_t value=0;
         FILE *in = stdin, *out = stdout;
 
         if (fn != NULL) {
@@ -64,7 +68,7 @@ static int convert(char *fn, int ConvType)
                      * random filename based (and in the same dir as)
                      * the input file... */
                     gettimeofday (&tv, NULL);
-                    value += ((uint64_t) tv.tv_usec << 16) ^ tv.tv_sec ^ getpid ();
+                    value += ((bb_uint64_t) tv.tv_usec << 16) ^ tv.tv_sec ^ getpid ();
                     tempFn[++c] = letters[value % 62];
                     tempFn[c+1] = '\0';
                     value /= 62;
@@ -138,7 +142,9 @@ static int convert(char *fn, int ConvType)
                 return -2;
             }
 
-            /* Assume they are both on the same filesystem */
+            /* Assume they are both on the same filesystem (which
+             * should be true since we put them into the same directory
+             * so we _should_ be ok, but you never know... */
             if (rename(tempFn, fn) < 0) {
                 perror_msg("unable to rename '%s' as '%s'", tempFn, fn);
                 return -1;