<div dir="ltr"><div>Hi Thomas,</div><div><br></div>FYI, It's not checking the same string <span style="font-size:13.1999998092651px;line-height:19.7999992370605px">twice</span>, notice the case change (X86 vs x86)<div><br></div><div>Best,</div></div><br><div class="gmail_quote"><div dir="ltr">On Sat, Jul 18, 2015 at 4:28 PM Thomas Lamprecht <<a href="mailto:t.lamprecht@proxmox.com">t.lamprecht@proxmox.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Instead of checking if it's "[x86_64]" twice,<br>
check for "[x86_64]" or "[amd64]"<br>
Signed-off-by: Thomas Lamprecht <<a href="mailto:t.lamprecht@proxmox.com" target="_blank">t.lamprecht@proxmox.com</a>><br>
<br>
Signed-off-by: Thomas Lamprecht <<a href="mailto:t.lamprecht@proxmox.com" target="_blank">t.lamprecht@proxmox.com</a>><br>
---<br>
src/lxc/seccomp.c | 2 +-<br>
1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/src/lxc/seccomp.c b/src/lxc/seccomp.c<br>
index 108faa0..374d54f 100644<br>
--- a/src/lxc/seccomp.c<br>
+++ b/src/lxc/seccomp.c<br>
@@ -333,7 +333,7 @@ static int parse_config_v2(FILE *f, char *line, struct lxc_conf *conf)<br>
goto bad;<br>
}<br>
} else if (strcmp(line, "[X86_64]") == 0 ||<br>
- strcmp(line, "[x86_64]") == 0) {<br>
+ strcmp(line, "[amd64]") == 0) {<br>
if (native_arch != lxc_seccomp_arch_amd64) {<br>
cur_rule_arch = lxc_seccomp_arch_unknown;<br>
continue;<br>
--<br>
2.4.5<br>
<br>
_______________________________________________<br>
lxc-devel mailing list<br>
<a href="mailto:lxc-devel@lists.linuxcontainers.org" target="_blank">lxc-devel@lists.linuxcontainers.org</a><br>
<a href="http://lists.linuxcontainers.org/listinfo/lxc-devel" rel="noreferrer" target="_blank">http://lists.linuxcontainers.org/listinfo/lxc-devel</a><br>
</blockquote></div>