Re: malloc: more info in error message for write-after-free with option D

2023-10-22 Thread Masato Asou
ral write of free memory if malloc option D is active. Knowing the > place where allocations were done often helps to find out where the > overwrite happened. > > If option D is active malloc now saves caller info in a separate page > instead of only doing that for chunk index 0.

malloc: more info in error message for write-after-free with option D

2023-10-10 Thread Otto Moerbeek
Hi, This diff adds better error reporting for write-after-free or the more general write of free memory if malloc option D is active. Knowing the place where allocations were done often helps to find out where the overwrite happened. If option D is active malloc now saves caller info in a

Re: MALLOC_STATS: dump internal state and leak info via utrace(2)

2023-04-15 Thread Otto Moerbeek
tu your_program > > > > > > > $ kdump -u mallocdumpline > > > > > > > > > > > > > > and for profiling: > > > > > > > > > > > > > > $ kdump -u profil > gmon.out > > > >

Re: MALLOC_STATS: dump internal state and leak info via utrace(2)

2023-04-13 Thread Otto Moerbeek
> > > > > > filter on. > > > > > > > > > > > > $ MALLOC_OPTIONS=D ktrace -tu your_program > > > > > > $ kdump -u mallocdumpline > > > > > > > > > > > > and for profiling: > > > > &g

Re: MALLOC_STATS: dump internal state and leak info via utrace(2)

2023-04-11 Thread Otto Moerbeek
gt; > > > $ kdump -u mallocdumpline > > > > > > > > > > and for profiling: > > > > > > > > > > $ kdump -u profil > gmon.out > > > > > $ gprof your_program gmon.out > > > > > > > > > &

Re: MALLOC_STATS: dump internal state and leak info via utrace(2)

2023-04-09 Thread Sebastien Marie
o the call so I > > could enrich the ktrace with userland trace info. So if kdump -u is used > > for more then just mallocstats it should have a true binary mode. For > > example gmon.out is a binary format so the above example by semarie@ would > > not work. As usual this ca

Re: MALLOC_STATS: dump internal state and leak info via utrace(2)

2023-04-09 Thread Otto Moerbeek
> gmon.out > > > > $ gprof your_program gmon.out > > > > > > > > Thanks. > > > > > > Thanks! Your suggestions make a lot of sense. I'll rework the kdump > > > part to make it more flexable for different purposes. > > >

Re: MALLOC_STATS: dump internal state and leak info via utrace(2)

2023-04-09 Thread Sebastien Marie
gt; should be line buffered). It makes the regular kdump output more useable. > Right now you depend on kdump -u to put the lines back together. > > Whenever I used utrace() I normally passed binary objects to the call so I > could enrich the ktrace with userland trace info. So if kdump -

Re: MALLOC_STATS: dump internal state and leak info via utrace(2)

2023-04-09 Thread Claudio Jeker
of sense. I'll rework the kdump > > part to make it more flexable for different purposes. > > Anothew aspect of safety is the information availble in the heap > itself. I'm pretty sure the addresses of call sites into malloc are > interesting to attackers. To prevent a prog

Re: MALLOC_STATS: dump internal state and leak info via utrace(2)

2023-04-09 Thread Sebastien Marie
On Sun, Apr 09, 2023 at 09:15:12AM +0200, Otto Moerbeek wrote: > On Sun, Apr 09, 2023 at 08:20:43AM +0200, Otto Moerbeek wrote: > > > > > Thanks! Your suggestions make a lot of sense. I'll rework the kdump > > part to make it more flexable for different purposes. > > Anothew aspect of safety is

Re: MALLOC_STATS: dump internal state and leak info via utrace(2)

2023-04-09 Thread Otto Moerbeek
endif /* MALLOC_STATS */ +#endif while ((mopts.malloc_canary = arc4random()) == 0) ; @@ -596,9 +600,7 @@ insert(struct dir_info *d, void *p, size } d->r[index].p = p; d->r[index].size = sz; -#ifdef MALLOC_STATS - d->r[index].f = f; -#endif +

Re: MALLOC_STATS: dump internal state and leak info via utrace(2)

2023-04-08 Thread Otto Moerbeek
On Sun, Apr 09, 2023 at 07:53:31AM +0200, Sebastien Marie wrote: > On Fri, Apr 07, 2023 at 09:52:52AM +0200, Otto Moerbeek wrote: > > > Hi, > > > > > > This is work in progress. I have to think if the flags to kdump I'm > > > introducing should be two or a single one. > > > > > > Currently, mall

Re: MALLOC_STATS: dump internal state and leak info via utrace(2)

2023-04-08 Thread Sebastien Marie
On Fri, Apr 07, 2023 at 09:52:52AM +0200, Otto Moerbeek wrote: > > Hi, > > > > This is work in progress. I have to think if the flags to kdump I'm > > introducing should be two or a single one. > > > > Currently, malloc.c can be compiled with MALLOC_STATS defined. If run > > with option D it dump

Re: MALLOC_STATS: dump internal state and leak info via utrace(2)

