[libvirt] [PATCH v4 21/23] virSecuritySELinuxRestoreFileLabel: Adjust code pattern
John Ferlan
jferlan at redhat.com
Mon Sep 17 22:18:51 UTC 2018
On 09/10/2018 05:36 AM, Michal Privoznik wrote:
> Firstly, the following code pattern is harder to follow:
>
> if (func() < 0) {
> error();
> } else {
> /* success */
> }
>
> We should put 'goto cleanup' into the error branch and move the
> else branch one level up.
> Secondly, 'rc' should really be named 'ret' because it holds
> return value of the function. Not some intermediate value.
>
> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
> ---
> src/security/security_selinux.c | 13 ++++++++-----
> 1 file changed, 8 insertions(+), 5 deletions(-)
>
Reviewed-by: John Ferlan <jferlan at redhat.com>
John
can't wait to see the selinux patches for this list stuff... Oh boy!
Getting punchy, must be closer to dinner...
More information about the libvir-list
mailing list