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

2015-09-30 Thread Paul Moore
On Saturday, September 26, 2015 12:34:19 AM 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 > --- > security/selinux/ss/services.c | 5 + > 1

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

2015-09-30 Thread Rasmus Villemoes
On Tue, Sep 29 2015, Stephen Smalley 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 > > Acked-by: Stephen

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

2015-09-30 Thread Rasmus Villemoes
On Tue, Sep 29 2015, Stephen Smalley 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

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

2015-09-30 Thread Paul Moore
On Saturday, September 26, 2015 12:34:19 AM 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 > --- >

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

2015-09-29 Thread Stephen Smalley
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 Acked-by: Stephen Smalley --- security/selinux/ss/services.c | 5 + 1

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

2015-09-29 Thread Stephen Smalley
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 Acked-by: Stephen Smalley ---

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

2015-09-25 Thread Rasmus Villemoes
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 --- security/selinux/ss/services.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git

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

2015-09-25 Thread Rasmus Villemoes
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 --- security/selinux/ss/services.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git