[PATCH] doc: context_new(3): fix the return value description

[PATCH] doc: context_new(3): fix the return value description

[Pádraig Brady]

---
 libselinux/man/man3/context_new.3 |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/libselinux/man/man3/context_new.3 b/libselinux/man/man3/context_new.3
index 820f927..ca7140e 100644
--- a/libselinux/man/man3/context_new.3
+++ b/libselinux/man/man3/context_new.3
@@ -54,8 +54,8 @@ context_type_set, context_range_set, context_role_set, context_user_set
  Set a context component
 
 .SH "RETURN VALUE"
-On success, zero is returned. On failure, -1 is returned and errno is
-set appropriately.
+On failure, the context_*_set functions return non-zero, while other
+functions return zero or NULL.  errno is set appropriately.
 
 .SH "SEE ALSO"
 .BR selinux "(8)"
-- 
1.7.6.4


--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.

Re: [PATCH] doc: context_new(3): fix the return value description

[Eric Paris]

[-- Attachment #1: Type: text/plain, Size: 159 bytes --]

On Mon, Dec 3, 2012 at 9:33 PM, Pádraig Brady <P@draigbrady.com> wrote:

I applied (and gave you credit for) the following patch.  Does it look
ok to you?

[-- Attachment #2: tmp.patch --]
[-- Type: application/octet-stream, Size: 535 bytes --]

diff --git a/libselinux/man/man3/context_new.3 b/libselinux/man/man3/context_new.3
index 0257445..3dabac3 100644
--- a/libselinux/man/man3/context_new.3
+++ b/libselinux/man/man3/context_new.3
@@ -69,7 +69,13 @@ structure.
 set a context component.
 .
 .SH "RETURN VALUE"
-On success, zero is returned. On failure, \-1 is returned and
+On failure
+.BR context_*_set ()
+functions return non-zero and 0 on success.
+
+The other functions return NULL on failure and non-NULL on success.
+
+On failure
 .I errno
 is set appropriately.
 .

Re: [PATCH] doc: context_new(3): fix the return value description

[Pádraig Brady]

On 12/04/2012 07:05 PM, Eric Paris wrote:
> On Mon, Dec 3, 2012 at 9:33 PM, Pádraig Brady<P@draigbrady.com>  wrote:
>
> I applied (and gave you credit for) the following patch.  Does it look
> ok to you?
>
>
> tmp.patch
>
>
> diff --git a/libselinux/man/man3/context_new.3 b/libselinux/man/man3/context_new.3

> -On success, zero is returned. On failure, \-1 is returned and
> +On failure
> +.BR context_*_set ()
> +functions return non-zero and 0 on success.
> +
> +The other functions return NULL on failure and non-NULL on success.
> +
> +On failure
>   .I errno
>   is set appropriately.
>   .


thanks,
Pádraig.


--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.