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 <santoshprasadnayak@xxxxxxxxx> 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html