]> git.ipfire.org Git - thirdparty/dbus.git/commitdiff
Remove fallback implementation of strtoll(), strtoull()
authorSimon McVittie <smcv@collabora.com>
Tue, 19 Jul 2022 19:42:51 +0000 (20:42 +0100)
committerSimon McVittie <smcv@collabora.com>
Tue, 19 Jul 2022 20:11:09 +0000 (21:11 +0100)
We now require a (mostly-)C99 compiler, which guarantees the presence
of these functions.

Signed-off-by: Simon McVittie <smcv@collabora.com>
cmake/ConfigureChecks.cmake
cmake/config.h.cmake
configure.ac
meson.build
tools/Makefile.am
tools/dbus-send.c
tools/strtoll.c [deleted file]
tools/strtoull.c [deleted file]

index f4680f9b20c06051701d213e61b1c1cb47e8e357..3d4e0f2beb8e46ad0380fe33a3d969d8e85b7f0c 100644 (file)
@@ -50,8 +50,6 @@ check_symbol_exists(socketpair   "sys/socket.h"     HAVE_SOCKETPAIR)         #
 check_symbol_exists(setlocale    "locale.h"         HAVE_SETLOCALE)          #  dbus-test-main.c
 check_symbol_exists(localeconv   "locale.h"         HAVE_LOCALECONV)         #  dbus-sysdeps.c
 check_symbol_exists(poll         "poll.h"           HAVE_POLL)               #  dbus-sysdeps-unix.c
-check_symbol_exists(strtoll      "stdlib.h"         HAVE_STRTOLL)            #  dbus-send.c
-check_symbol_exists(strtoull     "stdlib.h"         HAVE_STRTOULL)           #  dbus-send.c
 set(CMAKE_REQUIRED_DEFINITIONS -D_GNU_SOURCE)
 check_symbol_exists(pipe2        "fcntl.h;unistd.h"         HAVE_PIPE2)
 check_symbol_exists(accept4      "sys/socket.h"             HAVE_ACCEPT4)
index 44b79c03b5478df28d5d25a5f8f8a6e382850eea..8899214931e92997b5d44afd3794ae9f15b9cf52 100644 (file)
 /* Define to 1 if you have localeconv */
 #cmakedefine   HAVE_LOCALECONV 1
 
-/* Define to 1 if you have strtoll */
-#cmakedefine   HAVE_STRTOLL 1
-
-/* Define to 1 if you have strtoull */
-#cmakedefine   HAVE_STRTOULL 1
-
 /* Define to 1 if you have pip2 */
 #cmakedefine   HAVE_PIPE2 1
 
 #  define gid_t int
 # else
    typedef int mode_t;
-#  if !defined(_WIN32_WCE)
-#    define strtoll _strtoi64
-#    define strtoull _strtoui64
-#    define HAVE_STRTOLL 1
-#    define HAVE_STRTOULL 1
-#  endif
 # endif
 #endif // defined(_WIN32) || defined(_WIN64)
 
index e4b75e4276e5e2bfd1aff5205f49cdcaf2ac861b..73ca1d23c15fdfca0890ceaf272d9f5f14ef90a7 100644 (file)
@@ -403,8 +403,6 @@ setlocale
 setresuid
 setrlimit
 socketpair
-strtoll
-strtoull
 unsetenv
 usleep
 ])
index c6089c4b8de19e06ad59d661a227d37437c6c135..a97e90cf1ed177514286484b01ffd5ca48f747a3 100644 (file)
@@ -595,8 +595,6 @@ check_functions = [
     'setresuid',
     'setrlimit',
     'socketpair',
-    'strtoll',
-    'strtoull',
     'unsetenv',
     'usleep',
 ]
index dee5f8621346a70d1c430592eac185f5ae74e084..5488b54d62d3d58e7a53e27d42a22149d466a8b4 100644 (file)
@@ -140,7 +140,7 @@ SUFFIXES = .rc
 dbus_update_activation_environment_SOURCES += disable-uac.rc
 endif
 
-EXTRA_DIST = run-with-tmp-session-bus.sh strtoll.c strtoull.c Win32.Manifest
+EXTRA_DIST = run-with-tmp-session-bus.sh Win32.Manifest
 CLEANFILES =                           \
        run-with-tmp-session-bus.conf \
        $(nodist_dbus_update_activation_environment_SOURCES)
index 86858c505cb0e4cda75b8deac8bfffc5c06e10de..3063290f16ac0c8dd226767c6b76df735a29587b 100644 (file)
 #include <dbus/dbus.h>
 #include "dbus/dbus-internals.h"
 
-#ifndef HAVE_STRTOLL
-#undef strtoll
-#define strtoll mystrtoll
-#include "strtoll.c"
-#endif
-
-#ifndef HAVE_STRTOULL
-#undef strtoull
-#define strtoull mystrtoull
-#include "strtoull.c"
-#endif
-
 #ifdef DBUS_WINCE
 #ifndef strdup
 #define strdup _strdup
