Alexey Dobriyan <[EMAIL PROTECTED]> wrote: > > Oh, and for the record: current sparse from Linus doesn't warn about > this. Slightly modified sparse warns. Bugs which were uncovered by > more or less trivial and slightly broken sparse patch [1] are: > > [PATCH] dscc4: fix dscc4_init_dummy_skb check > [PATCH] opl3sa2: fix adding second dma channel > [PATCH] gusclassic: fix adding second dma channel > [PATCH] ipw2200: fix ->eeprom[EEPROM_VERSION] check > [PATCH] ixj: fix writing silence check
The first two can also be caught with gcc -pedantic. In fact, catching the last two should be doable there as well. The difference between gcc -pedantic and sparse is that it doesn't warn about obviously correct cases like p != 0 or p = 0. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmV>HI~} <[EMAIL PROTECTED]> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html