2023-04-07 Thread Otto Moerbeek
; > Feedback appreciated. Second iteration. Main difference: docs and the leak report is greatly enhanced by adding info that can be fed to addr2line. Exmaple: ... Leak report: f sum #avg 0x9fd0eb58abb 40960 10 4096 addr2line -e ./a.out 0x1abb

MALLOC_STATS: dump internal state and leak info via utrace(2)

2023-04-05 Thread Otto Moerbeek
Hi, This is work in progress. I have to think if the flags to kdump I'm introducing should be two or a single one. Currently, malloc.c can be compiled with MALLOC_STATS defined. If run with option D it dumps its state to a malloc.out file at exit. This state can be used to find leaks amongst othe

Re: rpki-client keep http connection info around

2022-11-02 Thread Theo Buehler
On Wed, Nov 02, 2022 at 11:55:12AM +0100, Claudio Jeker wrote: > Job's diff made me realise that clearing the connection info (conn->res) > makes error reporting worse. It is not like we save lots of memory by > doing so. So do not call freeaddrinfo() in http_connect_done(), now &

rpki-client keep http connection info around

2022-11-02 Thread Claudio Jeker
Job's diff made me realise that clearing the connection info (conn->res) makes error reporting worse. It is not like we save lots of memory by doing so. So do not call freeaddrinfo() in http_connect_done(), now http_free() will free res0 before freeing conn. -- :wq Claudio Index

Re: cp.1: zap redundant info

2022-09-20 Thread Crystal Kolipe
On Tue, Sep 20, 2022 at 02:34:08PM +0100, Jason McIntyre wrote: > On Tue, Sep 20, 2022 at 08:37:27AM -0300, Crystal Kolipe wrote: > > On Tue, Sep 20, 2022 at 10:54:10AM +, Klemens Nanni wrote: > > > Reads like pointing out the obvious and the next sentence also explains > > > how -H and -L beha

Re: cp.1: zap redundant info

2022-09-20 Thread Klemens Nanni
Thanks Crystal and Jason, the existing wording makes more sense to me and I'd leave it as is.

Re: cp.1: zap redundant info

2022-09-20 Thread Jason McIntyre
On Tue, Sep 20, 2022 at 08:37:27AM -0300, Crystal Kolipe wrote: > On Tue, Sep 20, 2022 at 10:54:10AM +, Klemens Nanni wrote: > > Does this negation of the main sentence add anything I don't get? > > Yes. The wording you are proposing to remove makes it clear that > setting -R sets the 'defaul

Re: cp.1: zap redundant info

2022-09-20 Thread Crystal Kolipe
On Tue, Sep 20, 2022 at 10:54:10AM +, Klemens Nanni wrote: > Does this negation of the main sentence add anything I don't get? Yes. The wording you are proposing to remove makes it clear that setting -R sets the 'default' behaviour, (which can be overridden by other flags), rather than forcin

cp.1: zap redundant info

2022-09-20 Thread Klemens Nanni
Does this negation of the main sentence add anything I don't get? Reads like pointing out the obvious and the next sentence also explains how -H and -L behave due to -R, just like -H and -L descriptions themselves. Index: cp.1 === RC

Re: rpki-client: print info about encapsulated certs & PEM format in filemode

2022-08-25 Thread Theo Buehler
On Thu, Aug 25, 2022 at 05:06:33PM +, Job Snijders wrote: > On Thu, Aug 25, 2022 at 06:38:36PM +0200, Theo Buehler wrote: > > On Thu, Aug 25, 2022 at 04:04:27PM +, Job Snijders wrote: > > > On Thu, Aug 25, 2022 at 03:38:45PM +0200, Claudio Jeker wrote: > > > > I wonder why is PEM printing n

Re: rpki-client: print info about encapsulated certs & PEM format in filemode

2022-08-25 Thread Job Snijders
On Thu, Aug 25, 2022 at 06:38:36PM +0200, Theo Buehler wrote: > On Thu, Aug 25, 2022 at 04:04:27PM +, Job Snijders wrote: > > On Thu, Aug 25, 2022 at 03:38:45PM +0200, Claudio Jeker wrote: > > > I wonder why is PEM printing not part of -f? It seems to be something > > > that should be part of f

Re: rpki-client: print info about encapsulated certs & PEM format in filemode

2022-08-25 Thread Theo Buehler
On Thu, Aug 25, 2022 at 04:04:27PM +, Job Snijders wrote: > On Thu, Aug 25, 2022 at 03:38:45PM +0200, Claudio Jeker wrote: > > I wonder why is PEM printing not part of -f? It seems to be something > > that should be part of filemode. > > OK, how about this? That's a lot better. However X509_p

Re: rpki-client: print info about encapsulated certs & PEM format in filemode

2022-08-25 Thread Job Snijders
On Thu, Aug 25, 2022 at 03:38:45PM +0200, Claudio Jeker wrote: > I wonder why is PEM printing not part of -f? It seems to be something > that should be part of filemode. OK, how about this? Kind regards, Job Index: filemode.c === R

