aboutsummaryrefslogtreecommitdiff
path: root/gdb/exec.c
diff options
context:
space:
mode:
authorLuis Machado <lgustavo@codesourcery.com>2016-10-24 17:44:56 -0500
committerLuis Machado <lgustavo@codesourcery.com>2016-10-24 17:44:56 -0500
commit010ece9c47f1ac08c9436b133a74472713dd709b (patch)
tree1435312cf422a4fccc9b9ed62380dc55086f41be /gdb/exec.c
parentb8d38ee4250fb6ad001c2f03fa2cef8c2bc06b57 (diff)
downloadgdb-010ece9c47f1ac08c9436b133a74472713dd709b.zip
gdb-010ece9c47f1ac08c9436b133a74472713dd709b.tar.gz
gdb-010ece9c47f1ac08c9436b133a74472713dd709b.tar.bz2
Fix obvious gotcha in string comparison
This patch fixes a gotcha when comparing exception's messages in exception_print_same. It should've used the statically-allocated string versions msg1 and msg2 instead. As is, it could lead to crashes. gdb/ChangeLog: 2016-10-24 Luis Machado <lgustavo@codesourcery.com> * exec.c (exception_print_same): Fix string comparison to use statically-allocated ones.
Diffstat (limited to 'gdb/exec.c')
-rw-r--r--gdb/exec.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/gdb/exec.c b/gdb/exec.c
index d858e99..67ecc63 100644
--- a/gdb/exec.c
+++ b/gdb/exec.c
@@ -152,7 +152,7 @@ exception_print_same (struct gdb_exception e1, struct gdb_exception e2)
return (e1.reason == e2.reason
&& e1.error == e2.error
- && strcmp (e1.message, e2.message) == 0);
+ && strcmp (msg1, msg2) == 0);
}
/* See gdbcore.h. */