On 08/21, Andy Lutomirski wrote:
>
> On Wed, Aug 21, 2013 at 11:20 AM, Oleg Nesterov wrote:
> > Can't really comment the patch, just a nit:
> >
> > On 08/21, Andy Lutomirski wrote:
> >>
> >> +static bool may_flink(const struct path *path)
> >> +{
> >> + bool ret;
> >> + struct inode
On Wed, Aug 21, 2013 at 11:20 AM, Oleg Nesterov wrote:
> Can't really comment the patch, just a nit:
>
> On 08/21, Andy Lutomirski wrote:
>>
>> +static bool may_flink(const struct path *path)
>> +{
>> + bool ret;
>> + struct inode *inode = path->dentry->d_inode;
>> +
>> + /*
>> +
Can't really comment the patch, just a nit:
On 08/21, Andy Lutomirski wrote:
>
> +static bool may_flink(const struct path *path)
> +{
> + bool ret;
> + struct inode *inode = path->dentry->d_inode;
> +
> + /*
> + * This is racy: I_LINKABLE could be cleared between this check
> +
There have long been two ways to ask the kernel to create a new
hardlink to the inode represented by an fd: linkat(..., AT_EMPTY_PATH)
and AT_SYMLINK_FOLLOW on /proc/self/fd/N. The latter has no
particular security restrictions, but the former required privilege
until:
commit
There have long been two ways to ask the kernel to create a new
hardlink to the inode represented by an fd: linkat(..., AT_EMPTY_PATH)
and AT_SYMLINK_FOLLOW on /proc/self/fd/N. The latter has no
particular security restrictions, but the former required privilege
until:
commit
Can't really comment the patch, just a nit:
On 08/21, Andy Lutomirski wrote:
+static bool may_flink(const struct path *path)
+{
+ bool ret;
+ struct inode *inode = path-dentry-d_inode;
+
+ /*
+ * This is racy: I_LINKABLE could be cleared between this check
+ * and
On Wed, Aug 21, 2013 at 11:20 AM, Oleg Nesterov o...@redhat.com wrote:
Can't really comment the patch, just a nit:
On 08/21, Andy Lutomirski wrote:
+static bool may_flink(const struct path *path)
+{
+ bool ret;
+ struct inode *inode = path-dentry-d_inode;
+
+ /*
+ *
On 08/21, Andy Lutomirski wrote:
On Wed, Aug 21, 2013 at 11:20 AM, Oleg Nesterov o...@redhat.com wrote:
Can't really comment the patch, just a nit:
On 08/21, Andy Lutomirski wrote:
+static bool may_flink(const struct path *path)
+{
+ bool ret;
+ struct inode *inode =
8 matches
Mail list logo