ARMmbed
/
mbed-os
mirror of https://github.com/ARMmbed/mbed-os.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Fix Coverity warning 

Fixed Coverity warning "stored value is overwritten before it can be used".
pull/6569/head
Kimmo Vaisanen 2018-03-29 11:49:09 +03:00
parent 10c269bcfa
commit 01fd8804df
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
features/lorawan/lorastack/mac/LoRaMac.cpp
View File

@ -1102,7 +1102,6 @@ lorawan_status_t LoRaMac::schedule_tx(void)
{ {
lorawan_time_t dutyCycleTimeOff = 0; lorawan_time_t dutyCycleTimeOff = 0;
channel_selection_params_t nextChan; channel_selection_params_t nextChan;
lorawan_status_t status = LORAWAN_STATUS_PARAMETER_INVALID;
if (_params.sys_params.max_duty_cycle == 255) { if (_params.sys_params.max_duty_cycle == 255) {
return LORAWAN_STATUS_DEVICE_OFF; return LORAWAN_STATUS_DEVICE_OFF;
@ -1121,9 +1120,10 @@ lorawan_status_t LoRaMac::schedule_tx(void)
nextChan.joined = _is_nwk_joined; nextChan.joined = _is_nwk_joined;
nextChan.last_aggregate_tx_time = _params.timers.aggregated_last_tx_time; nextChan.last_aggregate_tx_time = _params.timers.aggregated_last_tx_time;
status = _lora_phy.set_next_channel(&nextChan, &_params.channel, lorawan_status_t status = _lora_phy.set_next_channel(&nextChan,
&dutyCycleTimeOff, &_params.channel,
&_params.timers.aggregated_timeoff); &dutyCycleTimeOff,
&_params.timers.aggregated_timeoff);
switch (status) { switch (status) {
case LORAWAN_STATUS_NO_CHANNEL_FOUND: case LORAWAN_STATUS_NO_CHANNEL_FOUND:
case LORAWAN_STATUS_NO_FREE_CHANNEL_FOUND: case LORAWAN_STATUS_NO_FREE_CHANNEL_FOUND: