summaryrefslogtreecommitdiff
path: root/apps/plugins/reversi
diff options
context:
space:
mode:
authorSolomon Peachy <pizza@shaftnet.org>2020-10-27 11:14:23 -0400
committerSolomon Peachy <pizza@shaftnet.org>2020-10-28 08:42:49 -0400
commita605cdf7008f856946cbf01193f4dffc3ee63fdb (patch)
tree53368641340ffd9d10f77f56b2bde66916a1cfd1 /apps/plugins/reversi
parent621e363e70e69a92169494515c5637551ceba219 (diff)
downloadrockbox-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.c7
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) {