aboutsummaryrefslogtreecommitdiff
path: root/sim
diff options
context:
space:
mode:
authorOzkan Sezer <sezero@sourceware.org>2010-05-26 22:40:24 +0000
committerOzkan Sezer <sezero@sourceware.org>2010-05-26 22:40:24 +0000
commit363a6e9f2c3fe1e25f8fa986347948897f189f39 (patch)
tree71182a438b05d879f96dc50f3dcd232c29f312cb /sim
parent9db8dccb1711d06bd180437fcd8c78c5f0981193 (diff)
downloadgdb-363a6e9f2c3fe1e25f8fa986347948897f189f39.zip
gdb-363a6e9f2c3fe1e25f8fa986347948897f189f39.tar.gz
gdb-363a6e9f2c3fe1e25f8fa986347948897f189f39.tar.bz2
2010-05-26 Ozkan Sezer <sezeroz@gmail.com>
gdb/ * ser-tcp.c (net_open): Check error return from socket() call by its equality to -1 not by it being negative. (net_close): Likewise. gdb/gdbserver/ * gdbreplay.c (remote_open): Check error return from socket() call by its equality to -1 not by it being negative. * remote-utils.c (remote_open): Likewise. sim/arm/ * communicate.c (MYread_char): Check error return from accept() call by its equality to -1 not by it being negative. (MYread_charwait): Likewise. * main.c (main): Likewise for both socket() and accept() calls. sim/common/ * dv-sockser.c (dv_sockser_init): Check error return from socket() call by its equality to -1 not by it being negative. (connected_p): Likewise for accept() call. sim/cris/ * dv-rv.c (hw_rv_init_socket): Check error return from socket() call by its equality to -1 not by it being negative. (hw_rv_write): Likewise. (hw_rv_handle_incoming): Likewise. (hw_rv_poll_once): Likewise. * rvdummy.c (setupsocket): Likewise. (main): Likewise for accept() call as returned from setupsocket(). sim/m32c/ * main.c (setup_tcp_console): Check error return from socket() call by its equality to -1 not by it being negative.
Diffstat (limited to 'sim')
-rw-r--r--sim/arm/ChangeLog7
-rw-r--r--sim/arm/communicate.c4
-rw-r--r--sim/arm/main.c4
-rw-r--r--sim/common/ChangeLog6
-rw-r--r--sim/common/dv-sockser.c4
-rw-r--r--sim/cris/ChangeLog10
-rw-r--r--sim/cris/dv-rv.c8
-rw-r--r--sim/cris/rvdummy.c4
-rw-r--r--sim/m32c/ChangeLog5
-rw-r--r--sim/m32c/main.c2
10 files changed, 41 insertions, 13 deletions
diff --git a/sim/arm/ChangeLog b/sim/arm/ChangeLog
index 192f07e..1e07144 100644
--- a/sim/arm/ChangeLog
+++ b/sim/arm/ChangeLog
@@ -1,3 +1,10 @@
+2010-05-26 Ozkan Sezer <sezeroz@gmail.com>
+
+ * communicate.c (MYread_char): Check error return from accept() call
+ by its equality to -1 not by it being negative.
+ (MYread_charwait): Likewise.
+ * main.c (main): Likewise for both socket() and accept() calls.
+
2010-04-14 Mike Frysinger <vapier@gentoo.org>
* wrapper.c (sim_write): Add const to buffer arg.
diff --git a/sim/arm/communicate.c b/sim/arm/communicate.c
index 2fabbaa..fac2b60 100644
--- a/sim/arm/communicate.c
+++ b/sim/arm/communicate.c
@@ -83,7 +83,7 @@ retry:
return -1;
fprintf (stderr, "Waiting for connection from debugger...");
debugsock = accept (sockethandle, &isa, &i);
- if (debugsock < 0)
+ if (debugsock == -1)
{ /* Now we are in serious trouble... */
perror ("accept");
return -1;
@@ -138,7 +138,7 @@ retry:
return -1;
fprintf (stderr, "Waiting for connection from debugger...");
debugsock = accept (sockethandle, &isa, &i);
- if (debugsock < 0)
+ if (debugsock == -1)
{ /* Now we are in serious trouble... */
perror ("accept");
return -1;
diff --git a/sim/arm/main.c b/sim/arm/main.c
index 1841ae4..9bc20fe 100644
--- a/sim/arm/main.c
+++ b/sim/arm/main.c
@@ -117,7 +117,7 @@ main (int argc, char *argv[])
/* Open a socket */
sockethandle = socket (hp->h_addrtype, SOCK_STREAM, 0);
- if (sockethandle < 0)
+ if (sockethandle == -1)
{
perror ("socket");
return 1;
@@ -147,7 +147,7 @@ main (int argc, char *argv[])
fprintf (stderr, "Waiting for connection from debugger...");
debugsock = accept (sockethandle, &isa, &i);
- if (debugsock < 0)
+ if (debugsock == -1)
{
perror ("accept");
return 1;
diff --git a/sim/common/ChangeLog b/sim/common/ChangeLog
index 3aa491a..2eebba8 100644
--- a/sim/common/ChangeLog
+++ b/sim/common/ChangeLog
@@ -1,3 +1,9 @@
+2010-05-26 Ozkan Sezer <sezeroz@gmail.com>
+
+ * dv-sockser.c (dv_sockser_init): Check error return from socket()
+ call by its equality to -1 not by it being negative.
+ (connected_p): Likewise for accept() call.
+
2010-04-23 Mike Frysinger <vapier@gentoo.org>
* sim-fpu.c (sim_fpu_zero, sim_fpu_qnan): Add 0 initializers.
diff --git a/sim/common/dv-sockser.c b/sim/common/dv-sockser.c
index a8626cd..b09d9c9 100644
--- a/sim/common/dv-sockser.c
+++ b/sim/common/dv-sockser.c
@@ -166,7 +166,7 @@ dv_sockser_init (SIM_DESC sd)
}
sockser_listen_fd = socket (PF_INET, SOCK_STREAM, 0);
- if (sockser_listen_fd < 0)
+ if (sockser_listen_fd == -1)
{
sim_io_eprintf (sd, "sockser init: unable to get socket: %s\n",
strerror (errno));
@@ -274,7 +274,7 @@ connected_p (SIM_DESC sd)
addrlen = sizeof (sockaddr);
sockser_fd = accept (sockser_listen_fd, &sockaddr, &addrlen);
- if (sockser_fd < 0)
+ if (sockser_fd == -1)
return 0;
/* Set non-blocking i/o. */
diff --git a/sim/cris/ChangeLog b/sim/cris/ChangeLog
index d7aadd0..699e9e8 100644
--- a/sim/cris/ChangeLog
+++ b/sim/cris/ChangeLog
@@ -1,3 +1,13 @@
+2010-05-26 Ozkan Sezer <sezeroz@gmail.com>
+
+ * dv-rv.c (hw_rv_init_socket): Check error return from socket() call
+ by its equality to -1 not by it being negative.
+ (hw_rv_write): Likewise.
+ (hw_rv_handle_incoming): Likewise.
+ (hw_rv_poll_once): Likewise.
+ * rvdummy.c (setupsocket): Likewise.
+ (main): Likewise for accept() call as returned from setupsocket().
+
2010-03-30 Mike Frysinger <vapier@gentoo.org>
* devices.c (device_error): Add const to message.
diff --git a/sim/cris/dv-rv.c b/sim/cris/dv-rv.c
index bdc6f2e..45ef79d 100644
--- a/sim/cris/dv-rv.c
+++ b/sim/cris/dv-rv.c
@@ -404,7 +404,7 @@ hw_rv_write (struct hw *me,
/* If we don't have a valid fd here, it's because we got an error
initially, and we suppressed that error. */
- if (rv->fd < 0)
+ if (rv->fd == -1)
hw_abort (me, "couldn't open a connection to %s:%d because: %s",
rv->host, rv->port, strerror (rv->saved_errno));
@@ -637,7 +637,7 @@ hw_rv_handle_incoming (struct hw *me,
{
hw_rv_read (me, cbuf, 3);
- if (rv->fd < 0)
+ if (rv->fd == -1)
return;
len = cbuf[0] + cbuf[1] * 256 - 3;
@@ -723,7 +723,7 @@ hw_rv_poll_once (struct hw *me)
int ret;
struct timeval tv;
- if (rv->fd < 0)
+ if (rv->fd == -1)
/* Connection has died or was never initiated. */
return;
@@ -887,7 +887,7 @@ hw_rv_init_socket (struct hw *me)
server.sin_port = htons (rv->port);
sock = socket (AF_INET, SOCK_STREAM, 0);
- if (sock < 0)
+ if (sock == -1)
hw_abort (me, "can't get a socket for %s:%d connection",
rv->host, rv->port);
diff --git a/sim/cris/rvdummy.c b/sim/cris/rvdummy.c
index edcd8d2..c716b8d 100644
--- a/sim/cris/rvdummy.c
+++ b/sim/cris/rvdummy.c
@@ -118,7 +118,7 @@ int setupsocket (void)
memset (&sa_in, 0, sizeof (sa_in));
s = socket (AF_INET, SOCK_STREAM, 0);
- if (s < 0)
+ if (s == -1)
return -1;
if (setsockopt (s, SOL_SOCKET, SO_REUSEADDR, &reuse, sizeof reuse) != 0)
@@ -517,7 +517,7 @@ main (int argc, char *argv[])
}
fd = setupsocket ();
- if (fd < 0)
+ if (fd == -1)
{
fprintf (stderr, "%s: problem setting up the connection: %s\n",
progname, strerror (errno));
diff --git a/sim/m32c/ChangeLog b/sim/m32c/ChangeLog
index 9827937..49dfa03 100644
--- a/sim/m32c/ChangeLog
+++ b/sim/m32c/ChangeLog
@@ -1,3 +1,8 @@
+2010-05-26 Ozkan Sezer <sezeroz@gmail.com>
+
+ * main.c (setup_tcp_console): Check error return from socket() call
+ by its equality to -1 not by it being negative.
+
2010-04-14 Mike Frysinger <vapier@gentoo.org>
* gdb-if.c (sim_write): Add const to buf arg.
diff --git a/sim/m32c/main.c b/sim/m32c/main.c
index 589a600..2f31242 100644
--- a/sim/m32c/main.c
+++ b/sim/m32c/main.c
@@ -98,7 +98,7 @@ setup_tcp_console (char *portname)
address.sin_port = htons (port);
isocket = socket (AF_INET, SOCK_STREAM, 0);
- if (isocket < 0)
+ if (isocket == -1)
{
perror ("socket");
exit (1);