Re: [media] dvb: Buffer Overfolow in cx24110_set_fec

2012-03-05 Thread santosh prasad nayak
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


[media] dvb: Buffer Overfolow in cx24110_set_fec

2012-03-04 Thread santosh prasad nayak
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   fecFEC_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