diff options
author | Nebuleon Fumika | 2013-01-18 01:18:11 -0500 |
---|---|---|
committer | Nebuleon Fumika | 2013-01-18 01:18:11 -0500 |
commit | 51e18ffb2fa40a3dbb28ddb9789bbf0e2665132a (patch) | |
tree | ab3cd5b993d9db7181bf0e842ddef4db74dbcc92 /source/nds/gui.c | |
parent | a8ad21f22c5ccf4886945a4a5f3c08e4feb21d61 (diff) | |
download | snesemu-51e18ffb2fa40a3dbb28ddb9789bbf0e2665132a.tar.gz snesemu-51e18ffb2fa40a3dbb28ddb9789bbf0e2665132a.tar.bz2 snesemu-51e18ffb2fa40a3dbb28ddb9789bbf0e2665132a.zip |
cppcheck: [nds/gui.c:3573]: (error) Possible null pointer dereference: current_option - otherwise it is redundant to check if current_option is null at line 3576
Diffstat (limited to 'source/nds/gui.c')
-rw-r--r-- | source/nds/gui.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/source/nds/gui.c b/source/nds/gui.c index baa6f6b..6efb59d 100644 --- a/source/nds/gui.c +++ b/source/nds/gui.c @@ -3570,10 +3570,10 @@ u32 menu(u16 *screen) current_option = current_menu->options + current_option_num; - if(current_option -> option_type & HIDEN_TYPE) + if(!current_option) break; - if(!current_option) + if(current_option -> option_type & HIDEN_TYPE) break; if(current_menu->key_function) |