ubifs: Check for name being NULL while mounting
[sfrench/cifs-2.6.git] / fs / ubifs / super.c
index c5466c70d620015aaa16ede0790aefb3c6efbc14..87d08f738632820e65921afa3addbe747b1a61e3 100644 (file)
@@ -89,9 +89,9 @@ static int validate_inode(struct ubifs_info *c, const struct inode *inode)
        if (ui->xattr && !S_ISREG(inode->i_mode))
                return 5;
 
-       if (!ubifs_compr_present(ui->compr_type)) {
+       if (!ubifs_compr_present(c, ui->compr_type)) {
                ubifs_warn(c, "inode %lu uses '%s' compression, but it was not compiled in",
-                          inode->i_ino, ubifs_compr_name(ui->compr_type));
+                          inode->i_ino, ubifs_compr_name(c, ui->compr_type));
        }
 
        err = dbg_check_dir(c, inode);
@@ -296,7 +296,7 @@ static int ubifs_write_inode(struct inode *inode, struct writeback_control *wbc)
        struct ubifs_info *c = inode->i_sb->s_fs_info;
        struct ubifs_inode *ui = ubifs_inode(inode);
 
-       ubifs_assert(!ui->xattr);
+       ubifs_assert(c, !ui->xattr);
        if (is_bad_inode(inode))
                return 0;
 
@@ -349,7 +349,7 @@ static void ubifs_evict_inode(struct inode *inode)
                goto out;
 
        dbg_gen("inode %lu, mode %#x", inode->i_ino, (int)inode->i_mode);
-       ubifs_assert(!atomic_read(&inode->i_count));
+       ubifs_assert(c, !atomic_read(&inode->i_count));
 
        truncate_inode_pages_final(&inode->i_data);
 
@@ -384,9 +384,10 @@ done:
 
 static void ubifs_dirty_inode(struct inode *inode, int flags)
 {
+       struct ubifs_info *c = inode->i_sb->s_fs_info;
        struct ubifs_inode *ui = ubifs_inode(inode);
 
-       ubifs_assert(mutex_is_locked(&ui->ui_mutex));
+       ubifs_assert(c, mutex_is_locked(&ui->ui_mutex));
        if (!ui->dirty) {
                ui->dirty = 1;
                dbg_gen("inode %lu",  inode->i_ino);
@@ -416,7 +417,7 @@ static int ubifs_statfs(struct dentry *dentry, struct kstatfs *buf)
        buf->f_namelen = UBIFS_MAX_NLEN;
        buf->f_fsid.val[0] = le32_to_cpu(uuid[0]) ^ le32_to_cpu(uuid[2]);
        buf->f_fsid.val[1] = le32_to_cpu(uuid[1]) ^ le32_to_cpu(uuid[3]);
-       ubifs_assert(buf->f_bfree <= c->block_cnt);
+       ubifs_assert(c, buf->f_bfree <= c->block_cnt);
        return 0;
 }
 
@@ -441,9 +442,10 @@ static int ubifs_show_options(struct seq_file *s, struct dentry *root)
 
        if (c->mount_opts.override_compr) {
                seq_printf(s, ",compr=%s",
-                          ubifs_compr_name(c->mount_opts.compr_type));
+                          ubifs_compr_name(c, c->mount_opts.compr_type));
        }
 
+       seq_printf(s, ",assert=%s", ubifs_assert_action_name(c));
        seq_printf(s, ",ubi=%d,vol=%d", c->vi.ubi_num, c->vi.vol_id);
 
        return 0;
@@ -921,6 +923,7 @@ static int check_volume_empty(struct ubifs_info *c)
  * Opt_chk_data_crc: check CRCs when reading data nodes
  * Opt_no_chk_data_crc: do not check CRCs when reading data nodes
  * Opt_override_compr: override default compressor
