hamradio/Hamlib
1
0
Fork 0
mirror of https://github.com/Hamlib/Hamlib.git synced 2024-09-22 10:57:21 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add error check for channel 90-99 split in ic10.c

Browse Source
This commit is contained in:
Michael Black 2020-04-07 17:11:35 -05:00
parent 624db802e4
commit 991e4a4f24
1 changed files with 21 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

35
rigs/kenwood/ic10.c
View File

@ -729,6 +729,10 @@ int ic10_set_channel(RIG *rig, const channel_t *chan)
freq = (int64_t) chan->freq;
if (chan->channel_num < 90 && chan->tx_freq != 0) {
rig_debug(RIG_DEBUG_ERR,"%s: Transmit/split can only be on channels 90-99\n", __func__);
return -RIG_EINVAL;
}
switch (chan->mode)
{
case RIG_MODE_CW : md = MD_CW; break;
@ -789,21 +793,24 @@ int ic10_set_channel(RIG *rig, const channel_t *chan)
return -RIG_EINVAL;
}
/* MWnxrrggmmmkkkhhhdzxxxx; */
len = sprintf(membuf, "MW1 %02d%011"PRIll"%c0 ;",
chan->channel_num,
freq,
md
);
retval = ic10_transaction(rig, membuf, len, NULL, 0);
// I assume we need to check the retval here -- W9MDB
// This was found from cppcheck
if (retval != RIG_OK)
if (chan->channel_num >= 90)
{
rig_debug(RIG_DEBUG_ERR, "%s: transaction failed: %s\n", __func__,
strerror(retval));
return retval;
/* MWnxrrggmmmkkkhhhdzxxxx; */
len = sprintf(membuf, "MW1 %02d%011"PRIll"%c0 ;",
chan->channel_num,
freq,
md
);
retval = ic10_transaction(rig, membuf, len, NULL, 0);
// I assume we need to check the retval here -- W9MDB
// This was found from cppcheck
if (retval != RIG_OK)
{
rig_debug(RIG_DEBUG_ERR, "%s: transaction failed: %s\n", __func__,
strerror(retval));
return retval;
}
}
return RIG_OK;