Cc to AKPM for include. On 05/23/13 06:50, Sunil Mushran wrote: > Acked-by: Sunil Mushran <[email protected] > <mailto:[email protected]>> > > > On Tue, May 21, 2013 at 7:44 PM, shencanquan <[email protected] > <mailto:[email protected]>> wrote: > > On 2013/5/22 10:38, xiaowei.hu <http://xiaowei.hu> wrote: > > if there is error happen in , for example EIO in > > __ocfs2_prepare_orphan_dir, ocfs2_prep_new_orphaned_file will release > > the inode_ac, the the caller of ocfs2_prep_new_orphaned_file get a 0 > > return, it will refer to a NULL ocfs2_alloc_context struct in the > > following functions.Kernel panic happens. > Hmm, I see, this patch looks good . > Reviewed-by: shencanquan <[email protected] > <mailto:[email protected]>> > > > > On 05/22/2013 10:31 AM, shencanquan wrote: > >> if return 0, what's wrong with this? > >> On 2013/5/22 9:43, [email protected] > <mailto:[email protected]> wrote: > >>> From: "Xiaowei.Hu"<[email protected] > <mailto:[email protected]>> > >>> > >>> --- > >>> fs/ocfs2/namei.c | 2 +- > >>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>> > >>> diff --git a/fs/ocfs2/namei.c b/fs/ocfs2/namei.c > >>> index 50c93a8..422cbe0 100644 > >>> --- a/fs/ocfs2/namei.c > >>> +++ b/fs/ocfs2/namei.c > >>> @@ -2216,7 +2216,7 @@ out: > >>> > >>> brelse(orphan_dir_bh); > >>> > >>> - return 0; > >>> + return ret; > >>> } > >>> > >>> int ocfs2_create_inode_in_orphan(struct inode *dir, > >> > >> > > > > > > > > > > _______________________________________________ > Ocfs2-devel mailing list > [email protected] <mailto:[email protected]> > https://oss.oracle.com/mailman/listinfo/ocfs2-devel > >
-- Oracle <http://www.oracle.com> Joe Jin | Software Development Senior Manager | +8610.6106.5624 ORACLE | Linux and Virtualization No. 24 Zhongguancun Software Park, Haidian District | 100193 Beijing _______________________________________________ Ocfs2-devel mailing list [email protected] https://oss.oracle.com/mailman/listinfo/ocfs2-devel
