This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[commit/obvious] PR remote/17946: Fix wrong comparison of pointer against char
- From: Sergio Durigan Junior <sergiodj at redhat dot com>
- To: GDB Patches <gdb-patches at sourceware dot org>
- Cc: Sergio Durigan Junior <sergiodj at redhat dot com>
- Date: Mon, 9 Feb 2015 19:01:46 -0500
- Subject: [commit/obvious] PR remote/17946: Fix wrong comparison of pointer against char
- Authentication-results: sourceware.org; auth=none
We were comparing a pointer against a char on remote.c. 'dcb' filed a
bug to inform us about that. I pushed the following patch under the
obvious rule.
gdb/ChangeLog:
2015-02-09 Sergio Durigan Junior <sergiodj@redhat.com>
PR remote/17946
* gdb/remote.c (remote_parse_stop_reply): Fix wrong comparison
of pointer against char.
---
gdb/ChangeLog | 6 ++++++
gdb/remote.c | 2 +-
2 files changed, 7 insertions(+), 1 deletion(-)
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 7b989d8..be32f3e 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,9 @@
+2015-02-09 Sergio Durigan Junior <sergiodj@redhat.com>
+
+ PR remote/17946
+ * gdb/remote.c (remote_parse_stop_reply): Fix wrong comparison
+ of pointer against char.
+
2015-02-09 Mark Wielaard <mjw@redhat.com>
* c-typeprint.c (cp_type_print_method_args): Handle '_Atomic'.
diff --git a/gdb/remote.c b/gdb/remote.c
index 3980aad..dbfc10b 100644
--- a/gdb/remote.c
+++ b/gdb/remote.c
@@ -5682,7 +5682,7 @@ Packet: '%s'\n"),
{
p++;
- if (p == '\0')
+ if (*p == '\0')
;
else if (strncmp (p,
"process:", sizeof ("process:") - 1) == 0)
--
1.9.3