diff --git a/tools/strtoll.c b/tools/strtoll.c
deleted file mode 100644 (file)
index 00f14c3..0000000
+++ /dev/null
@@ -1,166 +0,0 @@
-/*-
- * Copyright (c) 1992, 1993
- *     The Regents of the University of California.  All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- * 4. Neither the name of the University nor the names of its contributors
- *    may be used to endorse or promote products derived from this software
- *    without specific prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY THE REGENTS AND CONTRIBUTORS ``AS IS'' AND
- * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
- * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
- * ARE DISCLAIMED.  IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE LIABLE
- * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
- * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
- * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
- * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
- * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
- * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
- * SUCH DAMAGE.
- */
-
-#include "config.h"
-
-#include <limits.h>
-#ifdef HAVE_ERRNO_H
-#include <errno.h>
-#endif
-#include <stdlib.h>
-#ifdef DBUS_WINCE
-#include <windows.h>
-#endif
-
-#ifndef isspace
-#define isspace(c) ((c) == ' ' || (c) == '\t' || (c) == '\r' || (c) == '\n')
-#endif
-
-/* Minimum and maximum values a `signed long long int' can hold.  */
-#ifndef LLONG_MAX
-#   define LLONG_MAX   9223372036854775807LL
-#endif
-
-#ifndef LLONG_MIN
-#   define LLONG_MIN   (-LLONG_MAX - 1LL)
-#endif
-/* Maximum value an `unsigned long long int' can hold.  (Minimum is 0.)  */
-#ifndef ULLONG_MAX
-#   define ULLONG_MAX  18446744073709551615ULL
-#endif
-/*
- * Convert a string to a long long integer.
- *
- * Assumes that the upper and lower case
- * alphabets and digits are each contiguous.
- */
-long long strtoll (const char*, char **, int);
-
-long long
-strtoll(const char * nptr, char ** endptr, int base)
-{
-       const char *s;
-       unsigned long long acc;
-       char c;
-       unsigned long long cutoff;
-       int neg, any, cutlim;
-
-       /*
-        * Skip white space and pick up leading +/- sign if any.
-        * If base is 0, allow 0x for hex and 0 for octal, else
-        * assume decimal; if base is already 16, allow 0x.
-        */
-       s = nptr;
-       do {
-               c = *s++;
-       } while (isspace((unsigned char)c));
-       if (c == '-') {
-               neg = 1;
-               c = *s++;
-       } else {
-               neg = 0;
-               if (c == '+')
-                       c = *s++;
-       }
-       if ((base == 0 || base == 16) &&
-           c == '0' && (*s == 'x' || *s == 'X') &&
-           ((s[1] >= '0' && s[1] <= '9') ||
-           (s[1] >= 'A' && s[1] <= 'F') ||
-           (s[1] >= 'a' && s[1] <= 'f'))) {
-               c = s[1];
-               s += 2;
-               base = 16;
-       }
-       if (base == 0)
-               base = c == '0' ? 8 : 10;
-       acc = any = 0;
-       if (base < 2 || base > 36)
-               goto noconv;
-
-       /*
-        * Compute the cutoff value between legal numbers and illegal
-        * numbers.  That is the largest legal value, divided by the
-        * base.  An input number that is greater than this value, if
-        * followed by a legal input character, is too big.  One that
-        * is equal to this value may be valid or not; the limit
-        * between valid and invalid numbers is then based on the last
-        * digit.  For instance, if the range for quads is
-        * [-9223372036854775808..9223372036854775807] and the input base
-        * is 10, cutoff will be set to 922337203685477580 and cutlim to
-        * either 7 (neg==0) or 8 (neg==1), meaning that if we have
-        * accumulated a value > 922337203685477580, or equal but the
-        * next digit is > 7 (or 8), the number is too big, and we will
-        * return a range error.
-        *
-        * Set 'any' if any `digits' consumed; make it negative to indicate
-        * overflow.
-        */
-       cutoff = neg ? (unsigned long long)-(LLONG_MIN + LLONG_MAX) + LLONG_MAX
-           : LLONG_MAX;
-       cutlim = cutoff % base;
-       cutoff /= base;
-       for ( ; ; c = *s++) {
-               if (c >= '0' && c <= '9')
-                       c -= '0';
-               else if (c >= 'A' && c <= 'Z')
-                       c -= 'A' - 10;
-               else if (c >= 'a' && c <= 'z')
-                       c -= 'a' - 10;
-               else
-                       break;
-               if (c >= base)
-                       break;
-               if (any < 0 || acc > cutoff || (acc == cutoff && c > cutlim))
-                       any = -1;
-               else {
-                       any = 1;
-                       acc *= base;
-                       acc += c;
-               }
-       }
-       if (any < 0) {
-               acc = neg ? LLONG_MIN : LLONG_MAX;
-#ifdef DBUS_WINCE
-               SetLastError (ERROR_ARITHMETIC_OVERFLOW);
-#else
-               errno = ERANGE;
-#endif
-       } else if (!any) {
-noconv:
-#ifdef DBUS_WINCE
-               SetLastError (ERROR_INVALID_PARAMETER);
-#else
-               errno = EINVAL;
-#endif
-       } else if (neg)
-               acc = -acc;
-       if (endptr != NULL)
-               *endptr = (char *)(any ? s - 1 : nptr);
-       return (acc);
-}
diff --git a/tools/strtoull.c b/tools/strtoull.c
deleted file mode 100644 (file)
index 3559554..0000000
+++ /dev/null
@@ -1,145 +0,0 @@
-/*-
- * Copyright (c) 1992, 1993
- *     The Regents of the University of California.  All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- * 4. Neither the name of the University nor the names of its contributors
- *    may be used to endorse or promote products derived from this software
- *    without specific prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY THE REGENTS AND CONTRIBUTORS ``AS IS'' AND
- * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
- * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
- * ARE DISCLAIMED.  IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE LIABLE
- * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
- * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
- * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
- * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
- * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
- * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
- * SUCH DAMAGE.
- */
-
-#include "config.h"
-
-#include <limits.h>
-#ifdef HAVE_ERRNO_H
-#include <errno.h>
-#endif
-#include <stdlib.h>
-#ifdef DBUS_WINCE
-#include <windows.h>
-#endif
-
-#ifndef isspace
-#define isspace(c) ((c) == ' ' || (c) == '\t' || (c) == '\r' || (c) == '\n')
-#endif
-
-/* Minimum and maximum values a `signed long long int' can hold.  */
-#ifndef LLONG_MAX
-#   define LLONG_MAX   9223372036854775807LL
-#endif
-
-#ifndef LLONG_MIN
-#   define LLONG_MIN   (-LLONG_MAX - 1LL)
-#endif
-/* Maximum value an `unsigned long long int' can hold.  (Minimum is 0.)  */
-#ifndef ULLONG_MAX
-#   define ULLONG_MAX  18446744073709551615ULL
-#endif
-
-/*
- * Convert a string to an unsigned long long integer.
- *
- * Assumes that the upper and lower case
- * alphabets and digits are each contiguous.
- */
-unsigned long long strtoull (const char *, char **, int);
-
-unsigned long long
-strtoull(const char * nptr, char ** endptr, int base)
-{
-       const char *s;
-       unsigned long long acc;
-       char c;
-       unsigned long long cutoff;
-       int neg, any, cutlim;
-
-       /*
-        * See strtoq for comments as to the logic used.
-        */
-       s = nptr;
-       do {
-               c = *s++;
-       } while (isspace((unsigned char)c));
-       if (c == '-') {
-               neg = 1;
-               c = *s++;
-       } else {
-               neg = 0;
-               if (c == '+')
-                       c = *s++;
-       }
-       if ((base == 0 || base == 16) &&
-           c == '0' && (*s == 'x' || *s == 'X') &&
-           ((s[1] >= '0' && s[1] <= '9') ||
-           (s[1] >= 'A' && s[1] <= 'F') ||
-           (s[1] >= 'a' && s[1] <= 'f'))) {
-               c = s[1];
-               s += 2;
-               base = 16;
-       }
-       if (base == 0)
-               base = c == '0' ? 8 : 10;
-       acc = any = 0;
-       if (base < 2 || base > 36)
-               goto noconv;
-
-       cutoff = ULLONG_MAX / base;
-       cutlim = ULLONG_MAX % base;
-       for ( ; ; c = *s++) {
-               if (c >= '0' && c <= '9')
-                       c -= '0';
-               else if (c >= 'A' && c <= 'Z')
-                       c -= 'A' - 10;
-               else if (c >= 'a' && c <= 'z')
-                       c -= 'a' - 10;
-               else
-                       break;
-               if (c >= base)
-                       break;
-               if (any < 0 || acc > cutoff || (acc == cutoff && c > cutlim))
-                       any = -1;
-               else {
-                       any = 1;
-                       acc *= base;
-                       acc += c;
-               }
-       }
-       if (any < 0) {
-               acc = ULLONG_MAX;
-#ifdef DBUS_WINCE
-               SetLastError (ERROR_ARITHMETIC_OVERFLOW);
-#else
-               errno = ERANGE;
-#endif
-       } else if (!any) {
-noconv:
-#ifdef DBUS_WINCE
-               SetLastError (ERROR_INVALID_PARAMETER);
-#else
-               errno = EINVAL;
-#endif
-       } else if (neg)
-               acc = -acc;
-       if (endptr != NULL)
-               *endptr = (char *)(any ? s - 1 : nptr);
-       return (acc);
-}