On Sun, Dec 03, 2006 at 03:26:20PM +0000, Gerrit Renker wrote: > Quoting Ian McDonald: > | The > | reason I say this is good as in the back of my mind I am thinking of > | people looking for DoS vulnerabilities... > | > | However BUG_ON does work and forces the code to get tidied so the > | condition doesn't occur again. So whichever way I'm happy to go ahead > | with it. > What you are saying is very important and I suggest auditing the code against > calling this function with invalid parameters as next to-do. > > I would further like to inform you and Arnaldo that I have ironed out an error > of mine in this patch (uploaded), namely the following Heh, I was _just_ hitting enter to commit exactly this patch, so lemme get the_whole_lot again to get this one, thanks a lot! - Arnaldo - To unsubscribe from this list: send the line "unsubscribe dccp" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html