diff options
author | Andrew Cagney <cagney@redhat.com> | 2001-11-15 02:56:29 +0000 |
---|---|---|
committer | Andrew Cagney <cagney@redhat.com> | 2001-11-15 02:56:29 +0000 |
commit | 1ff9c3d6f7f11b7f95019062eade17e9f4bcd065 (patch) | |
tree | 47b2a167582e330712581a78e64aa2ff0526a03f | |
parent | 86884b3733ff6a828afe732e7b43ce23c5d0ea6b (diff) | |
download | gdb-1ff9c3d6f7f11b7f95019062eade17e9f4bcd065.zip gdb-1ff9c3d6f7f11b7f95019062eade17e9f4bcd065.tar.gz gdb-1ff9c3d6f7f11b7f95019062eade17e9f4bcd065.tar.bz2 |
* remote.c: Include "gdb_assert.h".
(tty_input): Wire buffer to 400 bytes.
(readsocket): Check tty_input doesn't overflow.
(build_remote_gdbarch_data, _initialize_remote): Don't multi-arch
tty_input.
-rw-r--r-- | gdb/ChangeLog | 8 | ||||
-rw-r--r-- | gdb/remote.c | 11 |
2 files changed, 13 insertions, 6 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index fc02819..009d948 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,11 @@ +2001-11-14 Andrew Cagney <ac131313@redhat.com> + + * remote.c: Include "gdb_assert.h". + (tty_input): Wire buffer to 400 bytes. + (readsocket): Check tty_input doesn't overflow. + (build_remote_gdbarch_data, _initialize_remote): Don't multi-arch + tty_input. + 2001-11-14 Michael Snyder <msnyder@redhat.com> * d10v-tdep.c (d10v_pointer_to_address): Use new type flag diff --git a/gdb/remote.c b/gdb/remote.c index e7b7169..b588f33 100644 --- a/gdb/remote.c +++ b/gdb/remote.c @@ -37,6 +37,7 @@ #include "remote.h" #include "regcache.h" #include "value.h" +#include "gdb_assert.h" #include <ctype.h> #include <sys/time.h> @@ -5404,8 +5405,9 @@ enum } minitelnet_return; -/* shared between readsocket() and readtty() */ -static char *tty_input; +/* Shared between readsocket() and readtty(). The size is arbitrary, + however all targets are known to support a 400 character packet. */ +static char tty_input[400]; static int escape_count; static int echo_check; @@ -5451,6 +5453,7 @@ readsocket (void) { if (tty_input[echo_check] == data) { + gdb_assert (echo_check <= sizeof (tty_input)); echo_check++; /* Character matched user input: */ continue; /* Continue without echoing it. */ } @@ -5787,9 +5790,6 @@ static void build_remote_gdbarch_data (void) { build_remote_packet_sizes (); - - /* Cisco stuff */ - tty_input = xmalloc (PBUFSIZ); remote_address_size = TARGET_ADDR_BIT; } @@ -5819,7 +5819,6 @@ _initialize_remote (void) /* architecture specific data */ build_remote_gdbarch_data (); - register_gdbarch_swap (&tty_input, sizeof (&tty_input), NULL); register_remote_packet_sizes (); register_gdbarch_swap (&remote_address_size, sizeof (&remote_address_size), NULL); |