Re: rpki-client pass repo info to parser process

2022-01-11 Thread Claudio Jeker
On Tue, Jan 11, 2022 at 11:36:19AM +, Job Snijders wrote: > On Mon, Jan 10, 2022 at 03:30:23PM +0100, Claudio Jeker wrote: > > + if (RB_INSERT(repo_tree, &repos, rp) != NULL) > > + errx(1, "repository already added to repo tree %d, %s", id, > > path); > > + warnx("repository adde

Re: rpki-client pass repo info to parser process

2022-01-11 Thread Job Snijders
On Mon, Jan 10, 2022 at 03:30:23PM +0100, Claudio Jeker wrote: > + if (RB_INSERT(repo_tree, &repos, rp) != NULL) > + errx(1, "repository already added to repo tree %d, %s", id, > path); > + warnx("repository added to repo tree %d, %s", id, path); The above warnx() probably nee

Re: rpki-client pass repo info to parser process

2022-01-11 Thread Theo Buehler
On Mon, Jan 10, 2022 at 03:30:23PM +0100, Claudio Jeker wrote: > This diff changes the way the parser figures out which file to work on. > Until now the parent process sent a full path to the parser but that does > not work well with the idea of splitting the repo up into validated, rsync > and rrd

rpki-client pass repo info to parser process

2022-01-10 Thread Claudio Jeker
@@ -86,7 +86,7 @@ struct tarepo { unsigned int id; enum repo_state state; }; -SLIST_HEAD(, tarepo) tarepos = SLIST_HEAD_INITIALIZER(tarepos); +static SLIST_HEAD(, tarepo)tarepos = SLIST_HEAD_INITIALIZER(tarepos); struct repo { SLIST_EN

bgpctl(8): Print MPLS label info in show rib detail output

2021-11-29 Thread Mitchell Krome
Hi, I noticed bgpctl didn't seem to have a way to show the MPLS label attached to MPLS L3VPN routes. The label was already there in the prefix info it just wasn't printed, so this diff adds printing the label info if the prefix has one in the show rib detail command. Example output be

Re: bgpd show proper info in Adj-RIB-Out

2021-06-16 Thread Claudio Jeker
On Tue, Jun 15, 2021 at 06:14:38PM +0200, Claudio Jeker wrote: > The Adj-RIB-Out should show what is sent to the peer. bgpd did not fully > do that since it adjusted the ASPATH and the nexthop afterwards when > building the actual UPDATE. > > This diff changes that and moves the ASPATH prepend for

bgpd show proper info in Adj-RIB-Out

2021-06-15 Thread Claudio Jeker
The Adj-RIB-Out should show what is sent to the peer. bgpd did not fully do that since it adjusted the ASPATH and the nexthop afterwards when building the actual UPDATE. This diff changes that and moves the ASPATH prepend for ebgp sessions and the selection of the nexthop. Thanks to this the outpu

Re: amdgpu(4): use DRM_INFO instead of printf for printing compute unit info

2020-12-03 Thread Charlie Burnett
; > On Sun, Nov 29, 2020 at 11:25 PM Jonathan Gray wrote: > > > > > On Sun, Nov 29, 2020 at 10:17:22PM -0600, Charlie Burnett wrote: > > > > Howdy all, > > > > For reasons that are beyond me, when printf is called in > amdgpu_device.c > > >

Re: amdgpu(4): use DRM_INFO instead of printf for printing compute unit info

2020-11-30 Thread Jonathan Gray
ote: > > > Howdy all, > > > For reasons that are beyond me, when printf is called in amdgpu_device.c > > to > > > print the CU info, it gives me a psp firmware load failure on a Radeon > > VII > > > (Vega 20) gpu. Switching the printf statement to a DRM_

Re: amdgpu(4): use DRM_INFO instead of printf for printing compute unit info

2020-11-29 Thread Charlie Burnett
Howdy all, > > For reasons that are beyond me, when printf is called in amdgpu_device.c > to > > print the CU info, it gives me a psp firmware load failure on a Radeon > VII > > (Vega 20) gpu. Switching the printf statement to a DRM_INFO statement as > > used in the rest of

Re: amdgpu(4): use DRM_INFO instead of printf for printing compute unit info

2020-11-29 Thread Jonathan Gray
On Sun, Nov 29, 2020 at 10:17:22PM -0600, Charlie Burnett wrote: > Howdy all, > For reasons that are beyond me, when printf is called in amdgpu_device.c to > print the CU info, it gives me a psp firmware load failure on a Radeon VII > (Vega 20) gpu. Switching the printf statement t

amdgpu(4): use DRM_INFO instead of printf for printing compute unit info

2020-11-29 Thread Charlie Burnett
Howdy all, For reasons that are beyond me, when printf is called in amdgpu_device.c to print the CU info, it gives me a psp firmware load failure on a Radeon VII (Vega 20) gpu. Switching the printf statement to a DRM_INFO statement as used in the rest of amdgpu seems to fix it though. ok? diff

Re: pfctl.8: mention hostid and checksum for -s info

2020-07-20 Thread Alexandr Nedvedicky
Hello, On Mon, Jul 20, 2020 at 03:23:41PM +0200, Klemens Nanni wrote: > Getting the checksum with pfctl(8) is either in your finger's muscle > memory or takes guess work as the manual doesn't mention it. > > I grepped the code to see that I need `-s info' with `-v

pfctl.8: mention hostid and checksum for -s info

2020-07-20 Thread Klemens Nanni
Getting the checksum with pfctl(8) is either in your finger's muscle memory or takes guess work as the manual doesn't mention it. I grepped the code to see that I need `-s info' with `-v'. (Setting) hostid is described in pf.conf(5) but pfctl(8) doesn't tell us how to pr

unfinished diff to let pcidump print VPD info

2020-06-07 Thread David Gwynne
I was looking for something in a devices VPD info and got this far before finding out it wasn't there. I'm a bit over it now though. PCI VPD stuff is weird because the capability doesn't contain the data itself, it's a register that lets you communicate with whatever is sto

Re: powerpc64: Target Info in clang for __OpenBSD__

2020-05-19 Thread Dale Rahn
On Tue, May 19, 2020 at 11:36:23PM +0200, Patrick Wildt wrote: > Hi, > > drahn@ was complaining to me that his cross-compiler wasn't defining > __OpenBSD__ or __ELF__, and I think the fix is pretty simple. We're > just missing a case in a switch-case. > > The .cpp file itself still compiles, but

Re: powerpc64: Target Info in clang for __OpenBSD__

2020-05-19 Thread Mark Kettenis
> Date: Tue, 19 May 2020 23:36:23 +0200 > From: Patrick Wildt > > Hi, > > drahn@ was complaining to me that his cross-compiler wasn't defining > __OpenBSD__ or __ELF__, and I think the fix is pretty simple. We're > just missing a case in a switch-case. > > The .cpp file itself still compiles,

powerpc64: Target Info in clang for __OpenBSD__

2020-05-19 Thread Patrick Wildt
Hi, drahn@ was complaining to me that his cross-compiler wasn't defining __OpenBSD__ or __ELF__, and I think the fix is pretty simple. We're just missing a case in a switch-case. The .cpp file itself still compiles, but I haven't built a full clang with it. Please give it a go and report back.

make ifconfig print lacp actor and partner info

2019-07-20 Thread David Gwynne
this adds a couple of lines to the lacp output so you can see what params are used on the wire. this can help if you've put switchports in different port-channels or such. an example of that is: aggr0: flags=8847 mtu 1500 lladdr fe:e1:ba:d0:35:21 index 10 priority 0 llprio 7

Re: tcpdump: print some more info about md5 auth in ospf

2019-04-11 Thread Alexander Bluhm
On Thu, Apr 11, 2019 at 08:28:37AM +1000, David Gwynne wrote: > seeing the key-id in particular helped me debug a problem here. > > ok? OK bluhm@ > Index: ospf.h > === > RCS file: /cvs/src/usr.sbin/tcpdump/ospf.h,v > retrieving revis

tcpdump: print some more info about md5 auth in ospf

2019-04-10 Thread David Gwynne
seeing the key-id in particular helped me debug a problem here. ok? Index: ospf.h === RCS file: /cvs/src/usr.sbin/tcpdump/ospf.h,v retrieving revision 1.10 diff -u -p -r1.10 ospf.h --- ospf.h 4 Aug 2010 16:47:01 - 1.10

Re: sfp module info and diagnostics

2019-04-09 Thread Hrvoje Popovski
On 8.4.2019. 22:13, Stuart Henderson wrote: > On 2019/04/08 19:55, Hrvoje Popovski wrote: >> On 8.4.2019. 11:33, David Gwynne wrote: >>> this updates the ifconfig part of the diff >> This is great feature... thank you .. >> it would be great if dBm could be exported via snmp :) > You can do this vi

Re: sfp module info and diagnostics

2019-04-09 Thread Hrvoje Popovski
On 8.4.2019. 19:55, Hrvoje Popovski wrote: > On 8.4.2019. 11:33, David Gwynne wrote: >> this updates the ifconfig part of the diff > > This is great feature... thank you .. maybe to put laser wavelength in sff output? ix0 - 1000BASE-LX x3550m4# ifconfig ix0 sff ix0: identifier SFP (03) c

Re: sfp module info and diagnostics

2019-04-08 Thread Stuart Henderson
FWIW (I was interested so put something together), here's a sample diff on top with one method of printing alarms. (I went for the < > checks rather than table 3.18 flag bits, mostly because I didn't fancy the "check again after 100ms" that the latter wanted). --- sff.c.orig Mon Apr 8 22:27:45 2

