Messages in this thread | | | Date | Mon, 30 Jun 2014 09:23:54 -0700 | From | Greg Kroah-Hartman <> | Subject | Re: lzo1x_decompress_safe: Macro compares unsigned to 0 (NO_EFFECT) |
| |
On Mon, Jun 30, 2014 at 09:34:44AM -0600, Don A. Bailey wrote: > In the case of macro instances that include a '0', the overflow test is > purposefully benign. We don't need a test for these cases. It was easy to throw > a '0' in there to accommodate for the new two-param call. But, to appease > Coverity, a second single-param macro that doesn't check for overflow could be > created.
Not needed, we don't need to "apease" coverity :)
Let's just leave this as-is.
thanks,
greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |