Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1236426

Re: [PATCH 5/5] selinux: use sprintf return value

From Rasmus Villemoes <linux@rasmusvillemoes.dk>
Newsgroups linux.kernel
Subject Re: [PATCH 5/5] selinux: use sprintf return value
Date 2015-09-30 17:10 +0200
Message-ID <qerb4-9v-7@gated-at.bofh.it> (permalink)
References <qcJON-6x2-3@gated-at.bofh.it> <qcJON-6x2-1@gated-at.bofh.it> <qe7Fp-5O1-39@gated-at.bofh.it>
Organization D03

Show all headers | View raw


On Tue, Sep 29 2015, Stephen Smalley <sds@tycho.nsa.gov> wrote:

> On 09/25/2015 06:34 PM, Rasmus Villemoes wrote:
>> sprintf returns the number of characters printed (excluding '\0'), so
>> we can use that and avoid duplicating the length computation.
>>
>> Signed-off-by: Rasmus Villemoes <linux@rasmusvillemoes.dk>
>
> Acked-by:  Stephen Smalley <sds@tycho.nsa.gov>

Thanks. I also like Joe's add-on suggestion (except I'd declare user,
role, type as const char*).

Rasmus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 5/5] selinux: use sprintf return value Rasmus Villemoes <linux@rasmusvillemoes.dk> - 2015-09-26 00:40 +0200
  Re: [PATCH 5/5] selinux: use sprintf return value Rasmus Villemoes <linux@rasmusvillemoes.dk> - 2015-09-30 17:10 +0200
  Re: [PATCH 5/5] selinux: use sprintf return value Paul Moore <paul@paul-moore.com> - 2015-09-30 18:50 +0200

csiph-web