diff options
author | João Valverde <joao.valverde@tecnico.ulisboa.pt> | 2017-10-24 21:04:19 +0100 |
---|---|---|
committer | João Valverde <j@v6e.pt> | 2017-10-25 19:10:57 +0000 |
commit | 90644c83725f979f82e7990792faaf6e847f74ea (patch) | |
tree | 2b0b1eccab4d2bc2296dc446e69f93a494e223bc /wsutil | |
parent | c5529dd7f1269e88b4c122fa3c755d5696b58a70 (diff) |
Remove inet_aton() usage everywhere
Including where it says not to in comments. Use IPv4 dotted-decimal
notation.
Change-Id: Iafe1f6fbd2bd5867c41642dc27411f47dff8ce6a
Reviewed-on: https://code.wireshark.org/review/24044
Petri-Dish: João Valverde <j@v6e.pt>
Tested-by: Petri Dish Buildbot
Reviewed-by: João Valverde <j@v6e.pt>
Diffstat (limited to 'wsutil')
-rw-r--r-- | wsutil/.editorconfig | 4 | ||||
-rw-r--r-- | wsutil/CMakeLists.txt | 4 | ||||
-rw-r--r-- | wsutil/Makefile.am | 2 | ||||
-rw-r--r-- | wsutil/inet_aton.c | 155 | ||||
-rw-r--r-- | wsutil/inet_aton.h | 52 |
5 files changed, 0 insertions, 217 deletions
diff --git a/wsutil/.editorconfig b/wsutil/.editorconfig index aed2ea1bca..f3406ee5a9 100644 --- a/wsutil/.editorconfig +++ b/wsutil/.editorconfig @@ -57,10 +57,6 @@ indent_size = tab indent_style = tab indent_size = tab -[inet_aton.[ch]] -indent_style = tab -indent_size = tab - [inet_ntop.[ch]] indent_style = tab indent_size = tab diff --git a/wsutil/CMakeLists.txt b/wsutil/CMakeLists.txt index 4f117a757a..e00c493d19 100644 --- a/wsutil/CMakeLists.txt +++ b/wsutil/CMakeLists.txt @@ -213,10 +213,6 @@ if(NOT HAVE_GETOPT_LONG) list(APPEND WSUTIL_FILES getopt_long.c) endif() -if(NOT HAVE_INET_ATON) - list(APPEND WSUTIL_FILES inet_aton.c) -endif() - if(NOT HAVE_POPCOUNT) list(APPEND WSUTIL_FILES popcount.c) endif() diff --git a/wsutil/Makefile.am b/wsutil/Makefile.am index 0c0feb2ce7..5f0fad20b8 100644 --- a/wsutil/Makefile.am +++ b/wsutil/Makefile.am @@ -192,8 +192,6 @@ EXTRA_DIST = \ file_util.h \ getopt_long.c \ getopt_long.h \ - inet_aton.c \ - inet_aton.h \ inet_ntop.c \ inet_pton.c \ popcount.c \ diff --git a/wsutil/inet_aton.c b/wsutil/inet_aton.c deleted file mode 100644 index 08939f7929..0000000000 --- a/wsutil/inet_aton.c +++ /dev/null @@ -1,155 +0,0 @@ -/* - * Copyright (c) 1983, 1990, 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. - * 3. 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 "inet_aton.h" - -#if defined(LIBC_SCCS) && !defined(lint) -static char sccsid[] = "@(#)inet_addr.c 8.1 (Berkeley) 6/17/93"; -#endif /* LIBC_SCCS and not lint */ - -#include <ctype.h> - -#ifdef HAVE_SYS_PARAM_H -#include <sys/param.h> -#endif - - - -/* - * Check whether "cp" is a valid ascii representation - * of an Internet address and convert to a binary address. - * Returns 1 if the address is valid, 0 if not. - * This replaces inet_addr, the return value from which - * cannot distinguish between failure and a local broadcast address. - */ -int -inet_aton(cp_arg, addr) - const char *cp_arg; - struct in_addr *addr; -{ - register const u_char *cp = cp_arg; - register u_long val; - register int base; - register size_t n; - register u_char c; - u_int parts[4]; - register u_int *pp = parts; - - for (;;) { - /* - * Collect number up to ``.''. - * Values are specified as for C: - * 0x=hex, 0=octal, other=decimal. - */ - val = 0; base = 10; - if (*cp == '0') { - if (*++cp == 'x' || *cp == 'X') - base = 16, cp++; - else - base = 8; - } - while ((c = *cp) != '\0') { - if (isascii(c) && isdigit(c)) { - val = (val * base) + (c - '0'); - cp++; - continue; - } - if (base == 16 && isascii(c) && isxdigit(c)) { - val = (val << 4) + - (c + 10 - (islower(c) ? 'a' : 'A')); - cp++; - continue; - } - break; - } - if (*cp == '.') { - /* - * Internet format: - * a.b.c.d - * a.b.c (with c treated as 16-bits) - * a.b (with b treated as 24 bits) - */ - if (pp >= parts + 3 || val > 0xff) - return (0); - *pp++ = val, cp++; - } else - break; - } - /* - * Check for trailing characters. - */ - if (*cp && (!isascii(*cp) || !isspace(*cp))) - return (0); - /* - * Concoct the address according to - * the number of parts specified. - */ - n = pp - parts + 1; - switch (n) { - - case 1: /* a -- 32 bits */ - break; - - case 2: /* a.b -- 8.24 bits */ - if (val > 0xffffff) - return (0); - val |= parts[0] << 24; - break; - - case 3: /* a.b.c -- 8.8.16 bits */ - if (val > 0xffff) - return (0); - val |= (parts[0] << 24) | (parts[1] << 16); - break; - - case 4: /* a.b.c.d -- 8.8.8.8 bits */ - if (val > 0xff) - return (0); - val |= (parts[0] << 24) | (parts[1] << 16) | (parts[2] << 8); - break; - } - if (addr) - addr->s_addr = htonl(val); - return (1); -} - -/* - * Editor modelines - http://www.wireshark.org/tools/modelines.html - * - * Local variables: - * c-basic-offset: 8 - * tab-width: 8 - * indent-tabs-mode: t - * End: - * - * vi: set shiftwidth=8 tabstop=8 noexpandtab: - * :indentSize=8:tabSize=8:noTabs=false: - */ diff --git a/wsutil/inet_aton.h b/wsutil/inet_aton.h deleted file mode 100644 index 3d89dc23c2..0000000000 --- a/wsutil/inet_aton.h +++ /dev/null @@ -1,52 +0,0 @@ -/* inet_aton.h - * - * Wireshark - Network traffic analyzer - * By Gerald Combs <gerald@wireshark.org> - * Copyright 1998 Gerald Combs - * - * This program is free software; you can redistribute it and/or - * modify it under the terms of the GNU General Public License - * as published by the Free Software Foundation; either version 2 - * of the License, or (at your option) any later version. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - * - * You should have received a copy of the GNU General Public License - * along with this program; if not, write to the Free Software - * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. - */ - -/* - * Version of "inet_aton()", for the benefit of OSes that don't have it. - */ - -#ifndef __INET_ATON_H__ -#define __INET_ATON_H__ - -#include "ws_symbol_export.h" - -#ifdef HAVE_SYS_SOCKET_H -#include <sys/socket.h> /* needed to define AF_ values on UNIX */ -#endif - -#ifdef HAVE_NETINET_IN_H -# include <netinet/in.h> -#endif - -#ifdef HAVE_ARPA_INET_H -#include <arpa/inet.h> -#endif - -#ifdef HAVE_WINSOCK2_H -#include <winsock2.h> -#endif - -#ifndef HAVE_INET_ATON -struct in_addr; -WS_DLL_PUBLIC int inet_aton(const char* cp_arg, struct in_addr *addr); -#endif - -#endif |