diff options
| author | Mathieu Desnoyers <compudj@krystal.dyndns.org> | 2006-11-24 19:48:19 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2006-12-14 00:38:45 +0100 |
| commit | 63223a0654c2a473ae64835819b87826cb7415ee (patch) | |
| tree | 8c8d74c7bb4cd9b9f538d4701d92b2e6a3cca496 /fs/debugfs | |
| parent | DebugFS : coding style fixes (diff) | |
| download | linux-63223a0654c2a473ae64835819b87826cb7415ee.tar.xz linux-63223a0654c2a473ae64835819b87826cb7415ee.zip | |
DebugFS : file/directory creation error handling
Fix error handling of file and directory creation in DebugFS.
The error path should release the file system because no _remove will be called
for this erroneous creation.
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@polymtl.ca>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'fs/debugfs')
| -rw-r--r-- | fs/debugfs/inode.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/debugfs/inode.c b/fs/debugfs/inode.c index 05d1a9c4fb76..d6c5fb53c746 100644 --- a/fs/debugfs/inode.c +++ b/fs/debugfs/inode.c @@ -206,13 +206,15 @@ struct dentry *debugfs_create_file(const char *name, mode_t mode, pr_debug("debugfs: creating file '%s'\n",name); - error = simple_pin_fs(&debug_fs_type, &debugfs_mount, &debugfs_mount_count); + error = simple_pin_fs(&debug_fs_type, &debugfs_mount, + &debugfs_mount_count); if (error) goto exit; error = debugfs_create_by_name(name, mode, parent, &dentry); if (error) { dentry = NULL; + simple_release_fs(&debugfs_mount, &debugfs_mount_count); goto exit; } |