aboutsummaryrefslogtreecommitdiff
path: root/gdb
diff options
context:
space:
mode:
authorMartin Galvan <martin.galvan@tallertechnologies.com>2014-11-30 19:34:15 +0400
committerJoel Brobecker <brobecker@adacore.com>2014-11-30 19:37:31 +0400
commitf0d4ba1fc4c702890bf090d1d6d2413fc6c34dd0 (patch)
tree9cc4f91784f0301b33ceec87d5c8e429f23a5d49 /gdb
parentdb76a70026ab100148eb274322fac01f1a1dd466 (diff)
downloadgdb-f0d4ba1fc4c702890bf090d1d6d2413fc6c34dd0.zip
gdb-f0d4ba1fc4c702890bf090d1d6d2413fc6c34dd0.tar.gz
gdb-f0d4ba1fc4c702890bf090d1d6d2413fc6c34dd0.tar.bz2
frame.c: Fix the check for FID_STACK_INVALID in frame_id_eq()
I noticed in frame_id_eq() we were checking for the "l" frame_id being invalid twice instead of checking both "l" and "r", so this patch corrects it. gdb/ChangeLog: * frame.c (frame_id_eq): Fix the check for FID_STACK_INVALID.
Diffstat (limited to 'gdb')
-rw-r--r--gdb/ChangeLog5
-rw-r--r--gdb/frame.c2
2 files changed, 6 insertions, 1 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 4112438..f706f29 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2014-11-30 Martin Galvan <martin.galvan@tallertechnologies.com> (tiny patch, obvious)
+
+ Pushed by Joel Brobecker <brobecker@adacore.com>.
+ * frame.c (frame_id_eq): Fix the check for FID_STACK_INVALID.
+
2014-11-29 Siva Chandra Reddy <sivachandra@google.com>
* eval.c (evaluate_subexp): Check that the thread stack temporaries
diff --git a/gdb/frame.c b/gdb/frame.c
index 0663af9..75ad341 100644
--- a/gdb/frame.c
+++ b/gdb/frame.c
@@ -614,7 +614,7 @@ frame_id_eq (struct frame_id l, struct frame_id r)
outer_frame_id. */
eq = 1;
else if (l.stack_status == FID_STACK_INVALID
- || l.stack_status == FID_STACK_INVALID)
+ || r.stack_status == FID_STACK_INVALID)
/* Like a NaN, if either ID is invalid, the result is false.
Note that a frame ID is invalid iff it is the null frame ID. */
eq = 0;