Re: sfp module info and diagnostics

2019-04-08 Thread Stuart Henderson
On 2019/04/08 19:55, Hrvoje Popovski wrote: > On 8.4.2019. 11:33, David Gwynne wrote: > > this updates the ifconfig part of the diff > > This is great feature... thank you .. > it would be great if dBm could be exported via snmp :) You can do this via net-snmp already: http://sysadvent.blogspot.c

Re: sfp module info and diagnostics

2019-04-08 Thread Sebastian Benoit
Flag = False > SFP+ 6 Voltage Low Warning Flag = False > SFP+ 6 Tx Bias Low Warning Flag = False > SFP+ 6 Tx Power Low Warning Flag= False > SFP+ 6 Rx Power Low Warning Flag= False > > 10GBASE-SR > > > On Mon, Apr 08, 2019 at 02:21:36

Re: sfp module info and diagnostics

2019-04-08 Thread Hrvoje Popovski
On 8.4.2019. 11:33, David Gwynne wrote: > this updates the ifconfig part of the diff This is great feature... thank you .. it would be great if dBm could be exported via snmp :) switch - Dell S4810 ix0 - sfp+ 10GBASE-SR optics ix1 - sfp 1000BASE-SX optics ixl0 - 10G passive DAC cables x3550m4#

Re: sfp module info and diagnostics

