encoding: fix gen_freq_params(): do not check pdch twice
This is a left-over from an earlier version of [1] that makes Coverity think that there can be NULL pointer dereference, even despite we assert(pdch != NULL). [1] I8adc0cdb1b05a87b4df5d4bc196f6d381283a06f Change-Id: I3490c38e0c1186dfd2fae63526a05c694547cebb Fixes: CID#214230
This commit is contained in:
parent
542e388e4f
commit
2597cf1494
|
@ -565,7 +565,7 @@ static void gen_freq_params(Frequency_Parameters_t *freq_params,
|
||||||
freq_params->TSC = pdch->tsc;
|
freq_params->TSC = pdch->tsc;
|
||||||
|
|
||||||
/* If frequency hopping is not in use, encode a single ARFCN */
|
/* If frequency hopping is not in use, encode a single ARFCN */
|
||||||
if (pdch == NULL || !pdch->fh.enabled) {
|
if (!pdch->fh.enabled) {
|
||||||
freq_params->UnionType = 0x00;
|
freq_params->UnionType = 0x00;
|
||||||
freq_params->u.ARFCN = tbf->trx->arfcn;
|
freq_params->u.ARFCN = tbf->trx->arfcn;
|
||||||
return;
|
return;
|
||||||
|
|
Loading…
Reference in New Issue