Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751957Ab3CBK0S (ORCPT ); Sat, 2 Mar 2013 05:26:18 -0500 Received: from ns3.aprogsys.com ([91.121.69.92]:44437 "EHLO ns3.aprogsys.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751486Ab3CBK0Q (ORCPT ); Sat, 2 Mar 2013 05:26:16 -0500 X-Greylist: delayed 630 seconds by postgrey-1.27 at vger.kernel.org; Sat, 02 Mar 2013 05:26:16 EST Message-ID: <5131D14E.8020802@vetienne.net> Date: Sat, 02 Mar 2013 11:15:42 +0100 From: Vincent ETIENNE User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130301 Thunderbird/17.0.3 MIME-Version: 1.0 To: Richard Weinberger CC: ocfs2-devel@oss.oracle.com, ocfs2-user@oss.oracle.com, linux-kernel@vger.kernel.org, viro@zeniv.linux.org.uk, mfasheh@suse.com, jlbec@evilplan.org, linux-fsdevel@vger.kernel.org, david@sigma-star.at Subject: Re: OCFS2 mainline state? References: <512BCA9E.4090702@nod.at> In-Reply-To: <512BCA9E.4090702@nod.at> Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3258 Lines: 94 Hi I was testing this path in august and i have two patch applied for ocfs2 The one your referring and also a changed in file.c At the time of the test, the two patch were necessary ( and both suppress a BUG although not the same one) Sorry i have no idea if it's still necessary or not, but i prefer to send you a notice Vincent ETIENNE --- a/fs/ocfs2/file.c +++ b/fs/ocfs2/file.c @@ -1998,7 +1998,7 @@ static long ocfs2_fallocate(struct file *file, int mode, loff_t offset, sr.l_start = (s64)offset; sr.l_len = (s64)len; - return __ocfs2_change_file_space(NULL, inode, offset, cmd, &sr, + return __ocfs2_change_file_space(file, inode, offset, cmd, &sr, change_size); } diff --git a/fs/ocfs2/symlink.c b/fs/ocfs2/symlink.c index f1fbb4b..66edce7 100644 --- a/fs/ocfs2/symlink.c +++ b/fs/ocfs2/symlink.c @@ -57,7 +57,7 @@ static int ocfs2_fast_symlink_readpage(struct file *unused, struct page *page) { struct inode *inode = page->mapping->host; - struct buffer_head *bh; + struct buffer_head *bh = NULL; int status = ocfs2_read_inode_block(inode, &bh); struct ocfs2_dinode *fe; const char *link; Le 25/02/2013 21:33, Richard Weinberger a ?crit : > Hi! > > Today I encountered the following problem on v3.8: > [ 28.940032] general protection fault: 0000 [#1] PREEMPT SMP > ... > [ 28.984953] Call Trace: > [ 28.986628 [] > ocfs2_fast_symlink_readpage+0x70/0x1b0 [ocfs2] > [ 28.988302] [] ? add_to_page_cache_lru+0x29/0x40 > [ 28.989942] [] do_read_cache_page+0x7a/0x170 > [ 28.991573] [] read_cache_page_async+0x14/0x20 > [ 28.993212] [] read_cache_page+0x9/0x20 > [ 28.994827] [] page_getlink.isra.9+0x25/0x80 > [ 28.996442] [] page_follow_link_light+0x21/0x40 > [ 28.998049] [] generic_readlink+0x3d/0xa0 > [ 28.999653] [] sys_readlinkat+0xfb/0x130 > [ 29.001246] [] sys_readlink+0x16/0x20 > [ 29.002839] [] system_call_fastpath+0x1a/0x1f > > To my utter astonishment I found out that the issue is known since > August 2012. > And there is also a trivial fix for it.[0] > The said fix found it's way into Oracle Unbreakable Linux very quickly, > but not into mainline.[1] > > Now I'm wondering how much Oracle really cares about OCFS2 in mainline? > Maybe there are some more unfixed vulnerabilities? > > Not amused, > //richard > > [0] http://www.mail-archive.com/ocfs2-devel@oss.oracle.com/msg07774.html > [1] https://oss.oracle.com/pipermail/el-errata/2012-October/003103.html > -- > 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/ > -- 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/