+ * Opt_assert: set ubifs_assert() action
  * Opt_err: just end of array marker
  */
 enum {
@@ -931,6 +934,7 @@ enum {
        Opt_chk_data_crc,
        Opt_no_chk_data_crc,
        Opt_override_compr,
+       Opt_assert,
        Opt_ignore,
        Opt_err,
 };
@@ -945,6 +949,7 @@ static const match_table_t tokens = {
        {Opt_override_compr, "compr=%s"},
        {Opt_ignore, "ubi=%s"},
        {Opt_ignore, "vol=%s"},
+       {Opt_assert, "assert=%s"},
        {Opt_err, NULL},
 };
 
@@ -1045,6 +1050,26 @@ static int ubifs_parse_options(struct ubifs_info *c, char *options,
                        c->default_compr = c->mount_opts.compr_type;
                        break;
                }
+               case Opt_assert:
+               {
+                       char *act = match_strdup(&args[0]);
+
+                       if (!act)
+                               return -ENOMEM;
+                       if (!strcmp(act, "report"))
+                               c->assert_action = ASSACT_REPORT;
+                       else if (!strcmp(act, "read-only"))
+                               c->assert_action = ASSACT_RO;
+                       else if (!strcmp(act, "panic"))
+                               c->assert_action = ASSACT_PANIC;
+                       else {
+                               ubifs_err(c, "unknown assert action \"%s\"", act);
+                               kfree(act);
+                               return -EINVAL;
+                       }
+                       kfree(act);
+                       break;
+               }
                case Opt_ignore:
                        break;
                default:
@@ -1103,7 +1128,7 @@ static void destroy_journal(struct ubifs_info *c)
  */
 static void bu_init(struct ubifs_info *c)
 {
-       ubifs_assert(c->bulk_read == 1);
+       ubifs_assert(c, c->bulk_read == 1);
 
        if (c->bu.buf)
                return; /* Already initialized */
@@ -1134,7 +1159,7 @@ again:
  */
 static int check_free_space(struct ubifs_info *c)
 {
-       ubifs_assert(c->dark_wm > 0);
+       ubifs_assert(c, c->dark_wm > 0);
        if (c->lst.total_free + c->lst.total_dirty < c->dark_wm) {
                ubifs_err(c, "insufficient free space to mount in R/W mode");
                ubifs_dump_budg(c, &c->bi);
@@ -1234,9 +1259,9 @@ static int mount_ubifs(struct ubifs_info *c)
         * Make sure the compressor which is set as default in the superblock
         * or overridden by mount options is actually compiled in.
         */
-       if (!ubifs_compr_present(c->default_compr)) {
+       if (!ubifs_compr_present(c, c->default_compr)) {
                ubifs_err(c, "'compressor \"%s\" is not compiled in",
-                         ubifs_compr_name(c->default_compr));
+                         ubifs_compr_name(c, c->default_compr));
                err = -ENOTSUPP;
                goto out_free;
        }
@@ -1396,10 +1421,10 @@ static int mount_ubifs(struct ubifs_info *c)
                         * the journal head LEBs may also be accounted as
                         * "empty taken" if they are empty.
                         */
-                       ubifs_assert(c->lst.taken_empty_lebs > 0);
+                       ubifs_assert(c, c->lst.taken_empty_lebs > 0);
                }
        } else
-               ubifs_assert(c->lst.taken_empty_lebs > 0);
+               ubifs_assert(c, c->lst.taken_empty_lebs > 0);
 
        err = dbg_check_filesystem(c);
        if (err)
@@ -1429,7 +1454,7 @@ static int mount_ubifs(struct ubifs_info *c)
                  UBIFS_FORMAT_VERSION, UBIFS_RO_COMPAT_VERSION, c->uuid,
                  c->big_lpt ? ", big LPT model" : ", small LPT model");
 
-       dbg_gen("default compressor:  %s", ubifs_compr_name(c->default_compr));
+       dbg_gen("default compressor:  %s", ubifs_compr_name(c, c->default_compr));
        dbg_gen("data journal heads:  %d",
                c->jhead_cnt - NONDATA_JHEADS_CNT);
        dbg_gen("log LEBs:            %d (%d - %d)",
@@ -1610,7 +1635,7 @@ static int ubifs_remount_rw(struct ubifs_info *c)
                        goto out;
        } else {
                /* A readonly mount is not allowed to have orphans */
-               ubifs_assert(c->tot_orphans == 0);
+               ubifs_assert(c, c->tot_orphans == 0);
                err = ubifs_clear_orphans(c);
                if (err)
                        goto out;
@@ -1727,8 +1752,8 @@ static void ubifs_remount_ro(struct ubifs_info *c)
 {
        int i, err;
 
-       ubifs_assert(!c->need_recovery);
-       ubifs_assert(!c->ro_mount);
+       ubifs_assert(c, !c->need_recovery);
+       ubifs_assert(c, !c->ro_mount);
 
        mutex_lock(&c->umount_mutex);
        if (c->bgt) {
@@ -1778,9 +1803,9 @@ static void ubifs_put_super(struct super_block *sb)
         * to write them back because of I/O errors.
         */
        if (!c->ro_error) {
-               ubifs_assert(c->bi.idx_growth == 0);
-               ubifs_assert(c->bi.dd_growth == 0);
-               ubifs_assert(c->bi.data_growth == 0);
+               ubifs_assert(c, c->bi.idx_growth == 0);
+               ubifs_assert(c, c->bi.dd_growth == 0);
+               ubifs_assert(c, c->bi.data_growth == 0);
        }
 
        /*
@@ -1887,7 +1912,7 @@ static int ubifs_remount_fs(struct super_block *sb, int *flags, char *data)
                mutex_unlock(&c->bu_mutex);
        }
 
-       ubifs_assert(c->lst.taken_empty_lebs > 0);
+       ubifs_assert(c, c->lst.taken_empty_lebs > 0);
        return 0;
 }
 
@@ -1929,6 +1954,9 @@ static struct ubi_volume_desc *open_ubi(const char *name, int mode)
        int dev, vol;
        char *endptr;
 
+       if (!name || !*name)
+               return ERR_PTR(-EINVAL);
+
        /* First, try to open using the device node path method */
        ubi = ubi_open_volume_path(name, mode);
        if (!IS_ERR(ubi))
@@ -2002,6 +2030,7 @@ static struct ubifs_info *alloc_ubifs_info(struct ubi_volume_desc *ubi)
                INIT_LIST_HEAD(&c->orph_list);
                INIT_LIST_HEAD(&c->orph_new);
                c->no_chk_data_crc = 1;
+               c->assert_action = ASSACT_RO;
 
                c->highest_inum = UBIFS_FIRST_INO;
                c->lhead_lnum = c->ltail_lnum = UBIFS_LOG_LNUM;
@@ -2053,7 +2082,9 @@ static int ubifs_fill_super(struct super_block *sb, void *data, int silent)
        if (c->max_inode_sz > MAX_LFS_FILESIZE)
                sb->s_maxbytes = c->max_inode_sz = MAX_LFS_FILESIZE;
        sb->s_op = &ubifs_super_operations;
+#ifdef CONFIG_UBIFS_FS_XATTR
        sb->s_xattr = ubifs_xattr_handlers;
+#endif
 #ifdef CONFIG_UBIFS_FS_ENCRYPTION
        sb->s_cop = &ubifs_crypt_operations;
 #endif
@@ -2061,7 +2092,7 @@ static int ubifs_fill_super(struct super_block *sb, void *data, int silent)
        mutex_lock(&c->umount_mutex);
        err = mount_ubifs(c);
        if (err) {
-               ubifs_assert(err < 0);
+               ubifs_assert(c, err < 0);
                goto out_unlock;
        }
 
@@ -2304,8 +2335,8 @@ late_initcall(ubifs_init);
 
 static void __exit ubifs_exit(void)
 {
-       ubifs_assert(list_empty(&ubifs_infos));
-       ubifs_assert(atomic_long_read(&ubifs_clean_zn_cnt) == 0);
+       WARN_ON(list_empty(&ubifs_infos));
+       WARN_ON(atomic_long_read(&ubifs_clean_zn_cnt) == 0);
 
        dbg_debugfs_exit();
        ubifs_compressors_exit();