Commits
- Commit:
a065ee49d9a6aa6e1798b970f010a744f7b83762- From:
- rsadowski <rsadowski@openbsd.org>
- Date:
Fix relayd reload race crash
Instead of terminating with fatalx() when a private key hash cannot be
found, log a warning and send an error back to the relay worker.
This prevents a race condition during "reload" where a request might
reach the CA process while keys are being repopulated.
Reported by Nick Owens; thanks!
OK tb@
- Commit:
802f4c96216ce8ba445b0fa0a1be479daa94db54- From:
- tb <tb@openbsd.org>
- Date:
relayd: replace unneeded engine.h with needed x509.h
Adjust a comment. Being non-existent, the OpenSSL engine layer cannot be
responsible for whatever unholy hacks this code requires.
- Commit:
18abd06bbeb3591e9b0d2b04d656879d2a33e186- From:
- rsadowski <rsadowski@openbsd.org>
- Date:
Set User-Agent for HTTP healthchecks
Joel Carnat (Thanks) notice that GoToSocial does not like it when we sent
no User-Agent and returns an HTTP/418.
Lloyd pointed to use RELAYD_SERVERNAME instead hardcoded "relayd"
OK sthen, claudio (diff without RELAYD_SERVERNAME)
- Commit:
499b552cbb56e2d6bee51512f4a3f109f4616662- From:
- tb <tb@openbsd.org>
- Date:
relayd/log.h: missed cvs add
- Commit:
6b5c4000434cc5dc67e202d8aede2f543b5b2bf3- From:
- rsadowski <rsadowski@openbsd.org>
- Date:
Standardize logging with bgpd
OK claudio@
- Commit:
7d3002f34b21f1b776e623940190ca244b453503- From:
- rsadowski <rsadowski@openbsd.org>
- Date:
imsg_composev() / imsgbuf_flush() handle return values
OK claudio@
- Commit:
3ec811e2b4b4c300c5e15e1b61cc363f0efadc40- From:
- rsadowski <rsadowski@openbsd.org>
- Date:
fix memory leak in rsae_send_imsg
If the cookie doesn't match, we bail with a continue and totally forget to
free the imsg.
OK claudio@
- Commit:
80347df32a3fd8cd66aeaf059fbfd16c95a2ffe7- From:
- kirill <kirill@openbsd.org>
- Date:
usr.sbin/relayd: add support for PROXY protocol in TCP relays
patch from Christoph Liebender
OK: rsadowski@
- Commit:
4f8b63696d9758baaed91352d95e5d1084674c8b- From:
- schwarze <schwarze@openbsd.org>
- Date:
Slightly iprove a confusing wording in the parse.y manuals:
The things that need quoting are not necessarily "argument names",
and not even necessarily "names" at all, so just talk about "arguments".
"I guess?" florian@ and no objection from otto@, both back in July 2025.
Actually, the quoting rules are more complicated than the text makes
believe, but i do not know how to better describe them. It may not be
easy because some suspect the implementation may be somewhat adhoc
rather than based on cleary defined lexical rules.
- Commit:
4111100134779c504b2e09d76dd2b79569a5a550- From:
- rsadowski <rsadowski@openbsd.org>
- Date:
fix scan-build dead stores findings
OK stsp
- Commit:
158279cc4bb747491259166e29f248ce32e3d6b4- From:
- rsadowski <rsadowski@openbsd.org>
- Date:
fix missing initialisation
It is possible that "request_method" is checked without having been
initialised in line 439.
OK kirill@ stsp@
- Commit:
80854f0233cbd6df2ed2ea07a17d910fb988a59d- From:
- rsadowski <rsadowski@openbsd.org>
- Date:
relayd: fix dead store and unindent carp_demote_init
Found by scan-build:
carp.c:64:7: warning: Although the value stored to 'c' is used in the
enclosing expression, the value is never actually read from 'c'
[deadcode.DeadSt ores]
64 | if ((c = carp_group_find(group)) == NULL)
Feedback from Crystal Kolipe and tb@, OK tb@
- Commit:
952c015b38f9d4dcaf68a269c4018a920f35eb5f- From:
- tb <tb@openbsd.org>
- Date:
relayd: fix relay_http_time() to emit GMT times again
The HTTP standard RFC 9110 requires GMT, in HTTP-date. We used to do this
until a recent modification to localtime.c changed GMT to UTC.
sync from httpd
- Commit:
37603cbd06eaaba2570979c140c6c94d51279148- From:
- deraadt <deraadt@openbsd.org>
- Date:
For IMSG_BINDANY, bnd.bnd_proc wasn't range checked to ensure it is positive.
As a result IF the other side of the privsep was succesfully exploited, it
could then send such a flawed message and cause a cause an array bounds
violation over the privsep boundary.
Reported by S. Ai, H. Lefeuvre, Systopia team
ok claudio
- Commit:
9b58f4a4879445d21104976317b2d25c8743c98a- From:
- schwarze <schwarze@openbsd.org>
- Date:
Make internal hyperlinking work by moving custom sections from .Sh to .Ss
and the titles from all caps to sentence case such that they match the
table of contents, and switch from .Sy to .Sx as needed.
OK florian@
- Commit:
abfa295f9a51b0479be9e96ab74f6ead7e650100- From:
- schwarze <schwarze@openbsd.org>
- Date:
In the manual pages for configuration files based on parse.y, describe
the syntax of both defining and using macros, rather than exclusively
relying on examples, which some of the pages do not even provide.
In those pages containing tables of content, also clarify that
the "Macros" section contains *definitions* of variables.
Both changes were already committed to vm.conf(5) earlier.
In those few pages that referenced cpp(1) and m4(1), stop doing that
because the macro definition syntax and the macro dereferencing syntax
of both languages is totally different from the parse.y syntax.
OK florian@, and deraadt also requests keeping these manuals in sync.
- Commit:
21645604d9bdeecb445a3c36eb1b1e997bf0801e- From:
- claudio <claudio@openbsd.org>
- Date:
Ensure that string buffers are '\0' terminated when handling them in
the privileged parent.
Again code does not expect strings that are not terminated and by default
nothing sends such strings but lets fix this bug anyway.
Reported by S. Ai, H. Lefeuvre, Systopia team
OK tb@ benno@
- Commit:
f30efb9a7b86a2912c9176374b21e3268cbd1fe2- From:
- martijn <martijn@openbsd.org>
- Date:
unveil the agentx socket path. This was probably broken with adding unix
sockets to unveil.
reported by pascal@
OK sthen@ benno@
- Commit:
418ab141e444db9b7f3d7c2afc4351f7ef4bccfc- From:
- claudio <claudio@openbsd.org>
- Date:
Convert various reyk proc.c daemons over to new imsgbuf_init and
imsgbuf_allow_fdpass.
OK tb@
- Commit:
a523d270f0229581bdb138c7324f3f7379f381b5- From:
- claudio <claudio@openbsd.org>
- Date:
Convert the common imsgbuf_read calls to the post EAGAIN world.
OK tb@
- Commit:
4fa30ed3b5f0dc15e8df1354524856babc892291- From:
- claudio <claudio@openbsd.org>
- Date:
Use imsgbuf_queuelen() instead of accessing the w.queue member.
OK tb@
- Commit:
4e3b971d1bd7f46fd63873b5cc1733984b0e24dc- From:
- claudio <claudio@openbsd.org>
- Date:
Use imsgbuf_clear() where appropriate instead of msgbuf_clear().
OK tb@
- Commit:
585eb3254921938d12e233e6029f136bc0f8b987- From:
- claudio <claudio@openbsd.org>
- Date:
Rename imsg_init, imsg_clear, imsg_read, imsg_write and imsg_flush to
imsgbuf_init, imsgbuf_clear, imsgbuf_read, imsgbuf_write and imsgbuf_flush.
This separates the imsgbuf API from the per-imsg API.
OK tb@
- Commit:
41ddc9d163020f85bddcaf4c7261ab2818848c59- From:
- claudio <claudio@openbsd.org>
- Date:
Convert imsg_write() callers to the new simplified return logic.
OK tb@
- Commit:
1be2558a99f45bf2af156906cd98ba2a7a97198d- From:
- claudio <claudio@openbsd.org>
- Date:
Introduce imsg_write() and use it instead of msgbuf_write().
imsg_write() is just a thin wrapper around msgbuf_write(). So this is
mostly search and replace.
OK tb@
