On 11/22/2016 4:47 PM, James Morris wrote:
> On Tue, 22 Nov 2016, Dan Jurgens wrote:
>
>> + *out_sid = c->sid[0];
>> + } else {
>> + *out_sid = SECINITSID_UNLABELED;
>> + }
> Per previous comment about the braces.
The coding style says if one branch requires brackets they should be used on all branches:
"This does not apply if only one branch of a conditional statement is a single statement; in the latter case use braces in both branches: "
_______________________________________________
Selinux mailing list
Selinux@xxxxxxxxxxxxx
To unsubscribe, send email to Selinux-leave@xxxxxxxxxxxxx.
To get help, send an email containing "help" to Selinux-request@xxxxxxxxxxxxx.
