diff options
author | Solomon Peachy <pizza@shaftnet.org> | 2020-10-27 11:14:23 -0400 |
---|---|---|
committer | Solomon Peachy <pizza@shaftnet.org> | 2020-10-28 08:42:49 -0400 |
commit | a605cdf7008f856946cbf01193f4dffc3ee63fdb (patch) | |
tree | 53368641340ffd9d10f77f56b2bde66916a1cfd1 /apps/plugins/reversi | |
parent | 621e363e70e69a92169494515c5637551ceba219 (diff) | |
download | rockbox-a605cdf7008f856946cbf01193f4dffc3ee63fdb.tar.gz rockbox-a605cdf7008f856946cbf01193f4dffc3ee63fdb.zip |
Fix multiple potential null pointer dereferencess
GCC's optimizer thinks all of these _will_ fail at some point
Change-Id: I287eeb574162a5d3b3347654d25aa1f53e9f5563
Diffstat (limited to 'apps/plugins/reversi')
-rw-r--r-- | apps/plugins/reversi/reversi-gui.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/apps/plugins/reversi/reversi-gui.c b/apps/plugins/reversi/reversi-gui.c index aca54a1ea3..74dd98b676 100644 --- a/apps/plugins/reversi/reversi-gui.c +++ b/apps/plugins/reversi/reversi-gui.c | |||
@@ -637,12 +637,13 @@ enum plugin_status plugin_start(const void *parameter) { | |||
637 | draw_screen = false; | 637 | draw_screen = false; |
638 | } | 638 | } |
639 | switch(cur_player) { | 639 | switch(cur_player) { |
640 | case BLACK: | ||
641 | cur_strategy = black_strategy; | ||
642 | break; | ||
643 | case WHITE: | 640 | case WHITE: |
644 | cur_strategy = white_strategy; | 641 | cur_strategy = white_strategy; |
645 | break; | 642 | break; |
643 | case BLACK: | ||
644 | default: | ||
645 | cur_strategy = black_strategy; | ||
646 | break; | ||
646 | } | 647 | } |
647 | 648 | ||
648 | if(cur_strategy->is_robot && !game_finished) { | 649 | if(cur_strategy->is_robot && !game_finished) { |