Re: [PATCH] iwlwifi:Fix incorrect fallthrough in switch statement in the function iwl_mvm_check_running_scans

Re: [PATCH] iwlwifi:Fix incorrect fallthrough in switch statement in the function iwl_mvm_check_running_scans

[Coelho, Luciano]

T24gVHVlLCAyMDE1LTA5LTIyIGF0IDIwOjI0IC0wNDAwLCBOaWNob2xhcyBLcmF1c2Ugd3JvdGU6
DQo+IFRoaXMgZml4ZXMgaW5jb3JyZWN0IGZhbGx0aHJvdWdoIGluIHRoZSBzd2l0Y2ggc3RhdG1l
bnQgY2hlY2tpbmcNCj4gdGhlIHNjYW4gdHlwZSBwYXNzZWQgYnkgdGhlIGNhbGxlciB0byBpd2xf
bXZtX2NoZWNrX3J1bm5pbmdfc2NhbnMNCj4gZm9yIHRoZSBzd2l0Y2ggY2FzZSBJV0xfTVZNX1ND
QU5fU0NIRUQgdG8gcmV0dXJuIGRpcmVjdGx5IGFmdGVyDQo+IHRoZSBjYWxsIHRvIGl3bF9tdm1f
c2Nhbl9zdG9wIGluIG9yZGVyIHRvIGF2b2lkIGZhbGx0aHJvdWdoIGludG8NCj4gdGhlIG5leHQg
Y2FzZSBhbmQgaW5jb3JyZWN0bHkgcmV0dXJuIHplcm8gaW5kaWNhdGluZyBzdWNjZXNzIHRvDQo+
IHRoZSBjYWxsZXIgZXZlbiBpZiB0aGUgZnVuY3Rpb24gY2FsbCB0byBpd2xfbXZtX3NjYW5fc3Rv
cCBmYWlscy4NCj4gDQo+IFNpZ25lZC1vZmYtYnk6IE5pY2hvbGFzIEtyYXVzZSA8eGVyb2ZvaWZ5
QGdtYWlsLmNvbT4NCj4gLS0tDQoNCkFwcGxpZWQgdG8gb3VyIGludGVybmFsIHRyZWUgd2l0aCBh
IG1vZGlmaWVkIGNvbW1pdCBtZXNzYWdlLg0KDQpUaGFua3MhDQoNCi0tDQpMdWNhLg==

Re: [PATCH] iwlwifi:Fix incorrect fallthrough in switch statement in the function iwl_mvm_check_running_scans

[Coelho, Luciano]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 763 bytes --]

On Tue, 2015-09-22 at 20:24 -0400, Nicholas Krause wrote:
> This fixes incorrect fallthrough in the switch statment checking
> the scan type passed by the caller to iwl_mvm_check_running_scans
> for the switch case IWL_MVM_SCAN_SCHED to return directly after
> the call to iwl_mvm_scan_stop in order to avoid fallthrough into
> the next case and incorrectly return zero indicating success to
> the caller even if the function call to iwl_mvm_scan_stop fails.
> 
> Signed-off-by: Nicholas Krause <xerofoify@gmail.com>
> ---

Applied to our internal tree with a modified commit message.

Thanks!

--
Luca.ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥

Re: [PATCH] iwlwifi:Fix incorrect fallthrough in switch statement in the function iwl_mvm_check_running_scans

[Coelho, Luciano]

On Tue, 2015-09-22 at 20:24 -0400, Nicholas Krause wrote:
> This fixes incorrect fallthrough in the switch statment checking
> the scan type passed by the caller to iwl_mvm_check_running_scans
> for the switch case IWL_MVM_SCAN_SCHED to return directly after
> the call to iwl_mvm_scan_stop in order to avoid fallthrough into
> the next case and incorrectly return zero indicating success to
> the caller even if the function call to iwl_mvm_scan_stop fails.
> 
> Signed-off-by: Nicholas Krause <xerofoify@gmail.com>
> ---

Applied to our internal tree with a modified commit message.

Thanks!

--
Luca.