Commit 52e35e23 authored by Wilson Thong's avatar Wilson Thong

#221 fixed crash when processing false DCI1A due to incorrect HARQ-PID checking

parent 2f094e56
...@@ -4799,7 +4799,7 @@ int check_dci_format1_1a_coherency(DCI_format_t dci_format, ...@@ -4799,7 +4799,7 @@ int check_dci_format1_1a_coherency(DCI_format_t dci_format,
return(0); return(0);
} }
if(harq_pid >8) if(harq_pid>=8)
{ {
LOG_I(PHY,"bad harq id \n"); LOG_I(PHY,"bad harq id \n");
return(0); return(0);
...@@ -5013,7 +5013,7 @@ int check_dci_format2_2a_coherency(DCI_format_t dci_format, ...@@ -5013,7 +5013,7 @@ int check_dci_format2_2a_coherency(DCI_format_t dci_format,
#endif #endif
// I- check dci content minimum coherency // I- check dci content minimum coherency
if(harq_pid >8) if(harq_pid>=8)
{ {
LOG_I(PHY,"bad harq pid\n"); LOG_I(PHY,"bad harq pid\n");
return(0); return(0);
......
...@@ -242,7 +242,7 @@ uint32_t ulsch_encoding(uint8_t *a, ...@@ -242,7 +242,7 @@ uint32_t ulsch_encoding(uint8_t *a,
return(-1); return(-1);
} }
if (harq_pid > 7) { if (harq_pid >= 8) {
LOG_E(PHY,"Illegal harq_pid %d\n",harq_pid); LOG_E(PHY,"Illegal harq_pid %d\n",harq_pid);
return(-1); return(-1);
} }
......
...@@ -402,7 +402,7 @@ void ulsch_modulation(int32_t **txdataF, ...@@ -402,7 +402,7 @@ void ulsch_modulation(int32_t **txdataF,
// x1 is set in lte_gold_generic // x1 is set in lte_gold_generic
x2 = (ulsch->rnti<<14) + (subframe<<9) + frame_parms->Nid_cell; //this is c_init in 36.211 Sec 6.3.1 x2 = (ulsch->rnti<<14) + (subframe<<9) + frame_parms->Nid_cell; //this is c_init in 36.211 Sec 6.3.1
if (harq_pid > 7) { if (harq_pid>=8) {
printf("ulsch_modulation.c: Illegal harq_pid %d\n",harq_pid); printf("ulsch_modulation.c: Illegal harq_pid %d\n",harq_pid);
return; return;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment