| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
| |
old bugs, as well as some unwanted side effects. I will do more
investigation and fix these issues first.
Pointed out by: dwmalone
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Teach inetd(8) about kqueue, originally implemented by jmg@[1].
- Use new C99 style function prototypes instead of K&Rs.
- Raise WARNS from 2 to 6
Glanced at by: ru
MFC After: 2 weeks
[1] http://people.freebsd.org/~jmg/inetd.kq.patch,
http://people.freebsd.org/~jmg/inetd.kq.html
|
|
|
|
|
|
|
| |
them for reading. When user can open file for reading, he can also
flock(2) it, which can lead to confusions.
Pointed out by: green
|
| |
|
|
|
|
|
| |
NI_WITHSCOPEID, and our getaddrinfo(3) does nothing special
for it, now.
|
|
|
|
| |
Approved by: dwmalone
|
|
|
|
| |
(with permission of addtional copyright holders where appropriate)
|
|
|
|
|
|
|
| |
services database. Now only services that are actually used need to be
defined.
Submitted by: ume
|
|
|
|
|
|
|
| |
not NULL.
Submitted by: Stefan Farfeleder <stefan@fafoe.narf.at>
MFC after: 3 days
|
|
|
|
|
|
|
|
|
|
|
| |
exactly the same as patch from the PR, which also exited if the
config file was missing. I didn't use Jeff's patch because I was
worried that some people might start inetd, create the config file
and then HUP inetd.
PR: 60806
Submitted by: Jeff Ito <jeffi@rcn.com>
MFC after: 2 weeks
|
|
|
|
| |
variable.
|
|
|
|
|
|
| |
use IPV6_RECVPKTINFO instead.
Reported by: someone (I had removed the mail wrongly, sorry)
|
|
|
|
|
| |
to log data. Clean up an unused variable that was hidden by the WARNS?=2
being commented out. Uncomment it now that it compiles cleanly again.
|
|
|
|
|
|
|
|
|
|
|
| |
(or possibly testing) the previous formula worked for the default
constants compiled into inetd, but if you recompiled with different
values of CHTSIZE and CHTGRAN the calculation might not have worked.
PR: 54354
Submitted by: Claus Assmann <ca@sendmail.org>
Submitted by: Jose Marcio Martins da Cruz <Jose-Marcio.Martins@ensmp.fr>
MFC after: 5 days
|
|
|
|
|
|
|
|
|
|
| |
into a child process. Rather than closing the discriptors manually,
mark all discriptors as close-on-exec.
PR: 47694
Submitted by: Max Okumoto <okumoto@ucsd.edu>
Obtained from: NetBSD
MFC after: 2 weeks
|
|
|
|
|
|
|
|
|
| |
we don't leak memory. Only one of these two cases (reconfig) actually
causes a leak because the other is usually followed by an exec.
PR: 46845
Reviewed by: David Wang <dsw@juniper.net>
MFC after: 2 weeks
|
|
|
|
|
|
|
| |
cases, the `struct sockaddr' will not be allocated.
Reported by: nectar
MFC after: 2 days
|
|
|
|
|
|
|
|
|
|
| |
rather than specifically setting the process priority and resource class;
otherwise, we improperly set other aspects of the login class. We have
a bit more to do here, but the proper fix will probably involve breaking
out MAC labels from the login class at some point, as well as further
clarifying the logic here.
Pointed out by: kuriyama, max
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
with a class, rather than all aspects of the class when switching
classes for an inetd service. Because we hard-code /daemon in the
current inetd implementation, using SETALL has unfortunate side-effects
involving the MAC code, and potentially other credential related
settings in the future. This change maintains the DoS-resistent
aspects of the class behavior, which is all that is promised in the
inetd man page.
A larger set of diffs providing more pluggability and configurability
was deferred for this more simple approach in the short term.
Reviewed by: ache
Obtained from: TrustedBSD Project
Sponsored by: DARPA, Network Associates Laboratories
|
| |
|
|
|
|
|
|
| |
PR: 41912
Submitted by: Aaron Smith <aaron@mutex.org>
MFC after: 2 weeks
|
|
|
|
|
|
|
| |
this avoids some double negatives which are a bit difficult to
parse.
Always tread v[46]bind{,_ok} as booleans.
|
|
|
|
|
| |
PR: 40771
Submitted by: Jean-Luc Richier <Jean-Luc.Richier@imag.fr>
|
|
|
|
|
|
|
|
|
|
|
| |
declared - it was bad style and caused a bug. v[46]bind need to be
reset whenever we go to the "more:" label.
Jean-Luc and I came up with this patch independently, so it had
better be right!
PR: 40771
Submitted by: Jean-Luc Richier <Jean-Luc.Richier@imag.fr>
|
|
|
|
|
| |
PR: 40771
Submitted by: Jean-Luc Richier <Jean-Luc.Richier@imag.fr>
|
|
|
|
|
|
| |
rename a parameter to avoid shadowing a global).
MFC after: 1 month
|
|
|
|
|
|
|
|
|
| |
invocations of each service from a single IP address.
Requested by: matusita
Reviewed by: dwmalone
Tested by: matusita on snapshots.jp.FreeBSD.org
MFC after: 2 weeks
|
|
|
|
| |
MFC after: 1 week
|
|
|
|
| |
Submitted by: Jean-Luc Richier <Jean-Luc.Richier@imag.fr>
|
|
|
|
|
|
|
|
|
| |
is appropriate to avoid using typeof/__typeof__. It is worth noting that
SWAP() is only ever used to swap pointer values so 'void *' assumptions would
have been acceptable, but I'd gladly pay you tuesday for a cheeseburger^W
cleaner interface today.
Poked into submission by: bde
|
| |
|
| |
|
| |
|
|
|
|
| |
Built standalone, inetd(8) is WARNS=5 clean, WARNS=6 if you ignore %m fits.
|
| |
|
| |
|
|
|
|
|
|
|
| |
the terminal(-d fix from dwmalone).
Approved by: dwmalone
MFC after: 2 weeks
|
| |
|
|
|
|
|
| |
Requested by: matusita
Reviewed by: matusita
|
|
|
|
|
|
|
|
| |
o Mark unused variables.
o Set WARNS?=2
o Results in no code changes.
Submitted by: Mike Barcroft <mike@q9media.com>
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
how to use this feature are in the man page. This is based on work
by Lyndon Nerenberg.
(The only difficult part about this patch is the fact that you
can't fchown a unix domain socket, which means the sockets must be
put in a secure directory).
Reviewed by: dillon
|
|
|
|
|
|
|
| |
to have been accidently imported when ipv6 support was added to
inetd.
Approved by: ume
|
|
|
|
|
|
|
|
| |
it already, their syntax is not compatible with ours. It will confuse
users. So, we have compatibility with their syntex.
Approved by: dwmalone
Obtained from: NetBSD
|
|
|
|
|
|
|
| |
instead of ints, don't cast to char *, clear up some remote name
handling code which had become a little odd.
Should result in no functional changes.
|
|
|
|
|
|
|
| |
doesn't actually use it.
PR: 24307
Submitted by: opentrax@email.com
|
|
|
|
|
|
|
| |
make sure there is exactly one prototype for each function,
use K&R style definitions everywhere to match dominant style,
make flag_signal take an int to avoid problems if we have
ANSI prototypes and K&R definitions.
|
|
|
|
|
| |
PR: 22268
Submitted by: Daniel S. Lewart <d-lewart@uiuc.edu>
|
|
|
|
|
|
| |
loops.
Submitted by: Ian Dowse <iedowse@maths.tcd.ie>
|
|
|
|
|
|
| |
PR: 21650
Submitted by: ben
Tested by: dan@ducky.nz.freebsd.org
|