aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJohannes Berg <johannes.berg@intel.com>2013-05-13 15:44:42 -0400
committerJohannes Berg <johannes.berg@intel.com>2013-05-16 17:13:45 -0400
commit82598b4f43312c01b976e02add9a5205e371f858 (patch)
tree8f53438636d20c71269932d115e493ecc6ef301f
parentbf0fd5da8afdea8e1b1a2b42b8e9db068fbac96b (diff)
iwlwifi: mvm: fix NVM parsing error path
If NVM parsing fails and returns NULL, we continue in the code flow and eventually crash accessing the NULL pointer. Return an error from iwl_nvm_init() if the parsing failed. Reviewed-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com> Signed-off-by: Johannes Berg <johannes.berg@intel.com>
-rw-r--r--drivers/net/wireless/iwlwifi/mvm/nvm.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/net/wireless/iwlwifi/mvm/nvm.c b/drivers/net/wireless/iwlwifi/mvm/nvm.c
index b8ec02f89acc..c6c15f21065f 100644
--- a/drivers/net/wireless/iwlwifi/mvm/nvm.c
+++ b/drivers/net/wireless/iwlwifi/mvm/nvm.c
@@ -231,8 +231,9 @@ int iwl_nvm_init(struct iwl_mvm *mvm)
231 if (ret < 0) 231 if (ret < 0)
232 return ret; 232 return ret;
233 233
234 ret = 0;
235 mvm->nvm_data = iwl_parse_nvm_sections(mvm); 234 mvm->nvm_data = iwl_parse_nvm_sections(mvm);
235 if (!mvm->nvm_data)
236 return -ENODATA;
236 237
237 return ret; 238 return 0;
238} 239}