diff options
| author | Serge E. Hallyn <serue@us.ibm.com> | 2009-02-02 15:07:33 -0800 |
|---|---|---|
| committer | James Morris <jmorris@namei.org> | 2009-02-03 11:02:51 +1100 |
| commit | faa3aad75a959f55e7783f4dc7840253c7506571 (patch) | |
| tree | 88e6c94cda322ff2b32f72bb8d96f9675cdad8aa /security/inode.c | |
| parent | 5626d3e86141390c8efc7bcb929b6a4b58b00480 (diff) | |
| download | linux-3.10-faa3aad75a959f55e7783f4dc7840253c7506571.tar.gz linux-3.10-faa3aad75a959f55e7783f4dc7840253c7506571.tar.bz2 linux-3.10-faa3aad75a959f55e7783f4dc7840253c7506571.zip | |
securityfs: fix long-broken securityfs_create_file comment
If there is an error creating a file through securityfs_create_file,
NULL is not returned, rather the error is propagated.
Signed-off-by: Serge E. Hallyn <serue@us.ibm.com>
Signed-off-by: James Morris <jmorris@namei.org>
Diffstat (limited to 'security/inode.c')
| -rw-r--r-- | security/inode.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/security/inode.c b/security/inode.c index efea5a60546..b41e708147a 100644 --- a/security/inode.c +++ b/security/inode.c @@ -205,12 +205,11 @@ static int create_by_name(const char *name, mode_t mode, * This function returns a pointer to a dentry if it succeeds. This * pointer must be passed to the securityfs_remove() function when the file is * to be removed (no automatic cleanup happens if your module is unloaded, - * you are responsible here). If an error occurs, %NULL is returned. + * you are responsible here). If an error occurs, the function will return + * the erorr value (via ERR_PTR). * * If securityfs is not enabled in the kernel, the value %-ENODEV is - * returned. It is not wise to check for this value, but rather, check for - * %NULL or !%NULL instead as to eliminate the need for #ifdef in the calling - * code. + * returned. */ struct dentry *securityfs_create_file(const char *name, mode_t mode, struct dentry *parent, void *data, |