diff options
| author | Xavier Del Campo Romero <xavi.dcr@tutanota.com> | 2022-06-27 21:43:29 +0200 |
|---|---|---|
| committer | Xavier Del Campo Romero <xavi.dcr@tutanota.com> | 2022-06-27 21:43:32 +0200 |
| commit | a746ecd0b6422ef5454ceba801fae1f73bdf0203 (patch) | |
| tree | 408dd55537f971fb986207e879010763b4bbd8a4 /src | |
| parent | e869bfd0f66a6107c764d9bd27b14e129e4b404f (diff) | |
| download | rts-a746ecd0b6422ef5454ceba801fae1f73bdf0203.tar.gz | |
human_player_gui.c: fix segfault when selecting one unit
Checking against `sel` is just plain wrong, since it always expected to
be non-NULL, since `h->sel` is an array defined in `struct
human_player`. Instead, `sel->d.i` is the pointer that should be
checked.
Diffstat (limited to 'src')
| -rw-r--r-- | src/player/src/human_player_gui.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/player/src/human_player_gui.c b/src/player/src/human_player_gui.c index b5caeea..ea39bca 100644 --- a/src/player/src/human_player_gui.c +++ b/src/player/src/human_player_gui.c @@ -206,7 +206,7 @@ static int render_sel_single(const struct human_player *const h, { const struct sel_instance *const sel = &h->sel[i]; - if (sel) + if (sel->d.i) { switch (sel->type) { |