On Tue, Aug 30, 2016 at 07:39:49PM +0200, Phil Sutter wrote: > I have been told that the flex scanner won't return empty strings, so > strlen(data) should always be greater 0. To avoid a hard to debug issue > though, add an assert() to make sure this is always the case before > risking an unsigned variable underrun. > > A real issue though is the check for 'datalen - 1 >= 0', which will > never fail due to datalen being unsigned. Fix this by incrementing both > sides by one, hence checking 'datalen >= 1'.
Applied, thanks Phil. -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
