From 49cfc158f33e509c5cfc6a9c6511c392398f1e45 Mon Sep 17 00:00:00 2001 From: "Serge E. Hallyn" Date: Mon, 2 Feb 2009 15:07:33 -0800 Subject: [PATCH] --- yaml --- r: 132595 b: refs/heads/master c: faa3aad75a959f55e7783f4dc7840253c7506571 h: refs/heads/master i: 132593: c50b8b368fe94a4e61865e9e48f216e737becd0a 132591: 8a68145bedd4920e5fd109ec04ee73d10b00988a v: v3 --- [refs] | 2 +- trunk/security/inode.c | 7 +++---- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/[refs] b/[refs] index c3f0a381d4e9..11306b003ed6 100644 --- a/[refs] +++ b/[refs] @@ -1,2 +1,2 @@ --- -refs/heads/master: 5626d3e86141390c8efc7bcb929b6a4b58b00480 +refs/heads/master: faa3aad75a959f55e7783f4dc7840253c7506571 diff --git a/trunk/security/inode.c b/trunk/security/inode.c index efea5a605466..b41e708147ae 100644 --- a/trunk/security/inode.c +++ b/trunk/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,