2019-04-08 Thread Stuart Henderson
On 2019/04/08 13:57, Stuart Henderson wrote: > Here's a complete diff. Tested on amd64 with ix, working great there. > distrib/special/ifconfig still builds okay, I'll run a full mkr later on > i386/amd64. > > This is extremely useful for the bgpd "target market" ;) oops, here's attempt #2 with t

Re: sfp module info and diagnostics

2019-04-08 Thread Stuart Henderson
Here's a complete diff. Tested on amd64 with ix, working great there. distrib/special/ifconfig still builds okay, I'll run a full mkr later on i386/amd64. This is extremely useful for the bgpd "target market" ;) Index: sbin/ifconfig/Makefile ===

Re: sfp module info and diagnostics

2019-04-08 Thread David Gwynne
wrote: > this adds support to ifconfig for reading info from transceivers. > > it looks like this: > > dlg@ix ifconfig$ sudo ./obj/ifconfig ix0 transceiver > ix0: identifier SFP (03) > connector: Copper Pigtail (21) > vendor: Amphenol > product: 61

sfp module info and diagnostics

2019-04-07 Thread David Gwynne
this adds support to ifconfig for reading info from transceivers. it looks like this: dlg@ix ifconfig$ sudo ./obj/ifconfig ix0 transceiver ix0: identifier SFP (03) connector: Copper Pigtail (21) vendor: Amphenol product: 616740001 revision: B serial

Re: Patch for install64.octeon : EdgeRouter 6 info

2018-12-21 Thread Visa Hankala
On Thu, Dec 20, 2018 at 07:39:44PM -0500, Chris McGee wrote: > That looks much better to me, though it does get machine-specific > > I feel like it would be more clear if the examples uniformly used your > new variable > convention, as some do and some do not. This allows us to eliminate one > ex

Re: Patch for install64.octeon : EdgeRouter 6 info

2018-12-21 Thread Janne Johansson
Den fre 21 dec. 2018 kl 01:40 skrev Chris McGee : > > I feel like it would be more clear if the examples uniformly used your > new variable > convention, as some do and some do not. This allows us to eliminate one > example > too. (I assume the ER Lite does OK if you specify numcores=1 ? ) "does

Re: Patch for install64.octeon : EdgeRouter 6 info

2018-12-20 Thread Chris McGee
octlinux rootdev=sd0 numcores= + usb reset; fatload usb 0 ${loadaddr} bsd; bootoctlinux rootdev=sd0 numcores=2 OpenBSDCongratulations On Tue, Dec 18, 2018 at 11:40 AM Visa Hankala wrote: > > On Mon, Dec 17, 2018 at 11:22:40PM -0500, Chris McGee wrote: > > Hi: > > > >

Re: Patch for install64.octeon : EdgeRouter 6 info

2018-12-18 Thread Visa Hankala
On Mon, Dec 17, 2018 at 11:22:40PM -0500, Chris McGee wrote: > Hi: > > I would like to add some info for Edgerouter 6 > (and presumably ER4, and maybe also ER12?) to install64.octeon. > The document is great but it won't get a new user booting on the new > 4-core m

Re: Patch for install64.octeon : EdgeRouter 6 info

2018-12-18 Thread Stefan Sperling
On Mon, Dec 17, 2018 at 11:22:40PM -0500, Chris McGee wrote: > so for example it mentions that you're going to need to drop > bsd.mp into the msdos kernel loader partition but doesn't explain how > to do that. Seemed to be the right level of detail for this document. The installer is supposed to p

Patch for install64.octeon : EdgeRouter 6 info

2018-12-17 Thread Chris McGee
Hi: I would like to add some info for Edgerouter 6 (and presumably ER4, and maybe also ER12?) to install64.octeon. The document is great but it won't get a new user booting on the new 4-core machines with MMC drives. I tried to make it as brief as possible while pointing the user in the

Re: [diff] httpd: pass through basic tls peer info to fcgi

