On Sun, Oct 16, 2016 at 06:25:19PM +0200, Gerhard Sittig wrote: > Commit 2c9c0df86eaf removed the sentinel from the samplerates[] array, > but did not adjust the test which checked whether a rate is listed in > the set of supported rates. This could result in an out-of-range access > beyond the array's last item. > > Fix the "listed?" check after iterating the table of supported rates. > Cope with either presence or absence of a sentinel in the array. > > Address some more style nits while we are here. Rename an identifier > for a local variable which unintentionally might suggest that it would > be a preprocessor macro (all-caps). Reduce redundancy in data type > references as well as in the determination of the array size. > > Signed-off-by: Gerhard Sittig <gerhard.sit...@gmx.net>
Merged, thanks a lot! Uwe. -- http://hermann-uwe.de | http://randomprojects.org | http://sigrok.org ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, SlashDot.org! http://sdm.link/slashdot _______________________________________________ sigrok-devel mailing list sigrok-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/sigrok-devel
