diff options
| author | Jiri Slaby <[email protected]> | 2011-05-26 16:25:17 -0700 |
|---|---|---|
| committer | Linus Torvalds <[email protected]> | 2011-05-26 17:12:34 -0700 |
| commit | dcb3a08e69629ea65a3e9647da730bfaf670497d (patch) | |
| tree | da89f661c9e0b2ae3d1dabcc3e27dcf06d2ae8fa /Documentation/filesystems/configfs/configfs_example_macros.c | |
| parent | 02d54f092697b6046e466e447cc694b0e6ed45d0 (diff) | |
Documentation: configfs examples crash fix
When configfs_register_subsystem() fails, we unregister too many
subsystems in configfs_example_init. Decrement i by one to not unregister
non-registered subsystem.
[[email protected]: coding-style fixes]
Signed-off-by: Jiri Slaby <[email protected]>
Cc: Joel Becker <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
Diffstat (limited to 'Documentation/filesystems/configfs/configfs_example_macros.c')
| -rw-r--r-- | Documentation/filesystems/configfs/configfs_example_macros.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/Documentation/filesystems/configfs/configfs_example_macros.c b/Documentation/filesystems/configfs/configfs_example_macros.c index d8e30a0378aa..327dfbc640a9 100644 --- a/Documentation/filesystems/configfs/configfs_example_macros.c +++ b/Documentation/filesystems/configfs/configfs_example_macros.c @@ -427,9 +427,8 @@ static int __init configfs_example_init(void) return 0; out_unregister: - for (; i >= 0; i--) { + for (i--; i >= 0; i--) configfs_unregister_subsystem(example_subsys[i]); - } return ret; } @@ -438,9 +437,8 @@ static void __exit configfs_example_exit(void) { int i; - for (i = 0; example_subsys[i]; i++) { + for (i = 0; example_subsys[i]; i++) configfs_unregister_subsystem(example_subsys[i]); - } } module_init(configfs_example_init); |