linux-kernel - Re: [PATCH 1/1] reiserfs:Adding tmpfile support

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAFLxGvx1m_HL7wPA1=LXpjJvwN7oMCKjxJk4AHO+cgQW0FPf+Q@mail.gmail.com>
Date:	Wed, 25 Dec 2013 14:39:48 +0100
From:	Richard Weinberger <richard.weinberger@...il.com>
To:	Fabian Frederick <fabf@...net.be>
Cc:	LKML <linux-kernel@...r.kernel.org>,
	linux-fsdevel <linux-fsdevel@...r.kernel.org>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Al Viro <viro@...iv.linux.org.uk>,
	Jeff Mahoney <jeffm@...e.com>, Jan Kara <jack@...e.cz>
Subject: Re: [PATCH 1/1] reiserfs:Adding tmpfile support

On Wed, Dec 25, 2013 at 2:23 PM, Fabian Frederick <fabf@...net.be> wrote:
> This patch is largely based on reiserfs_create function.
> -Removing dquot_initialize (cf ext2/tmpfile)
> -Removing add_entry code.
> -Adding d_tmpfile generic call.
>
> It was not extensively tested; other solutions would
> be to work with hidden entries or merge
> creation - tmpfile process.

Is it worth adding new features to a legacy file system like reiserfs?

> Signed-off-by: Fabian Frederick <fabf@...net.be>
> ---
>  fs/reiserfs/namei.c | 60 +++++++++++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 60 insertions(+)
>
> diff --git a/fs/reiserfs/namei.c b/fs/reiserfs/namei.c
> index dc5236f..125060b 100644
> --- a/fs/reiserfs/namei.c
> +++ b/fs/reiserfs/namei.c
> @@ -637,6 +637,65 @@ static int reiserfs_create(struct inode *dir, struct dentry *dentry, umode_t mod
>         return retval;
>  }
>
> +
> +/* tmpfile function based on reiserfs_create without filename entry
> +*/
> +
> +static int reiserfs_tmpfile(struct inode *dir, struct dentry *dentry,
> +       umode_t mode)
> +{
> +       int retval;
> +       struct inode *inode;
> +       int jbegin_count =
> +           JOURNAL_PER_BALANCE_CNT * 2 +
> +           2 * (REISERFS_QUOTA_INIT_BLOCKS(dir->i_sb) +
> +                REISERFS_QUOTA_TRANS_BLOCKS(dir->i_sb));
> +       struct reiserfs_transaction_handle th;
> +       struct reiserfs_security_handle security;
> +
> +       inode = new_inode(dir->i_sb);
> +       if (!inode)
> +               return -ENOMEM;
> +
> +       new_inode_init(inode, dir, mode);
> +
> +       jbegin_count += reiserfs_cache_default_acl(dir);
> +       retval = reiserfs_security_init(dir, inode, &dentry->d_name, &security);
> +       if (retval < 0) {
> +               drop_new_inode(inode);
> +               return retval;
> +       }
> +       jbegin_count += retval;
> +       reiserfs_write_lock(dir->i_sb);
> +
> +       retval = journal_begin(&th, dir->i_sb, jbegin_count);
> +       if (retval) {
> +               drop_new_inode(inode);
> +               goto out_failed;
> +       }
> +
> +       retval =
> +           reiserfs_new_inode(&th, dir, mode, NULL, 0 /*i_size */ , dentry,
> +                              inode, &security);
> +       if (retval)
> +               goto out_failed;
> +
> +       inode->i_op = &reiserfs_file_inode_operations;
> +       inode->i_fop = &reiserfs_file_operations;
> +       inode->i_mapping->a_ops = &reiserfs_address_space_operations;
> +
> +       reiserfs_update_inode_transaction(inode);
> +       reiserfs_update_inode_transaction(dir);
> +
> +       unlock_new_inode(inode);
> +       d_tmpfile(dentry, inode);
> +       retval = journal_end(&th, dir->i_sb, jbegin_count);
> +
> +      out_failed:
> +       reiserfs_write_unlock(dir->i_sb);
> +       return retval;
> +}
> +
>  static int reiserfs_mknod(struct inode *dir, struct dentry *dentry, umode_t mode,
>                           dev_t rdev)
>  {
> @@ -1522,6 +1581,7 @@ const struct inode_operations reiserfs_dir_inode_operations = {
>         .removexattr = reiserfs_removexattr,
>         .permission = reiserfs_permission,
>         .get_acl = reiserfs_get_acl,
> +       .tmpfile = reiserfs_tmpfile,
>  };
>
>  /*
> --
> 1.8.1.4
> --
> 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/



-- 
Thanks,
//richard
--
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/