Messages in this thread | | | Date | Sun, 27 Mar 2005 23:45:58 +0100 | From | Russell King <> | Subject | Re: Do not misuse Coverity please (Was: sound/oss/cs46xx.c: fix a check after use) |
| |
On Mon, Mar 28, 2005 at 12:34:01AM +0200, Jean Delvare wrote: > I think that you'd be better just telling the > maintainers about the problem without providing an arbitrary patch, so > that they will actually look into the problem with their advanced > knowledge of the driver
FWIW, I agree with Jean.
Applying these patches without adequate review is like applying __iomem or __user fixes where they've been generated with the aim of shutting up sparse, rather than considering whether the warning is actually valid. Once the warning is gone, the real problem is lost and never gets looked at.
-- Russell King Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/ maintainer of: 2.6 Serial core - 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/
| |