Re: [patch] file(1) determine file type of stdin

2015-05-28 Thread Nicholas Marriott
Hi Can you make a patch against -current? Your source tree seems pretty out of date. Yes please check the return values from fcntl and fstat. On Thu, May 28, 2015 at 08:37:02PM +0200, S??bastien Marie wrote: > Hi, > > It was possible, using the previous file(1) version, to determine the > fil

Re: pf_create_state() is sometimes better to use pf_unlink_state()

2015-05-28 Thread Alexandr Nedvedicky
> >> > >> But we'll drop this "reference" in pf_src_tree_remove_state, > >> then how will sns[PF_SN_NAT] and sns[PF_SN_ROUTE] be different? > > > > I think I should take PF class again ;-) I've just realized there > > is a test in pf_remove_src_node(): > > > > 572 if (sn->states > 0 ||

Re: pf_create_state() is sometimes better to use pf_unlink_state()

2015-05-28 Thread Mike Belopuhov
On 28 May 2015 at 21:22, Alexandr Nedvedicky wrote: >> > > > sorry I was too brief. The snippet below comes from >> > > > pf_create_state() with your patch applied: >> > > > >> > > >3560 for (i = 0; i < PF_SN_MAX; i++) >> > > >3561 if (sns[i] != NULL) { >> > > >

Re: pf_create_state() is sometimes better to use pf_unlink_state()

2015-05-28 Thread Alexandr Nedvedicky
> > > > sorry I was too brief. The snippet below comes from > > > > pf_create_state() with your patch applied: > > > > > > > >3560 for (i = 0; i < PF_SN_MAX; i++) > > > >3561 if (sns[i] != NULL) { > > > >3562 struct pf_sn_item *sni;

Re: Fix a typo in httpd.conf(5) manpage

2015-05-28 Thread Jason McIntyre
On Tue, May 26, 2015 at 02:47:32AM +0400, Yegor Timoschenko wrote: > Context in httpd.conf(5): > > block [return code [uri]] > Close the connection and send an error page. If the optional > return code is not specified, httpd(8) denies access with a `403 >

Re: [patch] correct file(1) printed usage

2015-05-28 Thread Jason McIntyre
On Thu, May 28, 2015 at 07:37:52PM +0200, S??bastien Marie wrote: > Hi, > > I would report (and correct) an invalid usage statement of file(1). > > $ file > usage: file [-bchiLsW] [file ...] > > $ man file | grep -A1 'SY' > SYNOPSIS > file [-bchiLsW] file ... > > As at least one argument i

Re: [patch] file(1) determine file type of stdin

2015-05-28 Thread Stuart Henderson
On 2015/05/28 20:37, Sébastien Marie wrote: > file(1) in 5.5: > --- > $ echo foobar | file - > /dev/stdin: ASCII text > > after patching: > --- > $ echo foobar | file - > -: fifo (named pipe) > - should I rename "-" to "/dev/stdin" in order to have similar output >

[patch] file(1) determine file type of stdin

2015-05-28 Thread Sébastien Marie
Hi, It was possible, using the previous file(1) version, to determine the file type of stdin stream. The current version don't permit that (or I don't found how to do that). I would propose a patch in order to support '-' argument to be trait as STDIN_FILENO descriptor. file(1) in 5.5:

[patch] correct file(1) printed usage

2015-05-28 Thread Sébastien Marie
Hi, I would report (and correct) an invalid usage statement of file(1). $ file usage: file [-bchiLsW] [file ...] $ man file | grep -A1 'SY' SYNOPSIS file [-bchiLsW] file ... As at least one argument is mandatory, removing the [] would make sens. Thanks. -- Sébastien Marie Index: file.c

Re: pf_create_state() is sometimes better to use pf_unlink_state()

2015-05-28 Thread Mike Belopuhov
On Thu, May 28, 2015 at 13:34 +0200, Alexandr Nedvedicky wrote: > On Thu, May 28, 2015 at 11:43:02AM +0200, Mike Belopuhov wrote: > > On Thu, May 28, 2015 at 01:17 +0200, Alexandr Nedvedicky wrote: > > > Hello, > > > > > > > > > On Wed, May 27, 2015 at 07:44:15PM +0200, Mike Belopuhov wrote: > >

Re: OpenBSD/NTRU policy mismatch [Was: NTRU Open Source Project / Post-quantum era]

2015-05-28 Thread ertetlen barmok
thanks for the comments! Luckily there are still a few algorithms if NTRU is not good yet: https://en.wikipedia.org/wiki/Post-quantum_cryptography As time goes, maybe this will be a more and more relevant thing. Original Message From: Okembe Mbwambo Apparently from: owner-t

[PATCH] Fix ospfd/carp start-up issue

2015-05-28 Thread Johan Ymerson
Hi, After the fix in carp to correctly initialize link state (ip_carp.c r1.257), ospfd no longer detect all carp interfaces in "backup" mode reliably on start-up. The problem is that carp interfaces in backup state isn't handled the same way on start-up as it is when up and running. Here is an e

Re: pf_create_state() is sometimes better to use pf_unlink_state()

2015-05-28 Thread Alexandr Nedvedicky
On Thu, May 28, 2015 at 11:43:02AM +0200, Mike Belopuhov wrote: > On Thu, May 28, 2015 at 01:17 +0200, Alexandr Nedvedicky wrote: > > Hello, > > > > > > On Wed, May 27, 2015 at 07:44:15PM +0200, Mike Belopuhov wrote: > > > On Wed, May 27, 2015 at 10:39 +0200, Alexandr Nedvedicky wrote: > > > > He

Re: pf_create_state() is sometimes better to use pf_unlink_state()

2015-05-28 Thread Mike Belopuhov
On Thu, May 28, 2015 at 01:17 +0200, Alexandr Nedvedicky wrote: > Hello, > > > On Wed, May 27, 2015 at 07:44:15PM +0200, Mike Belopuhov wrote: > > On Wed, May 27, 2015 at 10:39 +0200, Alexandr Nedvedicky wrote: > > > Hello, > > > > > > > > - if (pf_state_insert(BOUND_IFACE(r, pd->kif), skw

tun(4) and if_input()

2015-05-28 Thread Martin Pieuchot
Replace the last ether_input_mbuf() by if_input(). Index: net/if_tun.c === RCS file: /cvs/src/sys/net/if_tun.c,v retrieving revision 1.144 diff -u -p -r1.144 if_tun.c --- net/if_tun.c26 May 2015 11:36:26 - 1.144 +++ n

bridge(4) and if_input()

2015-05-28 Thread Martin Pieuchot
Let's use if_input() rather than ether_input_mbuf(). This simplifies the bpf(4) bits and removes some duplicate "rcvif" assignments. Once ether_input_mbuf() is no more my plan is to start merging various ifp counters and put an interface index in the packet header. ok? Index: net/if_bridge.c ==

Re: carp(4) is out

2015-05-28 Thread Martin Pieuchot
On 23/05/15(Sat) 19:08, mxb wrote: > [...] > so far no problems. Here's an updated version to match recent changes. It also includes a nitpick fix from bluhm@. Tests and oks welcome. Index: net/if_ethersubr.c === RCS file: /cvs/src