diff options
| author | Andrew Burgess <andrew.burgess@embecosm.com> | 2020-02-04 09:07:16 +0000 |
|---|---|---|
| committer | Andrew Burgess <andrew.burgess@embecosm.com> | 2020-02-04 16:46:10 +0000 |
| commit | dcc9fbc6da9536f65c55dbb67f50ab3c2e086c6e (patch) | |
| tree | 453805a56438e9e0f827e24e069dfdbf66786327 /gdb/remote-sim.c | |
| parent | f8dcc90b6030b641fa7b0b33e4203498bbb0f0e0 (diff) | |
| download | binutils-gdb-dcc9fbc6da9536f65c55dbb67f50ab3c2e086c6e.tar.gz | |
gdb/sim: Assert that the simulator ptid is not null_ptid
We assign the simulator inferior a fake ptid. If this ptid is ever
set to null_ptid then we are going to run into problems - the
simulator ptid is what we return from gdbsim_target::wait, and this in
turn is used to look up the inferior data with a call to
find_inferior_pid, which asserts the pid is not 0 (which it is in
null_pid).
This commit adds an assert that the simulator's fake pid is not
null_ptid. There should be no user visible changes after this commit.
gdb/ChangeLog:
* remote-sim.c (sim_inferior_data::sim_inferior_data): Assert that
we don't set the fake simulator ptid to the null_ptid.
Change-Id: I6e08effe70e70855aea13c9caf4fd6913d5af56d
Diffstat (limited to 'gdb/remote-sim.c')
| -rw-r--r-- | gdb/remote-sim.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/gdb/remote-sim.c b/gdb/remote-sim.c index 281232cc4e5..b7ce4672a1a 100644 --- a/gdb/remote-sim.c +++ b/gdb/remote-sim.c @@ -86,6 +86,7 @@ struct sim_inferior_data { : gdbsim_desc (desc), remote_sim_ptid (next_pid, 0, next_pid) { + gdb_assert (remote_sim_ptid != null_ptid); ++next_pid; } |