Re: login_paswd.c: cosmetic diff

2009-05-09 Thread Theo de Raadt
> This diff just adds endpwdnet() info login_passwd.c > This is probably not needed as we will exit() anyway later on. There is absolutely no need for this since neither setpwent() or getpwent() were called. The database is not open. > > diff---

Re: [PATCH] bioctl(8): fix dd invocation

2009-05-09 Thread Joachim Schipper
On Sat, May 09, 2009 at 02:27:08PM +0200, Mark Kettenis wrote: > > Date: Sat, 9 May 2009 12:20:25 +0200 > > From: Joachim Schipper > > > > The bioctl(8) man page helpfully suggests dd for zeroing out the > > disklabel etc. on the new disk. However, the command doesn't work, since > > OpenBSD's dd

Re: [PATCH] bioctl(8): fix dd invocation

2009-05-09 Thread Pierre Riteau
On Sat, May 09, 2009 at 12:20:25PM +0200, Joachim Schipper wrote: > The bioctl(8) man page helpfully suggests dd for zeroing out the > disklabel etc. on the new disk. However, the command doesn't work, since > OpenBSD's dd doesn't understand '1M'. > > It may be slightly more readable to use 'bs=10

Re: [PATCH] bioctl(8): fix dd invocation

2009-05-09 Thread Pierre Riteau
On Sat, May 09, 2009 at 02:30:13PM +0200, Pierre Riteau wrote: > Or, rather, the following diff. But I'm not sure having to places ^^ two

Re: [PATCH] bioctl(8): fix dd invocation

2009-05-09 Thread Mark Kettenis
> Date: Sat, 9 May 2009 12:20:25 +0200 > From: Joachim Schipper > > The bioctl(8) man page helpfully suggests dd for zeroing out the > disklabel etc. on the new disk. However, the command doesn't work, since > OpenBSD's dd doesn't understand '1M'. Uh, indeed it doesn't, but it does understand '1

Re: [PATCH] bioctl(8): fix dd invocation

2009-05-09 Thread Thordur I. Bjornsson
Joachim Schipper wrote on Sat 9.May'09 at 12:20:25 +0200 > The bioctl(8) man page helpfully suggests dd for zeroing out the > disklabel etc. on the new disk. However, the command doesn't work, since > OpenBSD's dd doesn't understand '1M'. > > It may be slightly more readable to use 'bs=1024 co

[PATCH] bioctl(8): fix dd invocation

2009-05-09 Thread Joachim Schipper
The bioctl(8) man page helpfully suggests dd for zeroing out the disklabel etc. on the new disk. However, the command doesn't work, since OpenBSD's dd doesn't understand '1M'. It may be slightly more readable to use 'bs=1024 count=1024' or some variant. Joachim Index: bioctl.8 ==

login_paswd.c: cosmetic diff

2009-05-09 Thread Maxim Bourmistrov
This diff just adds endpwdnet() info login_passwd.c This is probably not needed as we will exit() anyway later on. diff-- --- login_passwd.c.orig Tue Aug 1 12:12:10 2006 +++ login_passwd.c Sat May 9 13:54:32 2009 @@ -57