lkml.org 
[lkml]   [2010]   [Jun]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectFwd: Fwd: fs/exec.c core dumping on NFS mounted directory
From
From: Lei Sun <leisun124@gmail.com>
Date: Fri, Jun 25, 2010 at 1:39 PM
Subject: Re: Fwd: fs/exec.c core dumping on NFS mounted directory
To: Trond Myklebust <trond.myklebust@fys.uio.no>


On Fri, Jun 25, 2010 at 1:01 PM, Trond Myklebust
<trond.myklebust@fys.uio.no> wrote:
> On Fri, 2010-06-25 at 12:32 -0700, Lei Sun wrote:
>>   In 2.6.3x, I am logging in as a root and running app from NFS
>> mounted directory. The app crashed with 0 bytes core file (I have set
>> "ulimit -c unlimited", so it's not an issue).
>>
>> Proposed patch:
>>
>> --- exec.c 2010-05-19 17:33:16.000000000 -0700
>> +++ exec.c.new 2010-05-19 17:33:07.000000000 -0700
>> @@ -1859,7 +1859,7 @@
>>      * Dont allow local users get cute and trick others to coredump
>>      * into their pre-created files:
>>      */
>> - if (inode->i_uid != current_fsuid())
>> + if (inode->i_bytes != 0 && inode->i_uid != current_fsuid())
>>         goto close_fail;
>>
>>     if (!file->f_op)
>>
> ... which would go ahead and reintroduce bugzilla entry 3043, but only
> for those filesystems that set inode->i_bytes to zero:
>           https://bugzilla.kernel.org/show_bug.cgi?id=3043
>
> NACKed...
>
> Trond
>
>

If existing core file size is 0, it will be replaced, otherwise it
won't , is this a problem?  The 0 bytes core file is not useful
anyway.
--
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/

\
 
 \ /
  Last update: 2010-06-25 22:49    [from the cache]
©2003-2011 Jasper Spaans