> -----Original Message----- > From: Kearney, Tadhg <tadhg.kearney at intel.com> <snip> > +static int > +parse_uncore_index(const char *choice) > +{ > + if (frequency_index > freq_array_len) { Should this be compared against freq_array_len-1 ?. Ex: If total 10 frequencies the index can be only 0-9. Thanks, Reshma