Messages in this thread | | | From | Rasmus Villemoes <> | Subject | Re: [PATCH 5/5] selinux: use sprintf return value | Date | Wed, 30 Sep 2015 17:00:04 +0200 |
| |
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
| |