[PATCH] security/selinux/hooks: remove unneeded semicolon

Subsystems: selinux security module, the rest

STALE2798d

2 messages, 2 authors, 2019-01-10 · open the first message on its own page

[PATCH] security/selinux/hooks: remove unneeded semicolon

From: Peng Hao <hidden>
Date: 2019-01-04 00:55:47

Remove unneeded semicolon.

Signed-off-by: Peng Hao <redacted>
---
 security/selinux/hooks.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/security/selinux/hooks.c b/security/selinux/hooks.c
index a67459e..73c679e 100644
--- a/security/selinux/hooks.c
+++ b/security/selinux/hooks.c
@@ -1263,7 +1263,7 @@ static void selinux_write_opts(struct seq_file *m,
 		default:
 			BUG();
 			return;
-		};
+		}
 		/* we need a comma before each option */
 		seq_putc(m, ',');
 		seq_puts(m, prefix);
-- 
1.8.3.1

Re: [PATCH] security/selinux/hooks: remove unneeded semicolon

From: Paul Moore <paul@paul-moore.com>
Date: 2019-01-10 00:54:33

On Thu, Jan 3, 2019 at 7:55 PM Peng Hao [off-list ref] wrote:
quoted hunk
Remove unneeded semicolon.

Signed-off-by: Peng Hao <redacted>
---
 security/selinux/hooks.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/security/selinux/hooks.c b/security/selinux/hooks.c
index a67459e..73c679e 100644
--- a/security/selinux/hooks.c
+++ b/security/selinux/hooks.c
@@ -1263,7 +1263,7 @@ static void selinux_write_opts(struct seq_file *m,
                default:
                        BUG();
                        return;
-               };
+               }
                /* we need a comma before each option */
                seq_putc(m, ',');
                seq_puts(m, prefix);
Thank you for your patch, but the selinux_write_opts() function was
just removed in v5.0-rc1 in commit e3489f8974e1 ("selinux: kill
selinux_sb_get_mnt_opts()").

-- 
paul moore
www.paul-moore.com
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help