diff options
| author | Tom Tromey <tromey@adacore.com> | 2023-05-01 10:10:29 -0600 |
|---|---|---|
| committer | Tom Tromey <tromey@adacore.com> | 2023-05-01 10:10:29 -0600 |
| commit | 87c84f07a0a1b7e12b6a0c9b21cc0e021e460786 (patch) | |
| tree | e6ad017a72b194b6f7ca6c03bc3a4db696f66dc7 /gdb/rust-parse.c | |
| parent | c819a3380fc1b110b88bc6ab5ef9323dbe7d4753 (diff) | |
| download | binutils-87c84f07a0a1b7e12b6a0c9b21cc0e021e460786.zip binutils-87c84f07a0a1b7e12b6a0c9b21cc0e021e460786.tar.gz binutils-87c84f07a0a1b7e12b6a0c9b21cc0e021e460786.tar.bz2 | |
Fix crash in Rust expression parser
A user found that an array expression with just a single value (like
"[23]") caused the Rust expression parser to crash.
Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=30410
Diffstat (limited to 'gdb/rust-parse.c')
| -rw-r--r-- | gdb/rust-parse.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/gdb/rust-parse.c b/gdb/rust-parse.c index 6c7922d..648e48d 100644 --- a/gdb/rust-parse.c +++ b/gdb/rust-parse.c @@ -1190,7 +1190,7 @@ rust_parser::parse_array () result = make_operation<rust_array_operation> (std::move (expr), std::move (rhs)); } - else if (current_token == ',') + else if (current_token == ',' || current_token == ']') { std::vector<operation_up> ops; ops.push_back (std::move (expr)); @@ -1205,7 +1205,7 @@ rust_parser::parse_array () int len = ops.size () - 1; result = make_operation<array_operation> (0, len, std::move (ops)); } - else if (current_token != ']') + else error (_("',', ';', or ']' expected")); require (']'); |