Can anyone comment on it ?

Is  "FEC_AUTO" should be moved up as shown below ?

 typedef enum fe_code_rate {
        ........
        FEC_6_7,
                      ,    // Should "FEC_AUTO,"  be placed here ?
        FEC_7_8,
        FEC_8_9,
                      ,   // At present FEC_AUTO is here
         .....
          ....
 } fe_code_rate_t;


     OR

 Should   rate[fec],  g1[fec],and  g2[fec]  be initialized for
"FEC_6_7 <  fec  <  FEC_AUTO"  ??
  If yes, what should be the initial values ?


Regards
Santosh





On Sun, Mar 4, 2012 at 7:11 PM, santosh prasad nayak
<santoshprasadna...@gmail.com> wrote:
> Hi,
>
> I am getting following error:
>
> "drivers/media/dvb/frontends/cx24110.c:210 cx24110_set_fec() error:
> buffer overflow 'rate' 7 <= 8"
>
>  In cx24110_set_fec, arrays " rate[] , g1[], g2[]"  have only 7 values.
>
>
> typedef enum fe_code_rate {
>        ........
>        FEC_6_7,   // index 7
>        FEC_7_8,   // index 8
>        FEC_8_9,
>        FEC_AUTO,
>         .....
>          ....
> } fe_code_rate_t;
>
>
> For     "FEC_6_7 <  fec  <  FEC_AUTO"  , rate[fec]. g1[fec], g2[fec]
> values are not defined initially.
> Is it expected or bug ?
>
>
> regards
> Santosh
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to