lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 20 Jul 2010 12:31:33 +0200 From: Lubomir Rintel <lkundrak@...sk> To: Christoph Hellwig <hch@....de> Cc: Linux Kernel Mailing List <linux-kernel@...r.kernel.org>, Andrew Morton <akpm@...ux-foundation.org> Subject: Re: [PATCH 2/3] [fs/sysv] V7: Adjust sanity checks for some volumes On Mon, 2010-07-19 at 16:54 +0200, Christoph Hellwig wrote: > > v7i = (struct sysv_inode *)(bh2->b_data + 64); > > if ((fs16_to_cpu(sbi, v7i->i_mode) & ~0777) != S_IFDIR || > > (fs32_to_cpu(sbi, v7i->i_size) == 0) || > > - (fs32_to_cpu(sbi, v7i->i_size) & 017) != 0) > > + (fs32_to_cpu(sbi, v7i->i_size) & 017) > > + (fs32_to_cpu(sbi, v7i->i_size) > V7_NFILES * > > + sizeof (struct sysv_dir_entry))) { > > Maybe I'm missing something, but without and additional || on the > first line you added this doesn't look like it will compile. You're right. In fact this hunk is fixed in a commit that followed it, I probably incorrectly merged the fixups in git (the latter commit also had an useless comment change and such). I'll follow up with a fixed commit tomorrow, since I'm unable to test it now. Thanks, Lubo -- Flash is the Web2.0 version of blink and animated gifs. -- Stephen Smoogen -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists