summaryrefslogtreecommitdiffstats
path: root/usr.sbin (follow)
Commit message (Collapse)AuthorAgeFilesLines
...
* Apply same cleanup to the control code as in ospfd.claudio2021-01-194-13/+14
| | | | Internalize control_state and ctl_conns.
* Make the struct imsgev static. With this ospf6d compiles with -fno-commonclaudio2021-01-193-9/+9
|
* Use ospfe_imsg_compose_rde() instead of exporting iev_rde and usingclaudio2021-01-192-10/+5
| | | | | imsg_compose_event() directly. This is needed to make the imsg event structures static in ospfe.c, rde.c and ospf6d.c.
* Like in ospfd rename nconf to noeconf in the ospf engine to removeclaudio2021-01-191-10/+10
| | | | a common symbol.
* Properly define ospfd_process as an extern in the header file.claudio2021-01-192-4/+6
|
* Like in ospfd use a static pkt_ptr buffer.claudio2021-01-193-9/+4
|
* Do the same control cleanup as in ospfd. Move control_state and ctl_connsclaudio2021-01-194-17/+19
| | | | into control.c nothing outside needs access to this.
* Make the imsg event structures static, properly define ospfd_process andclaudio2021-01-194-21/+23
| | | | | rename nconf in the ospf engine to noeconf. This fixes the last common symbols in ospfd.
* Internalize a few more things into the control code. Both control_stateclaudio2021-01-194-17/+18
| | | | and ctl_conns are not really needed outside of control.c
* Use a static pkt_ptr buffer instead of allocating it dynamically.claudio2021-01-193-9/+4
| | | | Removes another -fno-common issue from the build.
* Use ospfe_imsg_compose_rde() instead of calling imsg_compose_event() withclaudio2021-01-192-11/+5
| | | | | a local export of iev_rde. Remove the extern in preperation of -fno-common cleanup.
* Do the KAME embedded scope fixup in the two places where getifaddrs() isclaudio2021-01-194-37/+29
| | | | | | | used. With this there should be no more embedded scopes left and therefor in6addr_to_text() can be removed. getnameinfo() will just do the right thing now. OK eric@
* Fix the embedded scope hack. They way this was written is buggy since theclaudio2021-01-191-15/+10
| | | | | | | else does not skip enough of the original code. Instead use a temporary sockaddr pointer and adjust it to point to the sin6 one if an embedded scope was detected. OK jca@
* Fix httpd example configtb2021-01-181-1/+19
| | | | | | | | | | | | | | | | | | | A server configuration yields multiple struct server_config in the env->sc_servers queue: an actual server ("parent") and one for each location directive in httpd.conf. In parent_configure(), the parents are configured first, then the corresponding locations. parse.y r1.118 kills the parent if an error is encountered on loading the TLS keypairs but leaves the locations in the sc_servers queue. When running the default config without TLS keys already in place, this then leads to the self-explanatory "invalid location" error message. Its intention is to indicates the unexpected situation that config_setserver() encounters a location without an associated server. Fix this by not only destroying the parent but also removing all of its locations. ok jsing
* Remove unused variable in an effort to make this -fno-common clean.florian2021-01-181-7/+1
|
* Adjust code to the change of struct bgpd_addr in bgpd.claudio2021-01-181-5/+5
| | | | OK denis@
* Change struct bgpd_addr VPN encoding. Instead of including two almostclaudio2021-01-186-243/+144
| | | | | | | | | equal versions put the RD and lable stack right into struct bgpd_addr. For non-VPN addresses these extra fields are ignored. Since VPN and non-VPN addresses encode the prefix in the same way now some code can be simplified. In most cases a fallthrough or reuse of encoding functions is now possible. It should also reduce the size of struct bgpd_addr a bit. OK denis@
* Don't leak host address. Found with clang static analyzer.rob2021-01-171-1/+2
| | | | OK tb@
* Comply with man page intent of -dvv enabling BER level logging.rob2021-01-171-3/+3
| | | | Tweak and ok martijn@
* Put the clearing of possible embedded scope in an __KAME__ block.claudio2021-01-171-2/+7
| | | | | | This code does not really need to be optional since for non embedded scope link local addresses this is a NOP. OK jca@
* Put the clearing of possible embedded scope in an __KAME__ block.claudio2021-01-171-2/+7
| | | | | | This code does not really need to be optional since for non embedded scope link local addresses this is a NOP. OK jca@
* Remove a __KAME__ block for extracting the scope_id from IPV6_PKTINFO.claudio2021-01-171-3/+1
| | | | | | struct in6_pktinfo includes the interface index in ipi6_ifindex but no struct sockaddr_in6. OK jca@
* Remove a __KAME__ block for extracting the scope_id from IPV6_PKTINFO.claudio2021-01-171-3/+1
| | | | | | struct in6_pktinfo includes the interface index in ipi6_ifindex but no struct sockaddr_in6. OK jca@
* revert the previous, it makes everything think it can do 16.0 GT/s.dlg2021-01-171-4/+2
| | | | i'll have to find real pci4 hardware to test with.
* pcie4 can do 16.0 GT/s. sounds like a lot.dlg2021-01-171-2/+4
|
* dump the max payload size and max read request size in the pcie cap.dlg2021-01-171-3/+36
|
* aspath_neighbor() needs to return the local system AS for empty AS pathsclaudio2021-01-161-3/+8
| | | | | | | but also for AS paths starting with an AS_SET segment. RFC4271 wants this and it also makes sense for all the cases where aspath_neighbor() is used in bgpd. OK denis@ job@
* Use struct ip_mreqn for the IP_MULTICAST_IF setsockopt. This wayclaudio2021-01-161-2/+8
| | | | | | | ospfd will pick the right interface to send out hello packets. Clear the struct ip_mreqn in all cases because not all fields are used. OK bluhm@ phessler@
* Stop the -v command line argument from overriding logging brevity.rob2021-01-151-4/+4
| | | | Tweaked and OK claudio@
* Cleanup prefix_cmp() a bit. Make sure that the return value can not overflowclaudio2021-01-141-28/+60
| | | | | | | the int type by doing calculations on bigger types. Instead just do a > and < check. Also improve the remote_addr test by using the same address comparison as in other places. OK benno@
* Extend prefix_evaluate() to also be used when withdrawing a prefix.claudio2021-01-134-40/+33
| | | | | | | | | | | | Doing the LIST_REMOVE() outside of prefix_evalute() is no longer valid. As a benefit it is now simply possible to re-evaluate a prefix by passing it to prefix_evaluate() for both removal and insertion. prefix_evaluate() will then take care to ensure that a update is sent out if necessary. Also move rde_send_kroute() call to rde_generate_updates() to make it a bit easier to plug this module into a regress test. OK denis@
* In recv_packet() verify that non-multicast packets where sent to the rightclaudio2021-01-121-16/+11
| | | | | | | | | | | | IPv6 address. Compare against all possible interface addresses instead of only against iface->addr which is set to the last link-local address and so there is a high chance of failure when multiple link-local addresses are present. Also simplify send_packet(), there is no need to use sendmsg() just use sendto(). OK benno@
* Send lsupdate messages to the correct neighbor address on P2P interfaces.claudio2021-01-121-2/+2
| | | | | | iface->dst is not initalized so instead use nbr->addr (like it is done in ospfd). OK benno@
* rsync is using buffered output now, so remove this FIXME commentclaudio2021-01-121-3/+1
|
* Use the interface index of struct ip_mreqn in IP_ADD_MEMBERSHIP to sendclaudio2021-01-121-5/+5
| | | | | out packets. This way ospfd can work on interfaces that share the same IP. OK dlg@
* - grab fullpkgpath early (to decide: hard error if not provided) ?espie2021-01-111-6/+16
| | | | | | | | | | | | | | | - tweak the look-up mechanism for dependencies to NOT look at installed packages for inter-dependencies between subpackages (and debug packages) of the SAME pkgpath. The rationale for looking at installed packages in the usual case is that it's normally much faster rather than have the ports tree spew out packing-lists. In the case of multi-packages updates, the new version may have been built successfully (and correctly) while the old version is still around. However, if pkg_create looks at the installed version, it may fail to create the package because the shared library versions will be different.
* Stop deleting the control socket on daemon shutdown, like we did on othermestre2021-01-111-2/+1
| | | | | | | | | daemons. This avoids giving an additional permission (in this case unveil(2) "c") to the daemon just to be able to delete the socket and we already discussed in the past that leaving that file behind doesn't cause any problems. Discussed with deraadt@ rob@ florian@ OK rob@ benno@ deraadt@
* document how to specify multiple alternative names;jmc2021-01-111-3/+5
| | | | | modified version of diff from wolf on misc, improved by and ok florian benno sthen
* Exclude the 'hz' token from the lexer backdoor, so interval and profilejmatthew2021-01-101-4/+8
| | | | | | probes can be parsed again. ok anton@ kn@
* Move peer related checks from up_test_update() to rde_generate_updates()claudio2021-01-092-37/+18
| | | | | | | | | both the export check and the address family check should be done as early as possible since these peers will not need any kind of updates to happen. Also remove the non-standard ORIGINATOR_ID check. With this up_test_update() becomes a simple true/false function which makes the rest of the code a bit simpler. OK benno@
* Remove locally imposed limit on sockets.rob2021-01-092-20/+14
| | | | Suggested by martijn@, ok claudio@
* Add 'strip' directivedenis2021-01-095-18/+76
| | | | | | Feedback by Olivier Cherrier, Hiltjo Posthuma, Mischa OK benno@
* accept_reserve() counter function argument doesn't need to be volatile.millert2021-01-081-4/+3
| | | | | Revision 1.21 removed the volatile from the cgi_inflight global but accept_reserve() was not updated to match. OK florian@
* avoid extra warnings if undefined values are encounteredespie2021-01-081-2/+4
|
* Simplify the entity queue processing. There is no need to keep the entityclaudio2021-01-081-74/+42
| | | | | | | | queue element around once it is sent to the parser process. Instead just keep a count of pending entities and make the parser send back the entity type instead of the id (since this is the only field needed to figure out what to do). OK job@
* Start using the ibuf API (ibuf_dynamic, ibuf_add, ibuf_close) for writingclaudio2021-01-0810-238/+246
| | | | | | | data between processes. This completely decouples the write side. rpki-client can't really use the imsg framework but it can use the ibuf bits wich imsg is built on. OK benno@ job@
* bump VM shutdown event timeout ok mlarkin@ stsp@ florian@tracey2021-01-071-2/+2
| | | | | VMs with addition package daemons were not given enough time to shutdown gracefully.
* Fix multiple shift/reduce conflicts in the parser because of the dependonclaudio2021-01-071-13/+15
| | | | | | | | | | | | | synatx element with its optional / empty element. While this is needed for redistribute statements it causes the conflicts in the interface sections since there the statement can not be empty (this conflicts with empty new lines handled in optnl). Add a dependonopt for redistribute statements and make depend on no longer optional. Also sync the curly bracket blocks (interface, area, ...) to be like set blocks and the blocks in bgpd. This should result in more flexible newline handling. Tested by sthen@, benno@, Kapetanakis Giannis OK benno@
* if getting the vpd data fails, don't try and parse the vpd data.dlg2021-01-061-2/+4
| | | | pointed out by jsg@
* double word fix; from sean daviesjmc2021-01-061-3/+3
|
Copyright © 1996 – 2023 Jason A. Donenfeld. All Rights Reverse Engineered.