pledge in /bin/mv

2016-04-09 Thread Héctor Luis Gimbatti
Is ok? Index: mv.c === RCS file: /cvs/src/bin/mv/mv.c,v retrieving revision 1.43 diff -u -p -r1.43 mv.c --- mv.c17 Nov 2015 18:34:00 - 1.43 +++ mv.c9 Apr 2016 15:44:59 - @@ -72,6 +72,9 @@ main(int argc,

pledge in /usr.bin/arch

2016-04-09 Thread Héctor Luis Gimbatti
Index: arch.c === RCS file: /cvs/src/usr.bin/arch/arch.c,v retrieving revision 1.17 diff -u -p -r1.17 arch.c --- arch.c 7 Dec 2015 18:11:37 - 1.17 +++ arch.c 9 Apr 2016 15:04:14 - @@ -29,6 +29,7 @@ #include

Re: [patch] ftpd: ptr == NULL

2016-04-05 Thread Héctor Luis Gimbatti
Seems ok > -Original Message- > From: owner-t...@openbsd.org [mailto:owner-t...@openbsd.org] On Behalf > Of frit...@alokat.org > Sent: Tuesday, April 05, 2016 15:22 > To: tech@openbsd.org > Subject: [patch] ftpd: ptr == NULL > > Hi, > > cmd is a ptr. > > --f. > > > Index: ftpd.c >

OpenSSL: replace perror and exit in pledge

2016-04-04 Thread Héctor Luis Gimbatti
Greetings, Probably wasting some bandwith. This diff consists of replacing in openssl if (single_execution) { if (pledge("args", NULL) == -1) { perror("pledge"); exit(1); } } if (single_execution) if (pledge("args", NULL) == -1)

Re: Minor style rewrite for crypt.c

2016-04-03 Thread Héctor Luis Gimbatti
April 03, 2016 23:54 > To: Héctor Luis Gimbatti <h...@etale.com.ar> > Subject: Re: Minor style rewrite for crypt.c > > You'll need to check out the CVS repo and send a diff for things like this. > > Héctor Luis Gimbatti wrote: > > Greetings, > > The function crypt

[pledge] : Add pledge to sail (game)

2016-04-03 Thread Héctor Luis Gimbatti
Index: main.c === RCS file: /cvs/src/games/sail/main.c,v retrieving revision 1.11 diff -u -p -r1.11 main.c --- main.c 8 Jan 2016 20:26:33 - 1.11 +++ main.c 4 Apr 2016 01:19:59 - @@ -45,6 +45,9 @@

Re: bug in grep

2016-04-02 Thread Héctor Luis Gimbatti
Hi, Apparently the error seems to be in /usr/src/usr.bin/grep/util.c at line 400: if ((!(lflag || cflag)) && ((!(bol || eol)) && ((lastHalfDot) && ((firstHalfDot < 0) || ((fg->patternLen - (lastHalfDot + 1)) < firstHalfDot) { fg->reversedSearch

Minor style rewrite for crypt.c

2016-04-02 Thread Héctor Luis Gimbatti
Greetings, The function crypt (/usr/src/lib/libc/crypt/crypt.c) might be rewritten as: char * crypt(const char *key, const char *setting) { if (setting[0] == '$' && setting[1] == '2') return bcrypt(key, setting); errno = EINVAL; return (NULL); } --- HLG

Re: ifconfig segmentation fault

2014-05-16 Thread Héctor Luis Gimbatti
/etc/hostname.if Inet 1.2.3.4 255.255.255.0 NONE -inet6 # ksh /etc/netstart # ifconfig ## NO PROBLEM /etc/hostname.if Inet 1.2.3.4 255.255.255.0 -inet6 # ksh /etc/netstart ifconfig: -inet6: bad value ## NO SEGMENTATION FAULT So, IMHO, if there is any problem at all, of course it should be

Re: vfs references to strncpy and MFSNAMELEN

2014-05-04 Thread Héctor Luis Gimbatti
mention. Best regards -Original Message- From: owner-t...@openbsd.org [mailto:owner-t...@openbsd.org] On Behalf Of patrick keshishian Sent: viernes, 02 de mayo de 2014 00:17 To: tech@openbsd.org; Héctor Luis Gimbatti Subject: Re: vfs references to strncpy and MFSNAMELEN On 4/29/14, Héctor

vfs references to strncpy and MFSNAMELEN

2014-04-29 Thread Héctor Luis Gimbatti
The constant MFSNAMELEN as defined in: lib/libc/sys/getfsstat.2:#define MFSNAMELEN 16 lib/libc/sys/statfs.2:#define MFSNAMELEN 16 sys/sys/mount.h: #define MFSNAMELEN 16 defines the fs type name and, according to comments, it includes nul terminating character. The following code

Re: patch /etc/X11/xinit/xinitrc

2014-04-26 Thread Héctor Luis Gimbatti
. --- HLG Mensaje original De: Stuart Henderson Fecha:26/04/2014 06:38 (GMT-03:00) A: Héctor Luis Gimbatti CC: tech@openbsd.org Asunto: Re: patch /etc/X11/xinit/xinitrc On 2014/04/26 04:33, Héctor Luis Gimbatti wrote: I submit two patches for the file /etc/X11/xinit/xinitrc

patch /etc/X11/xinit/xinitrc

2014-04-25 Thread Héctor Luis Gimbatti
I submit two patches for the file /etc/X11/xinit/xinitrc: 1. patch.p1 it removes extra blank lines in the file. 2. patch.p2 it removes fvwm and xterm references since there is no reason why the user should choose such applications. --- HLG patch.p1 Description: patch.p1 patch.p2 Description: