Messages in this thread | | | Date | Thu, 04 Jan 2007 15:04:17 -0600 | From | Eric Sandeen <> | Subject | Re: [UPDATED PATCH] fix memory corruption from misinterpreted bad_inode_ops return values |
| |
Andrew Morton wrote: > On Thu, 4 Jan 2007 20:24:12 +0000 > Al Viro <viro@ftp.linux.org.uk> wrote: > >> So my main issue with fs/bad_inode.c is not even cast per se; it's that >> cast is to void *. > > But Eric's latest patch is OK in that regard, isn't it? It might confuse > parsers (in fixable ways), but it is type-correct and has no casts. (Well, > it kinda has an link-time cast).
Even if it is, I'm starting to wonder if all this tricksiness is really worth it for 400 bytes or so. :)
-Eric - 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/
| |