Hi,
Theo de Raadt wrote on Fri, Nov 13, 2015 at 08:21:41AM -0700:
> Theo Buehler wrote:
>> This straightforward pledge("stdio") is one of the last uncommitted ones
>> from Theo's big 'tame in userland' diff and seems to have been
>> overlooked so far.
> I think arch.c gains no value from being pledged. It adds a system
> call to a program which makes no user-input decisions.
Except for setlocale(LC_ALL, "");
but why the hell does it call that?
Tholo@ added that shortly after Caesar was murdered, but the commit
message doesn't explain why.
OK?
Ingo
Index: arch.c
===================================================================
RCS file: /cvs/src/usr.bin/arch/arch.c,v
retrieving revision 1.16
diff -u -p -r1.16 arch.c
--- arch.c 25 Sep 2015 16:19:26 -0000 1.16
+++ arch.c 13 Nov 2015 18:38:08 -0000
@@ -25,7 +25,6 @@
#include <sys/param.h> /* MACHINE MACHINE_ARCH */
-#include <locale.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
@@ -41,8 +40,6 @@ main(int argc, char *argv[])
extern char *__progname;
int short_form = 0, c;
char *arch, *opts;
-
- setlocale(LC_ALL, "");
machine = strcmp(__progname, "machine") == 0;
if (machine) {