2018-07-09 Thread Jack Burton
On Sat, 26 May 2018 22:44:56 +0930 Jack Burton wrote: > This diff adds three new TLS_PEER_* fastcgi variables: ISSUER, SERIAL > & SUBJECT. > > That seems the logical next step in making tls client certs usable for > authorisation & accounting. Last week's commit (thanks Joel!) took > care of auth

Looking for logitech webcams testers/device info

2018-06-29 Thread Landry Breuil
Hi, sending this to a wider audience on misc@, to fix the microphone (cf https://marc.info/?t=15298427072&r=1&w=2) on a variety of logitech webcams (mostly the Cxxx{,HD}?) i'd need the lsusb -v output for the corresponding devices. If you have a logitech webcam where the mic doesnt work (look

[diff] httpd: pass through basic tls peer info to fcgi

2018-05-26 Thread Jack Burton
This diff adds three new TLS_PEER_* fastcgi variables: ISSUER, SERIAL & SUBJECT. That seems the logical next step in making tls client certs usable for authorisation & accounting. Last week's commit (thanks Joel!) took care of authentication. The cert subject on its own should suffice for the sim

query usb device info at device creation

2018-04-25 Thread Landry Breuil
Hi, prompted by mpi@ i tried simplifying the way we get the vendor/product/serial info from the device. Currently this is done lazily, and sometimes several times. Instead of this, let's query the device in usbd_new_device(), fallback to the global known lists then to the USB IDs, cache the

strip exception handling info for armv7 kernels

2017-10-26 Thread Mark Kettenis
Similar to what we do on other architectures with .eh_frame. Reduces the growth when compiling kernels with clang. Index: arch/arm/conf/ldscript.tail === RCS file: /cvs/src/sys/arch/arm/conf/ldscript.tail,v retrieving revision 1.4 di

Re: [patch] hostname.if5 additional info on point to point addressing

2017-10-13 Thread Joe Holden
On 12/10/2017 17:04, Tom Smyth wrote: Hi Ingo, First of all thanks for the feedback it is appreciated, especially when you think the thread is a waste of time. so rather than look for feedback on another patch for now If you could bear with me and let me outline why I think this thread is impo

Re: [patch] hostname.if5 additional info on point to point addressing

2017-10-12 Thread Tom Smyth
Hi Ingo, First of all thanks for the feedback it is appreciated, especially when you think the thread is a waste of time. so rather than look for feedback on another patch for now If you could bear with me and let me outline why I think this thread is important important enough to be my first pr

Re: [patch] hostname.if5 additional info on point to point addressing

2017-10-12 Thread Ingo Schwarze
Hi Tom, it is still completely unclear what you are even trying to talk about. The patch is certainly not acceptable, it is vague, highly confusing, and fails to state what it is all about. If what you are trying to document is the so-called "IP unnumbered" hack: That is an abomination brewed b

Re: [patch] hostname.if5 additional info on point to point addressing

2017-10-11 Thread Tom Smyth
Hello Stuart, all, Thanks for the corrections Stuart, I have corrected the patch to take into account your suggestions and I hope this proposed patch is more correct and useful Index: src/share/man/man5/hostname.if.5 === RCS file: /cvs

Re: efiboot: Reuse GOP info struct

2017-10-06 Thread Klemens Nanni
Updated diff below now that the first bits got committed. gopi can be used safely instead of an extra info struct, so munge it. Declaring the gop and gopi strucutures globally makes things easier in preparation for the next commit. Both changes also improve consistency with regard to other

Re: [patch] hostname.if5 additional info on point to point addressing

2017-10-02 Thread Stuart Henderson
On 2017/10/02 03:04, Tom Smyth wrote: > Hello, > > But the Ip configuration syntax in hostname.if is the same. For a /31 you just use e.g. "inet 192.0.2.100/31" (and it works properly in other parts of the system, e.g. ospfd). > Is there anything specifically wrong in the proposed patch ? This

Re: [patch] hostname.if5 additional info on point to point addressing

2017-10-01 Thread Tom Smyth
Hello, But the Ip configuration syntax in hostname.if is the same. (appart from a /31 having a sequential ip address pair that starts on an even numbered ip) while a point to point / ip unumbered setup would have any arbitary pair of ips on the interface. Is there anything specifically wrong in th

Re: [patch] hostname.if5 additional info on point to point addressing

2017-10-01 Thread Stuart Henderson
On 2017/10/01 19:18, Tom Smyth wrote: > so the point to point addressing scheme is for saving ips sometimes > it can be referred to incorrectly in my opinion as /31 addressing It's totally different to /31. > (well it is more like 2x /32 addresses) but it can be a What you're suggesting is mor

Re: [patch] hostname.if5 additional info on point to point addressing

2017-10-01 Thread Tom Smyth
the hostname.if file and hey presto your link works comments suggestions and criticisms welcome Thanks On 24 September 2017 at 13:12, Tom Smyth wrote: > Hello lads, and ladies, > I have included some extra info on point to point addressing on > interfaces in OpenBSD thanks @tedu for

[patch] hostname.if5 additional info on point to point addressing

2017-09-24 Thread Tom Smyth
Hello lads, and ladies, I have included some extra info on point to point addressing on interfaces in OpenBSD thanks @tedu for the blog post that helpd me learn how to do point to point addressing (non Broadcast) on Openbsd and @theo @ingo for pointing me in the right direction on man page

efiboot: Reuse GOP info struct

2017-09-16 Thread Klemens Nanni
gopi is only used for filling ei after info's last usage. info is more descriptive imho and reusing it gets rid of the extra variable. Feedback? diff --git a/sys/arch/amd64/stand/efiboot/efiboot.c b/sys/arch/amd64/stand/efiboot/efiboot.c index 42e79fbea6b..1ed2968bffa 100644 --- a/sys

Re: signal info code SEGV_ACCERR

2017-07-13 Thread Mark Kettenis
> Date: Thu, 13 Jul 2017 14:54:41 +0200 > From: Alexander Bluhm > > Hi, > > The regress test src/regress/sys/kern/siginfo-fault checks wether > the si_code is set to SEGV_ACCERR after memory access with wrong > permissions has triggert a SIGSEGV. > > Relevant commit message of the test is: >

signal info code SEGV_ACCERR

2017-07-13 Thread Alexander Bluhm
Hi, The regress test src/regress/sys/kern/siginfo-fault checks wether the si_code is set to SEGV_ACCERR after memory access with wrong permissions has triggert a SIGSEGV. Relevant commit message of the test is: According to POSIX, SIGSEGV should specify SEGV_ACCERR if the memory pages are

Re: tcpdump: enable some more bgp info

2017-05-30 Thread Claudio Jeker
On Tue, May 30, 2017 at 11:06:10AM +0200, Michal Mazurek wrote: > On 10:43:30, 30.05.17, Job Snijders wrote: > > In the registry created by RFC 6608, the value "0" is the BGP Finite > > State Machine Error subcode meaning "Unspecified Error". I think that > > when a name is assigned to a value, the

Re: tcpdump: enable some more bgp info

2017-05-30 Thread Michal Mazurek
On 10:43:30, 30.05.17, Job Snijders wrote: > In the registry created by RFC 6608, the value "0" is the BGP Finite > State Machine Error subcode meaning "Unspecified Error". I think that > when a name is assigned to a value, the name should be printed (like > your patch does for subcode values 1, 2,

Re: tcpdump: enable some more bgp info

2017-05-30 Thread Job Snijders
On Tue, May 30, 2017 at 10:21:17AM +0200, Michal Mazurek wrote: > On 12:15:06, 29.05.17, Job Snijders wrote: > > perhaps add a comment like /* RFC 6608 */ above the below: > > Right, it will make it more consistent. > > > > +static const char *bgpnotify_minor_fsm[] = { > > > + NULL, "In OpenSent

Re: tcpdump: enable some more bgp info

2017-05-30 Thread Claudio Jeker
On Tue, May 30, 2017 at 10:21:17AM +0200, Michal Mazurek wrote: > On 12:15:06, 29.05.17, Job Snijders wrote: > > perhaps add a comment like /* RFC 6608 */ above the below: > > Right, it will make it more consistent. > > > > +static const char *bgpnotify_minor_fsm[] = { > > > + NULL, "In OpenSent

Re: tcpdump: enable some more bgp info

2017-05-30 Thread Peter Hessler
On 2017 May 30 (Tue) at 10:21:17 +0200 (+0200), Michal Mazurek wrote: :On 12:15:06, 29.05.17, Job Snijders wrote: :> perhaps add a comment like /* RFC 6608 */ above the below: : :Right, it will make it more consistent. : :> > +static const char *bgpnotify_minor_fsm[] = { :> > + NULL, "In OpenSent

Re: tcpdump: enable some more bgp info

2017-05-30 Thread Theo de Raadt
> > > +static const char *bgpnotify_minor_fsm[] = { > > > + NULL, "In OpenSent State", "In OpenConfirm State", > > > + "In Established State", > > > +}; > > > > and maybe s/NULL/"Unspecified Error"/ > > If it's NULL, then tcpdump will print out the number: > > if (p == NULL) { >

Re: tcpdump: enable some more bgp info

2017-05-30 Thread Michal Mazurek
On 12:15:06, 29.05.17, Job Snijders wrote: > perhaps add a comment like /* RFC 6608 */ above the below: Right, it will make it more consistent. > > +static const char *bgpnotify_minor_fsm[] = { > > + NULL, "In OpenSent State", "In OpenConfirm State", > > + "In Established State", > > +}; > >

Re: tcpdump: enable some more bgp info

2017-05-29 Thread Job Snijders
On Mon, May 29, 2017 at 12:02:33PM +0200, Michal Mazurek wrote: > The error information for bgp was commited in 2009 > (bgpnotify_minor_cease, bgpnotify_minor_cap) but never enabled, so do > that here. Also add FSM error codes. perhaps add a comment like /* RFC 6608 */ above the below: > +static

tcpdump: enable some more bgp info

2017-05-29 Thread Michal Mazurek
The error information for bgp was commited in 2009 (bgpnotify_minor_cease, bgpnotify_minor_cap) but never enabled, so do that here. Also add FSM error codes. Index: print-bgp.c === RCS file: /cvs/src/usr.sbin/tcpdump/print-bgp.c,v ret

Re: delete pf states by exact flow info and speed it up

2017-04-23 Thread Alexandr Nedvedicky
Hello, On Sun, Apr 23, 2017 at 12:18:07AM +0200, Hiltjo Posthuma wrote: > On Sat, Apr 22, 2017 at 08:14:06AM +0900, YASUOKA Masahiko wrote: > > On Fri, 21 Apr 2017 13:52:51 +0900 (JST) > > YASUOKA Masahiko wrote: > > > +int > > > +pfctl_parse_host(char *str, struct pf_rule_addr *addr) > > > +{ >

Re: delete pf states by exact flow info and speed it up

2017-04-23 Thread YASUOKA Masahiko
On Sun, 23 Apr 2017 00:18:07 +0200 Hiltjo Posthuma wrote: > On Sat, Apr 22, 2017 at 08:14:06AM +0900, YASUOKA Masahiko wrote: >> On Fri, 21 Apr 2017 13:52:51 +0900 (JST) >> YASUOKA Masahiko wrote: >> > +int >> > +pfctl_parse_host(char *str, struct pf_rule_addr *addr) >> > +{ >> (snip) >> > + if

Re: delete pf states by exact flow info and speed it up

2017-04-22 Thread Hiltjo Posthuma
On Sat, Apr 22, 2017 at 08:14:06AM +0900, YASUOKA Masahiko wrote: > On Fri, 21 Apr 2017 13:52:51 +0900 (JST) > YASUOKA Masahiko wrote: > > +int > > +pfctl_parse_host(char *str, struct pf_rule_addr *addr) > > +{ > (snip) > > + if ((sbs = strchr(s, '[')) != NULL || (sbe = strrchr(s, ']')) != NULL)

Re: delete pf states by exact flow info and speed it up

2017-04-21 Thread Alexandr Nedvedicky
On Sat, Apr 22, 2017 at 08:14:06AM +0900, YASUOKA Masahiko wrote: > On Fri, 21 Apr 2017 13:52:51 +0900 (JST) > YASUOKA Masahiko wrote: > > +int > > +pfctl_parse_host(char *str, struct pf_rule_addr *addr) > > +{ > (snip) > > + if ((sbs = strchr(s, '[')) != NULL || (sbe = strrchr(s, ']')) != NULL)

Re: delete pf states by exact flow info and speed it up

2017-04-21 Thread YASUOKA Masahiko
On Fri, 21 Apr 2017 13:52:51 +0900 (JST) YASUOKA Masahiko wrote: > +int > +pfctl_parse_host(char *str, struct pf_rule_addr *addr) > +{ (snip) > + if ((sbs = strchr(s, '[')) != NULL || (sbe = strrchr(s, ']')) != NULL) { > + hints.ai_family = AF_INET6; > + *(sbs++) = *sbe

Re: delete pf states by exact flow info and speed it up

2017-04-20 Thread YASUOKA Masahiko
Hi, On Fri, 21 Apr 2017 00:27:04 +0200 Alexandr Nedvedicky wrote: > I finally got to your patch. I'm fine with your to extend pfctl kill > operation. > > I've found just few nits in your change. > > 1) your manpage diff should include one more change below: > 8<---8<---

Re: delete pf states by exact flow info and speed it up

2017-04-20 Thread Alexandr Nedvedicky
Hello, I finally got to your patch. I'm fine with your to extend pfctl kill operation. I've found just few nits in your change. 1) your manpage diff should include one more change below: 8<---8<---8<--8< diff -r c435e977886a -r 40aa0af6

delete pf states by exact flow info and speed it up

2017-04-17 Thread YASUOKA Masahiko
Hi, I'd like to add "key" modifier to "pfctl -k" to delete pf states, like below: # pfctl -k key -k 'tcp 10.0.0.1:80 <- 10.0.0.101:32123' This deletion will be very fast if the RB tree of pf state table is used to find the state. The diff also includes the diff for that part (DIOCKILLSTATES o

dhcrelay(8): support replacing Relay Agent Info

2016-12-15 Thread Rafael Zalamena
This diff adds dhcrelay support for replacing Relay Agent Information on incoming packets with our configuration. This might be useful if you want to override user sent information or to adapt the incoming packet to another network. ok? Index: dhcrelay.8 ==

Re: texinfo: /usr/share/info/dir permissions

2016-11-16 Thread Theo Buehler
- > @@ -60,6 +60,7 @@ install:maninstall > sh ${.CURDIR}/util/gen-dir-node ${DESTDIR}/usr/share/info > \ > ${DESTDIR}/usr/share/info/dir > chown ${MANOWN}:${MANGRP} ${DESTDIR}/usr/share/info/dir > + chmod ${MANMODE} ${DESTDIR}/usr/share/info/dir > > clean cleandir: > rm -f ${CLEANFILES} >

  1   2   >